{"action":"created","issue":{"url":"https://api.github.com/repos/cefsharp/CefSharp/issues/1073","labels_url":"https://api.github.com/repos/cefsharp/CefSharp/issues/1073/labels{/name}","comments_url":"https://api.github.com/repos/cefsharp/CefSharp/issues/1073/comments","events_url":"https://api.github.com/repos/cefsharp/CefSharp/issues/1073/events","html_url":"https://github.com/cefsharp/CefSharp/issues/1073","id":87191239,"number":1073,"title":"New API Review","user":{"login":"amaitland","id":307872,"avatar_url":"https://avatars.githubusercontent.com/u/307872?v=3","gravatar_id":"","url":"https://api.github.com/users/amaitland","html_url":"https://github.com/amaitland","followers_url":"https://api.github.com/users/amaitland/followers","following_url":"https://api.github.com/users/amaitland/following{/other_user}","gists_url":"https://api.github.com/users/amaitland/gists{/gist_id}","starred_url":"https://api.github.com/users/amaitland/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amaitland/subscriptions","organizations_url":"https://api.github.com/users/amaitland/orgs","repos_url":"https://api.github.com/users/amaitland/repos","events_url":"https://api.github.com/users/amaitland/events{/privacy}","received_events_url":"https://api.github.com/users/amaitland/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/cefsharp/CefSharp/milestones/12","html_url":"https://github.com/cefsharp/CefSharp/milestones/43.0.0","labels_url":"https://api.github.com/repos/cefsharp/CefSharp/milestones/12/labels","id":934305,"number":12,"title":"43.0.0","description":"Based on 2357 branch of CEF\r\nThis is a major rewrite of `CefSharp` and will include many breaking changes","creator":{"login":"jornh","id":766423,"avatar_url":"https://avatars.githubusercontent.com/u/766423?v=3","gravatar_id":"","url":"https://api.github.com/users/jornh","html_url":"https://github.com/jornh","followers_url":"https://api.github.com/users/jornh/followers","following_url":"https://api.github.com/users/jornh/following{/other_user}","gists_url":"https://api.github.com/users/jornh/gists{/gist_id}","starred_url":"https://api.github.com/users/jornh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jornh/subscriptions","organizations_url":"https://api.github.com/users/jornh/orgs","repos_url":"https://api.github.com/users/jornh/repos","events_url":"https://api.github.com/users/jornh/events{/privacy}","received_events_url":"https://api.github.com/users/jornh/received_events","type":"User","site_admin":false},"open_issues":22,"closed_issues":43,"state":"open","created_at":"2015-01-13T23:37:19Z","updated_at":"2015-06-11T03:04:51Z","due_on":null,"closed_at":null},"comments":1,"created_at":"2015-06-11T03:04:46Z","updated_at":"2015-06-11T16:53:01Z","closed_at":null,"body":"Work on the new `API` is fairly well progressed (thanks in large part to @rassilon). Everyone will soon have access to the underlying `CefBrowser`, `CefFrame` and `CefBrowser` host instances. Also previously hidden callbacks will now be available for `async` use.\r\n\r\nOf particular note is the new `IPopupHandler` interface, which provides access to features that were previously routed directly into the `ChromiumWebBrowser` instance, most of which were ignored when the browser was a popup.\r\n\r\nhttps://github.com/cefsharp/CefSharp/blob/master/CefSharp/IPopupHandler.cs\r\n\r\nYou'll see that there are quite a few duplicate methods like `OnPreKeyEvent`, `OnKeyEvent`, `OnResourceRedirect`, `OnBeforeResourceLoad`, etc.\r\n\r\nIs that confusing to have the duplicates? Would it be better to only expose the methods that are hidden?\r\n\r\n@jankurianski @jornh @perlun @rassilon Any comments? Thoughts?"},"comment":{"url":"https://api.github.com/repos/cefsharp/CefSharp/issues/comments/111205356","html_url":"https://github.com/cefsharp/CefSharp/issues/1073#issuecomment-111205356","issue_url":"https://api.github.com/repos/cefsharp/CefSharp/issues/1073","id":111205356,"user":{"login":"jornh","id":766423,"avatar_url":"https://avatars.githubusercontent.com/u/766423?v=3","gravatar_id":"","url":"https://api.github.com/users/jornh","html_url":"https://github.com/jornh","followers_url":"https://api.github.com/users/jornh/followers","following_url":"https://api.github.com/users/jornh/following{/other_user}","gists_url":"https://api.github.com/users/jornh/gists{/gist_id}","starred_url":"https://api.github.com/users/jornh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jornh/subscriptions","organizations_url":"https://api.github.com/users/jornh/orgs","repos_url":"https://api.github.com/users/jornh/repos","events_url":"https://api.github.com/users/jornh/events{/privacy}","received_events_url":"https://api.github.com/users/jornh/received_events","type":"User","site_admin":false},"created_at":"2015-06-11T16:53:01Z","updated_at":"2015-06-11T16:53:01Z","body":"Sorry, haven't been able to look much at CefSharp lately, let alone use it. From your description above those additions sound great though :+1: Excellent work by you and @rassilon! \r\n\r\n> Everyone will soon have access to the underlying CefBrowser, CefFrame and CefBrowser host instances.\r\n\r\nCan't wait to see what the dual `CefBrowser`s can do for me... couldn't  help it ... sure that's just a typo in there :smiling_imp:\r\n\r\nI don't see having the duplicate methods in the `IPopupHandler` hurts anyone. Easier accessible and probably won't be that much of a maintenance burden - now that it's there. Do whatever you feel works best!\r\n"}}