{"action":"created","issue":{"url":"https://api.github.com/repos/cefsharp/CefSharp/issues/787","labels_url":"https://api.github.com/repos/cefsharp/CefSharp/issues/787/labels{/name}","comments_url":"https://api.github.com/repos/cefsharp/CefSharp/issues/787/comments","events_url":"https://api.github.com/repos/cefsharp/CefSharp/issues/787/events","html_url":"https://github.com/cefsharp/CefSharp/pull/787","id":56487545,"number":787,"title":"#672: Notify CEF when our owning form moves (WM_MOVE or WM_MOVING) just ...","user":{"login":"rassilon","id":698352,"avatar_url":"https://avatars.githubusercontent.com/u/698352?v=3","gravatar_id":"","url":"https://api.github.com/users/rassilon","html_url":"https://github.com/rassilon","followers_url":"https://api.github.com/users/rassilon/followers","following_url":"https://api.github.com/users/rassilon/following{/other_user}","gists_url":"https://api.github.com/users/rassilon/gists{/gist_id}","starred_url":"https://api.github.com/users/rassilon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rassilon/subscriptions","organizations_url":"https://api.github.com/users/rassilon/orgs","repos_url":"https://api.github.com/users/rassilon/repos","events_url":"https://api.github.com/users/rassilon/events{/privacy}","received_events_url":"https://api.github.com/users/rassilon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-02-04T06:48:40Z","updated_at":"2015-02-05T00:53:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cefsharp/CefSharp/pulls/787","html_url":"https://github.com/cefsharp/CefSharp/pull/787","diff_url":"https://github.com/cefsharp/CefSharp/pull/787.diff","patch_url":"https://github.com/cefsharp/CefSharp/pull/787.patch"},"body":"WM_MOVE and WM_MOVING only get sent to parent windows, so add a MovingListener that attaches to the control's parent form and sends the notifications to CEF.\r\n\r\nFyi,\r\nBill\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/cefsharp/CefSharp/issues/comments/72973828","html_url":"https://github.com/cefsharp/CefSharp/pull/787#issuecomment-72973828","issue_url":"https://api.github.com/repos/cefsharp/CefSharp/issues/787","id":72973828,"user":{"login":"rassilon","id":698352,"avatar_url":"https://avatars.githubusercontent.com/u/698352?v=3","gravatar_id":"","url":"https://api.github.com/users/rassilon","html_url":"https://github.com/rassilon","followers_url":"https://api.github.com/users/rassilon/followers","following_url":"https://api.github.com/users/rassilon/following{/other_user}","gists_url":"https://api.github.com/users/rassilon/gists{/gist_id}","starred_url":"https://api.github.com/users/rassilon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rassilon/subscriptions","organizations_url":"https://api.github.com/users/rassilon/orgs","repos_url":"https://api.github.com/users/rassilon/repos","events_url":"https://api.github.com/users/rassilon/events{/privacy}","received_events_url":"https://api.github.com/users/rassilon/received_events","type":"User","site_admin":false},"created_at":"2015-02-05T00:53:31Z","updated_at":"2015-02-05T00:53:31Z","body":"> If your interested then the related ones are #702 #774\r\n\r\nI'll check those out next. I don't think it'll matter for my use of CEFSharp (WinForms full screen, just the browser control, nothing else), but I could give it a go.\r\n\r\n> Partly code duplication, potentially both require a `NativeWindow` to hook a handle. \r\n> As a reusable piece of code it's better to have one that's generic enough to be used by both. \r\n> At it's core, it simply needs to forward message to `ChromiumWebBrowser`\r\n\r\nAh, after reading those two issues I see what you mean. `NativeWindow` is designed to only hook one **`HWNDs`** `WndProc` at a time.\r\n\r\nIf a `NativeWindow` subclass is necessary for the focus issue, it'd probably be in a separate subclass. (The logic would probably be slightly different, esp. if we need to hook a HWND inside CEF/Chromium.)\r\n\r\n> My point was that this is fixable in user land as it stands currently. \r\n> For most I imagine WM_MOVE would be enough, overriding WndProc on the users form and \r\n> implementing both is feasible.\r\n\r\nOh, completely. No argument there. It'd just be awesome if this is one of the details you don't have to worry about. :)\r\n\r\n> Appreciate the effort you've put in, and I'm hopeful that we can merge this soon, \r\n> just need to take a step back, look at the bigger picture.\r\n\r\nOh, not a problem at all. I've had code submissions to dead projects just rot on the vine, you're doing an amazing job keeping on top of this stuff.\r\n\r\nThanks again,\r\nBill\r\n"}}