Remove anchorPoint from WebLayer and GraphicsLayer.

Remove anchorPoint from WebLayer and GraphicsLayer.

TBR=abarth
BUG=372837
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175827

[ojan, 2014-06-05 21:52:22]

lgtm

https://codereview.chromium.org/311273008/diff/1/Source/platform/graphics/Gra...
File Source/platform/graphics/GraphicsLayer.h (right):

https://codereview.chromium.org/311273008/diff/1/Source/platform/graphics/Gra...
Source/platform/graphics/GraphicsLayer.h:342: bool m_transformOriginSet : 1;
This doesn't pack well (wastes 32 bits). Please put it adjacent to the other
bools below.

[ojan, 2014-06-05 21:53:31]

Sorry for the muliptle comments...new code review tool has some bugs.

https://codereview.chromium.org/311273008/diff/1/Source/platform/graphics/Gra...
File Source/platform/graphics/GraphicsLayer.cpp (right):

https://codereview.chromium.org/311273008/diff/1/Source/platform/graphics/Gra...
Source/platform/graphics/GraphicsLayer.cpp:566: if (m_transformOriginSet &&
m_transformOrigin != FloatPoint3D(m_size.width() * 0.5f, m_size.height() * 0.5f,
0)) {
Bikeshed: This soudns like a set of transformOrigin's to me. How about
m_hasTransformOrigin?

[chrishtr, 2014-06-05 22:24:24]

https://codereview.chromium.org/311273008/diff/1/Source/platform/graphics/Gra...
File Source/platform/graphics/GraphicsLayer.cpp (right):

https://codereview.chromium.org/311273008/diff/1/Source/platform/graphics/Gra...
Source/platform/graphics/GraphicsLayer.cpp:566: if (m_transformOriginSet &&
m_transformOrigin != FloatPoint3D(m_size.width() * 0.5f, m_size.height() * 0.5f,
0)) {
On 2014/06/05 21:53:31, ojan wrote:
> Bikeshed: This soudns like a set of transformOrigin's to me. How about
> m_hasTransformOrigin?

Done.

[chrishtr, 2014-06-05 22:24:30]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2014-06-05 22:25:12]

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

[commit-bot: I haz the power, 2014-06-06 01:34:37]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/7088)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/10502)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/10892)

[commit-bot: I haz the power, 2014-06-06 01:37:29]

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

[commit-bot: I haz the power, 2014-06-06 01:37:30]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/7094)

[chrishtr, 2014-06-06 16:27:41]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2014-06-06 16:28:43]

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

[commit-bot: I haz the power, 2014-06-06 17:37:11]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/7167)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/1...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/10613)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/11012)

[commit-bot: I haz the power, 2014-06-06 17:40:26]

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

[commit-bot: I haz the power, 2014-06-06 17:40:27]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/7186)

[chrishtr, 2014-06-09 21:57:32]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2014-06-09 21:58:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/chrishtr@chromium.org/311273008/100001

[commit-bot: I haz the power, 2014-06-09 22:11:21]

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

[chrishtr, 2014-06-09 22:13:29]

The CQ bit was unchecked by chrishtr@chromium.org

[chrishtr, 2014-06-09 22:13:36]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2014-06-09 22:14:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/chrishtr@chromium.org/311273008/100001

[commit-bot: I haz the power, 2014-06-09 22:15:23]

Change committed as 175827