Fix SurfaceLayerImpl scale under impl-side painting

Fix SurfaceLayerImpl scale under impl-side painting

With impl-side painting turned on (which is really property trees),
Layer::CalculateContentsScale isn't run and Layer::PushPropertiesTo
clobbers contents scale to 1.f, and content bounds to bounds.

The problem was that the renderer would submit quads at scale=2 and the
browser SurfaceLayerImpl would also see that it had a DSF would then
scale its surface quad.  To fix this, the SurfaceLayerImpl needs to
apply the inverse scale of its surface when generating quads.

Ideally, SurfaceAggregator could be changed in the future to just always
unapply the DSF of the frame and reduce plumbing, but in the short term,
this is a minimal change to ship impl-side painting.

This change is similar to https://codereview.chromium.org/913243003.

BUG=495142
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/c09eb94e92c8ca3698059eb471f71bb64acdf7b9
Cr-Commit-Position: refs/heads/master@{#333004}

[enne (OOO), 2015-06-04 21:22:17]

enne@chromium.org changed reviewers:
+ danakj@chromium.org, weiliangc@chromium.org

[enne (OOO), 2015-06-04 21:22:25]

enne@chromium.org changed reviewers:
+ jbauman@chromium.org

[enne (OOO), 2015-06-04 21:22:25]

enne@chromium.org changed reviewers:
+ jbauman@chromium.org

[danakj, 2015-06-04 21:25:18]

LGTM

https://codereview.chromium.org/1166983007/diff/1/cc/layers/surface_layer_imp...
File cc/layers/surface_layer_impl.cc (right):

https://codereview.chromium.org/1166983007/diff/1/cc/layers/surface_layer_imp...
cc/layers/surface_layer_impl.cc:65: render_pass, content_bounds(),
shared_quad_state, append_quads_data);
i thin this should match the size of the quad.

[weiliangc, 2015-06-04 21:48:22]

Thanks! LGTM

[enne (OOO), 2015-06-04 21:49:39]

The CQ bit was checked by enne@chromium.org

[enne (OOO), 2015-06-04 21:49:39]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org
Link to the patchset: https://codereview.chromium.org/1166983007/#ps20001
(title: "Fix borders")

[commit-bot: I haz the power, 2015-06-04 21:50:41]

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

[commit-bot: I haz the power, 2015-06-04 22:09:59]

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

[commit-bot: I haz the power, 2015-06-04 22:10:00]

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

[enne (OOO), 2015-06-04 22:11:01]

The CQ bit was checked by enne@chromium.org

[commit-bot: I haz the power, 2015-06-04 22:12:22]

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

[commit-bot: I haz the power, 2015-06-04 22:45:44]

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

[commit-bot: I haz the power, 2015-06-04 22:45:46]

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_...)

[enne (OOO), 2015-06-04 22:47:00]

The CQ bit was checked by enne@chromium.org

[enne (OOO), 2015-06-04 22:47:00]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org,
weiliangc@chromium.org
Link to the patchset: https://codereview.chromium.org/1166983007/#ps40001
(title: "Fix SLI test to set scale; remove SL test about content scale")

[commit-bot: I haz the power, 2015-06-04 22:48:21]

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

[commit-bot: I haz the power, 2015-06-04 23:31:54]

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

[commit-bot: I haz the power, 2015-06-04 23:31:55]

Try jobs failed on following builders:
  win8_chromium_rel on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)

[danakj, 2015-06-05 00:42:36]

https://codereview.chromium.org/1166983007/diff/40001/cc/layers/surface_layer...
File cc/layers/surface_layer_impl.cc (right):

https://codereview.chromium.org/1166983007/diff/40001/cc/layers/surface_layer...
cc/layers/surface_layer_impl.cc:15: : LayerImpl(tree_impl, id), surface_id_(-1),
surface_scale_(0.f) {
surface_id is unsigned, but it defaults to 0, you don't need to init it here.

[enne (OOO), 2015-06-05 01:06:39]

The CQ bit was checked by enne@chromium.org

[enne (OOO), 2015-06-05 01:06:40]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org,
weiliangc@chromium.org
Link to the patchset: https://codereview.chromium.org/1166983007/#ps10005
(title: "Windows compile fix OOPS")

[commit-bot: I haz the power, 2015-06-05 01:07:56]

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

[commit-bot: I haz the power, 2015-06-05 03:43:24]

Committed patchset #4 (id:10005)

[commit-bot: I haz the power, 2015-06-05 03:44:09]

Patchset 4 (id:??) landed as
https://crrev.com/c09eb94e92c8ca3698059eb471f71bb64acdf7b9
Cr-Commit-Position: refs/heads/master@{#333004}