Push inverse device scale factor in DRLI->DRLI pushes

Push inverse device scale factor in DRLI->DRLI pushes

DelegatedRendererLayerImpl computes an inverse device scale factor when
receiving frame data so it can map from the delegating renderer's physical
space to DIP space and then remap the quads from its renderer's DIP space
to physical again. When using impl side painting, the frame is received
by the DRLI on the pending tree. We need to push this factor to the active
tree when pushing the passes so we can produce the correct quads from the
active tree.

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

[danakj, 2014-07-09 21:12:57]

LGTM thanks for the test

https://codereview.chromium.org/385433002/diff/1/cc/layers/delegated_renderer...
File cc/layers/delegated_renderer_layer_impl_unittest.cc (right):

https://codereview.chromium.org/385433002/diff/1/cc/layers/delegated_renderer...
cc/layers/delegated_renderer_layer_impl_unittest.cc:1544: DCHECK_EQ(0.5f,
delegated_renderer_layer_impl->inverse_device_scale_factor());
EXPECT_EQ

https://codereview.chromium.org/385433002/diff/1/cc/layers/delegated_renderer...
cc/layers/delegated_renderer_layer_impl_unittest.cc:1551: DCHECK_EQ(0.5f,
other_layer->inverse_device_scale_factor());
EXPECT_EQ

[jamesr, 2014-07-09 21:38:00]

On 2014/07/09 21:12:57, danakj wrote:
> LGTM thanks for the test
> 
>
https://codereview.chromium.org/385433002/diff/1/cc/layers/delegated_renderer...
> File cc/layers/delegated_renderer_layer_impl_unittest.cc (right):
> 
>
https://codereview.chromium.org/385433002/diff/1/cc/layers/delegated_renderer...
> cc/layers/delegated_renderer_layer_impl_unittest.cc:1544: DCHECK_EQ(0.5f,
> delegated_renderer_layer_impl->inverse_device_scale_factor());
> EXPECT_EQ
> 
>
https://codereview.chromium.org/385433002/diff/1/cc/layers/delegated_renderer...
> cc/layers/delegated_renderer_layer_impl_unittest.cc:1551: DCHECK_EQ(0.5f,
> other_layer->inverse_device_scale_factor());
> EXPECT_EQ

Whoops!

[jamesr, 2014-07-09 22:17:42]

The CQ bit was checked by jamesr@chromium.org

[commit-bot: I haz the power, 2014-07-09 22:19:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jamesr@chromium.org/385433002/20001

[commit-bot: I haz the power, 2014-07-10 00:54:31]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[commit-bot: I haz the power, 2014-07-10 01:22:57]

Change committed as 282208