Make window.scroll properties relative to the layout viewport.

Make window.scroll properties relative to the layout viewport.

window.scroll properties being relative to visual viewport breaks some pages
under pinch-zoom. The original plan was to have all APIs reflect the layout
viewport (and introduce new APIs for visual). This CL is an initial step
towards that and the window.scroll properties will be relative to the layout
viewport if the 'inert-visual-viewport' flag is set.

BUG=489206
Committed: https://crrev.com/9b92c13df0e3c771a89a5beb6ea892704fd2ed05
Cr-Commit-Position: refs/heads/master@{#351104}

[ymalik, 2015-09-24 14:43:03]

ymalik@chromium.org changed reviewers:
+ bokan@chromium.org, rbyers@chromium.org, ymalik@google.com

[ymalik, 2015-09-24 14:43:50]

Just applied the patch after the blink repo merge. 

Will need another LGTM :)

[bokan, 2015-09-24 14:44:35]

lgtm

[Rick Byers, 2015-09-24 14:55:19]

RS lgtm, but please add a link to the original codereview (here in review
comments is fine) in case someone wants to follow the review history.

[ymalik, 2015-09-24 14:59:13]

On 2015/09/24 14:55:19, Rick Byers wrote:
> RS lgtm, but please add a link to the original codereview (here in review
> comments is fine) in case someone wants to follow the review history.

Original review: 
https://codereview.chromium.org/1356113002/

[ymalik, 2015-09-24 19:59:42]

@bokan PTAL, I added the chromium side changes to this CL.

[bokan, 2015-09-25 01:42:27]

still lgtm (you'll need new OWNERs for the Chromium side changes though)

[ymalik, 2015-09-25 14:19:50]

ymalik@chromium.org changed reviewers:
+ avi@chromium.org, kenrb@chromium.org

[ymalik, 2015-09-25 14:20:24]

+kenrb@ for common_param_traits_macros
+avi@ for content/public and content/renderer content/browser

[ymalik, 2015-09-25 14:21:45]

ymalik@chromium.org changed reviewers:
+ isherman@chromium.org

[ymalik, 2015-09-25 14:22:25]

+isherman@ for histograms.xml

[Avi (use Gerrit), 2015-09-25 14:28:42]

:/

https://codereview.chromium.org/1366893002/diff/20001/content/browser/rendere...
File content/browser/renderer_host/render_process_host_impl.cc (right):

https://codereview.chromium.org/1366893002/diff/20001/content/browser/rendere...
content/browser/renderer_host/render_process_host_impl.cc:1335:
switches::kInertVisualViewport,
If the switches in a list are in alphabetical order, please follow suit.

[ymalik, 2015-09-25 14:36:41]

https://codereview.chromium.org/1366893002/diff/20001/content/browser/rendere...
File content/browser/renderer_host/render_process_host_impl.cc (right):

https://codereview.chromium.org/1366893002/diff/20001/content/browser/rendere...
content/browser/renderer_host/render_process_host_impl.cc:1335:
switches::kInertVisualViewport,
On 2015/09/25 14:28:41, Avi wrote:
> If the switches in a list are in alphabetical order, please follow suit.

My apologies for missing the detail. Done.

[kenrb, 2015-09-25 14:56:39]

ipc lgtm

[Avi (use Gerrit), 2015-09-25 17:00:13]

lgtm

[Ilya Sherman, 2015-09-26 00:19:51]

histograms.xml lgtm

[ymalik, 2015-09-27 19:43:51]

The CQ bit was checked by ymalik@chromium.org

[ymalik, 2015-09-27 19:43:51]

The patchset sent to the CQ was uploaded after l-g-t-m from rbyers@chromium.org,
bokan@chromium.org
Link to the patchset: https://codereview.chromium.org/1366893002/#ps40001
(title: ""worked on review comments"")

[commit-bot: I haz the power, 2015-09-27 19:44:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1366893002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1366893002/40001

[commit-bot: I haz the power, 2015-09-27 19:59:56]

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

[commit-bot: I haz the power, 2015-09-27 19:59:57]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ymalik, 2015-09-28 13:56:04]

The CQ bit was checked by ymalik@chromium.org

[commit-bot: I haz the power, 2015-09-28 13:56:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1366893002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1366893002/40001

[commit-bot: I haz the power, 2015-09-28 14:16:36]

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

[commit-bot: I haz the power, 2015-09-28 14:16:38]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[ymalik, 2015-09-28 16:44:24]

The CQ bit was checked by ymalik@chromium.org

[commit-bot: I haz the power, 2015-09-28 16:44:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1366893002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1366893002/40001

[commit-bot: I haz the power, 2015-09-28 19:23:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-28 19:26:39]

Patchset 3 (id:??) landed as
https://crrev.com/9b92c13df0e3c771a89a5beb6ea892704fd2ed05
Cr-Commit-Position: refs/heads/master@{#351104}