Make sure Legacy HWND is created and has proper bounds on bounds change.

Make sure Legacy HWND is created and has proper bounds on bounds change.

BUG=436218
Committed: https://crrev.com/ffd2bddecb7349d0595dd87e802e08e37f18dc28
Cr-Commit-Position: refs/heads/master@{#307160}

[dmazzoni, 2014-12-05 18:12:12]

dmazzoni@chromium.org changed reviewers:
+ ananta@chromium.org

[ananta, 2014-12-05 19:58:02]

lgtm

https://codereview.chromium.org/771353004/diff/1/content/browser/web_contents...
File content/browser/web_contents/web_contents_view_aura.cc (right):

https://codereview.chromium.org/771353004/diff/1/content/browser/web_contents...
content/browser/web_contents/web_contents_view_aura.cc:1485: CHECK(parent_hwnd);
Please verify if this does not crash in Windows 8 mode.

[dmazzoni, 2014-12-05 20:56:06]

Thanks! Please check out the other legacy HWND code review I sent, too.

https://codereview.chromium.org/771353004/diff/1/content/browser/web_contents...
File content/browser/web_contents/web_contents_view_aura.cc (right):

https://codereview.chromium.org/771353004/diff/1/content/browser/web_contents...
content/browser/web_contents/web_contents_view_aura.cc:1485: CHECK(parent_hwnd);
On 2014/12/05 19:58:02, ananta wrote:
> Please verify if this does not crash in Windows 8 mode.

Verified manually. This same CHECK appears above too, and I haven't seen any bug
reports from it.

[dmazzoni, 2014-12-05 21:12:24]

dmazzoni@chromium.org changed reviewers:
+ avi@chromium.org

[dmazzoni, 2014-12-05 21:12:24]

+avi for owners review

[Avi (use Gerrit), 2014-12-05 22:03:38]

lgtm

Um, OK?

[dmazzoni, 2014-12-06 05:37:42]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2014-12-06 05:38:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/771353004/1

[commit-bot: I haz the power, 2014-12-06 06:17:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-12-06 06:17:51]

Patchset 1 (id:??) landed as
https://crrev.com/ffd2bddecb7349d0595dd87e802e08e37f18dc28
Cr-Commit-Position: refs/heads/master@{#307160}