Adds visual viewport size override to DevTools Emulation protocol.

(Obsolete.)

Adds visual viewport size override to DevTools Emulation protocol.

This enables resizing of the visual viewport for (headless)
screenshots. For more information, see crbug.com/625577
and the design doc: bit.ly/viewport-screenshots.

We will add additional overrides for scroll/scale of viewports in
crrev.com/2096633002.

BUG=625577

[Eric Seckler, 2016-07-22 12:20:40]

The CQ bit was checked by eseckler@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-22 12:20:52]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Eric Seckler, 2016-07-22 12:56:20]

Description was changed from

==========
Adds visual viewport size override to DevTools Emulation protocol.

BUG=625577
==========

to

==========
Adds visual viewport size override to DevTools Emulation protocol.

This enables resizing of the visual viewport for (headless)
screenshots. For more information, see crbug.com/625577
and the design doc: bit.ly/viewport-screenshots.

We will add additional overrides for scroll/scale of viewports in
crrev.com/2096633002.

BUG=625577
==========

[commit-bot: I haz the power, 2016-07-22 13:41:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-22 13:41:02]

Dry run: This issue passed the CQ dry run.

[Eric Seckler, 2016-07-22 14:42:39]

eseckler@chromium.org changed reviewers:
+ dgozman@chromium.org

[Eric Seckler, 2016-07-22 14:42:40]

Split off from crrev.com/2096633002.

[dgozman, 2016-07-25 22:13:54]

lgtm. Thanks for splitting this off.

https://codereview.chromium.org/2173783002/diff/1/third_party/WebKit/public/w...
File third_party/WebKit/public/web/WebDeviceEmulationParams.h (right):

https://codereview.chromium.org/2173783002/diff/1/third_party/WebKit/public/w...
third_party/WebKit/public/web/WebDeviceEmulationParams.h:56: // Emulated visual
viewport size. |x == 0| or |y == 0| disables the override
nit: x, y -> width, height

[Eric Seckler, 2016-07-26 08:33:49]

eseckler@chromium.org changed reviewers:
+ dcheng@chromium.org, pfeldman@chromium.org

[Eric Seckler, 2016-07-26 08:33:50]

+dcheng for ipc change (view_messages.h).
+pfeldman for content/ and WebKit/public/.

Thanks!

https://codereview.chromium.org/2173783002/diff/1/third_party/WebKit/public/w...
File third_party/WebKit/public/web/WebDeviceEmulationParams.h (right):

https://codereview.chromium.org/2173783002/diff/1/third_party/WebKit/public/w...
third_party/WebKit/public/web/WebDeviceEmulationParams.h:56: // Emulated visual
viewport size. |x == 0| or |y == 0| disables the override
On 2016/07/25 22:13:54, dgozman wrote:
> nit: x, y -> width, height

Done.

[dcheng, 2016-07-26 11:05:02]

LGTM

[Eric Seckler, 2016-09-23 09:56:32]

Description was changed from

==========
Adds visual viewport size override to DevTools Emulation protocol.

This enables resizing of the visual viewport for (headless)
screenshots. For more information, see crbug.com/625577
and the design doc: bit.ly/viewport-screenshots.

We will add additional overrides for scroll/scale of viewports in
crrev.com/2096633002.

BUG=625577
==========

to

==========
(Obsolete.)

Adds visual viewport size override to DevTools Emulation protocol.

This enables resizing of the visual viewport for (headless)
screenshots. For more information, see crbug.com/625577
and the design doc: bit.ly/viewport-screenshots.

We will add additional overrides for scroll/scale of viewports in
crrev.com/2096633002.

BUG=625577
==========