cc: Clean up transform tree

cc: Clean up transform tree

This CL deletes:
1) surface contents scale on transfrom tree
    TransformNode::needs_surface_contents_scale,
    TransformNode::surface_contents_scale,
    TransformTree::UpdateSurfaceContentsScale,
    DrawPropertyUtils::VerifySurfaceContentsScalesMatch
2) Dead code
    DrawPropertyUtils::VerifyTransformTreeCalculations,
    TransformTree::UpdateTargetSpaceTransform,
    TransformCachedNodeData::(from_target, to_target)
3) Redundant stuff in PropertyTrees
    TransfromTree::(ToTarget, FromTarget, SetToTarget, SetFromTarget),
    PropertyTrees::ComputeTransformToTarget,
    TransformTree::ComputeTransform is now used only in tests

BUG=622372
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[jaydasika, 2016-10-26 01:15:17]

Description was changed from

==========
cc: Clean up transform tree

This CL deletes:
1) surface contents scale on transfrom tree
    TransformNode::needs_surface_contents_scale,
    TransformNode::surface_contents_scale,
    TransformTree::UpdateSurfaceContentsScale,
    DrawPropertyUtils::VerifySurfaceContentsScalesMatch
2) Dead code
    DrawPropertyUtils::VerifyTransformTreeCalculations,
    TransformTree::UpdateTargetSpaceTransform,
    TransformCachedNodeData::(from_target, to_target)
3) Redundant stuff in PropertyTrees
    TransfromTree::(ToTarget, FromTarget, SetToTarget, SetFromTarget),
    PropertyTrees::ComputeTransformToTarget,
    TransformTree::ComputeTransform is now used only in tests

BUG=622372
==========

to

==========
cc: Clean up transform tree

This CL deletes:
1) surface contents scale on transfrom tree
    TransformNode::needs_surface_contents_scale,
    TransformNode::surface_contents_scale,
    TransformTree::UpdateSurfaceContentsScale,
    DrawPropertyUtils::VerifySurfaceContentsScalesMatch
2) Dead code
    DrawPropertyUtils::VerifyTransformTreeCalculations,
    TransformTree::UpdateTargetSpaceTransform,
    TransformCachedNodeData::(from_target, to_target)
3) Redundant stuff in PropertyTrees
    TransfromTree::(ToTarget, FromTarget, SetToTarget, SetFromTarget),
    PropertyTrees::ComputeTransformToTarget,
    TransformTree::ComputeTransform is now used only in tests

BUG=622372
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[jaydasika, 2016-10-26 01:15:32]

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

[commit-bot: I haz the power, 2016-10-26 01:15:59]

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

[commit-bot: I haz the power, 2016-10-26 01:27:10]

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

[commit-bot: I haz the power, 2016-10-26 01:27:11]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[jaydasika, 2016-10-26 01:28:29]

jaydasika@chromium.org changed reviewers:
+ ajuma@chromium.org, weiliangc@chromium.org

[jaydasika, 2016-10-26 01:28:30]

PTAL

sunxd : Most of this is possible because of your patch. If you already have a
WIP patch on similar lines, let me know, we can coordinate.

[jaydasika, 2016-10-26 01:49:16]

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

[commit-bot: I haz the power, 2016-10-26 01:50:02]

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

[commit-bot: I haz the power, 2016-10-26 02:13:33]

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

[commit-bot: I haz the power, 2016-10-26 02:13:34]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jaydasika, 2016-10-26 03:11:11]

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

[commit-bot: I haz the power, 2016-10-26 03:11:31]

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

[commit-bot: I haz the power, 2016-10-26 05:24:02]

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

[commit-bot: I haz the power, 2016-10-26 05:24:03]

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

[sunxd, 2016-10-26 14:17:49]

On 2016/10/26 01:28:30, jaydasika wrote:
> PTAL
> 
> sunxd : Most of this is possible because of your patch. If you already have a
> WIP patch on similar lines, let me know, we can coordinate.

No problem, I have switched to tiled mask after the draw transform stuff, so
please feel free to continue the detargeting work.

[weiliangc, 2016-10-26 17:38:45]

LGTM

[jaydasika, 2016-10-26 17:40:44]

jaydasika@chromium.org changed reviewers:
+ pdr@chromium.org

[jaydasika, 2016-10-26 17:40:45]

pdr: Please review changes in PaintArtifactCompositor

[pdr., 2016-10-26 17:43:29]

On 2016/10/26 at 17:40:45, jaydasika wrote:
> pdr: Please review changes in PaintArtifactCompositor

LGTM

[jaydasika, 2016-10-26 17:44:02]

The CQ bit was checked by jaydasika@chromium.org

[commit-bot: I haz the power, 2016-10-26 17:44:39]

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

[commit-bot: I haz the power, 2016-10-26 18:50:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-26 18:53:42]

Description was changed from

==========
cc: Clean up transform tree

This CL deletes:
1) surface contents scale on transfrom tree
    TransformNode::needs_surface_contents_scale,
    TransformNode::surface_contents_scale,
    TransformTree::UpdateSurfaceContentsScale,
    DrawPropertyUtils::VerifySurfaceContentsScalesMatch
2) Dead code
    DrawPropertyUtils::VerifyTransformTreeCalculations,
    TransformTree::UpdateTargetSpaceTransform,
    TransformCachedNodeData::(from_target, to_target)
3) Redundant stuff in PropertyTrees
    TransfromTree::(ToTarget, FromTarget, SetToTarget, SetFromTarget),
    PropertyTrees::ComputeTransformToTarget,
    TransformTree::ComputeTransform is now used only in tests

BUG=622372
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Clean up transform tree

This CL deletes:
1) surface contents scale on transfrom tree
    TransformNode::needs_surface_contents_scale,
    TransformNode::surface_contents_scale,
    TransformTree::UpdateSurfaceContentsScale,
    DrawPropertyUtils::VerifySurfaceContentsScalesMatch
2) Dead code
    DrawPropertyUtils::VerifyTransformTreeCalculations,
    TransformTree::UpdateTargetSpaceTransform,
    TransformCachedNodeData::(from_target, to_target)
3) Redundant stuff in PropertyTrees
    TransfromTree::(ToTarget, FromTarget, SetToTarget, SetFromTarget),
    PropertyTrees::ComputeTransformToTarget,
    TransformTree::ComputeTransform is now used only in tests

BUG=622372
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/b433552b9bfc3ade27dfe7a4b95496f609ba4c35
Cr-Commit-Position: refs/heads/master@{#427763}
==========

[commit-bot: I haz the power, 2016-10-26 18:53:43]

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