[DevTools] Pass inspected page bounds from frontend.

[DevTools] Pass inspected page bounds from frontend.

Passing bounds instead of insets is a more reliable approach in general, but has disadvantages when resizing the whole browser window (as new bounds are only available after round-trip to DevTools frontend).

BUG=373669
TBR=avi@chromium.org // For browser_window_controller_browsertest

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271379

[dgozman, 2014-05-15 09:40:23]

Could you please take a look?

This patch allows more sophisticated positioning inside DevTools frontend
(required for responsive design work):
- seems to have no visible effect on Aura;
- slightly improves DevTools splitter resize on Mac;
- regresses window resize with open DevTools on Mac (still on par with Aura
though);
- has no effect when DevTools is closed.

Follow-up blink-side patch: https://codereview.chromium.org/223803003/.

[ccameron, 2014-05-15 16:36:10]

RWHVMac part seems reasonable for now. There's going to be a bit of churn for
the next few weeks, but after that we should be able to start polishing a
non-moving target WRT resize behavior etc.

[dgozman, 2014-05-15 16:52:54]

On 2014/05/15 16:36:10, ccameron1 wrote:
> RWHVMac part seems reasonable for now. There's going to be a bit of churn for
> the next few weeks, but after that we should be able to start polishing a
> non-moving target WRT resize behavior etc.

Ok, I'll keep an eye on this. I also need your review as an OWNER of RWHVMac.

[pfeldman, 2014-05-15 18:27:19]

lgtm

[ccameron, 2014-05-16 06:15:28]

On 2014/05/15 18:27:19, pfeldman wrote:
> lgtm

Sorry, forgot to write lgtm

[dgozman, 2014-05-16 11:21:38]

Tom, take a look at devtools_embedder_message_dispatcher please.

[Tom Sepez, 2014-05-16 19:54:07]

lgtm

[dgozman, 2014-05-19 08:27:12]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2014-05-19 08:27:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dgozman@chromium.org/221283009/120001

[dgozman, 2014-05-19 11:10:49]

The CQ bit was unchecked by dgozman@chromium.org

[dgozman, 2014-05-19 11:27:27]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2014-05-19 11:27:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dgozman@chromium.org/221283009/140001

[commit-bot: I haz the power, 2014-05-19 14:51:14]

Change committed as 271379