Add a callback from Blink to Chromium for when page scale changes

Add a callback from Blink to Chromium for when page scale changes

This is to allow command-zero to reset page scale (in particular, to
allow Chromium to know when command-zero will have any effect).

BUG=456880
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191241

[ccameron (do not use), 2015-03-03 18:52:39]

ccameron@google.com changed reviewers:
+ ccameron@google.com, dcheng@chromium.org

[ccameron (do not use), 2015-03-03 18:52:39]

dcheng, can you take a look at this -- this is the Blink component of the
following Chromium patch:
https://codereview.chromium.org/954923005/

[dcheng, 2015-03-03 20:04:10]

lgtm

Note that long-term, the plan in content (due to the out-of-process iframes
work) is to get rid of RenderViewImpl. Any thoughts on what an alternative
locatino for this to live is in that world?

[ccameron, 2015-03-03 20:24:56]

Thanks!

Interesting -- is there a doc for the proposed changes that I can take a look
at?

[ccameron, 2015-03-03 20:26:36]

The CQ bit was checked by ccameron@chromium.org

[dcheng, 2015-03-03 20:28:57]

On 2015/03/03 at 20:24:56, ccameron wrote:
> Thanks!
> 
> Interesting -- is there a doc for the proposed changes that I can take a look
at?

There's a bunch of information at the bottom of
http://www.chromium.org/developers/design-documents/site-isolation in the "Site
Isolation Summit 2015" section. The slides will probably be the most helpful for
a quick overview.

jam@ also posted a PSA about RenderView and RenderViewHost here:
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/9qTMXFyPN_g

(I've CCed creis@ who might be able to link in more documents that I've missed)

[commit-bot: I haz the power, 2015-03-03 20:31:58]

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

[commit-bot: I haz the power, 2015-03-04 05:01:11]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191241

[Charlie Reis, 2015-03-04 22:55:14]

On 2015/03/03 20:28:57, dcheng wrote:
> On 2015/03/03 at 20:24:56, ccameron wrote:
> > Thanks!
> > 
> > Interesting -- is there a doc for the proposed changes that I can take a
look
> at?
> 
> There's a bunch of information at the bottom of
> http://www.chromium.org/developers/design-documents/site-isolation in the
"Site
> Isolation Summit 2015" section. The slides will probably be the most helpful
for
> a quick overview.
> 
> jam@ also posted a PSA about RenderView and RenderViewHost here:
>
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/9qTMXFyPN_g
> 
> (I've CCed creis@ who might be able to link in more documents that I've
missed)

Yes, those are the right places to look.  In particular, the Feature Update FAQ
for OOPIF covers it:
https://docs.google.com/document/d/1Iqe_CzFVA6hyxe7h2bUKusxsjB6frXfdAYLerM3Jj...

Thanks!