Rename renderer() to layoutObject().

Rename renderer() to layoutObject().

This CL renames the renderer() method to layoutObject() it does not
attempt to rename any associated variables at this point. These will
be done in smaller followups. The renderer() rename has to happen
over all layout files at the same time.

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

[dsinclair, 2015-03-04 16:38:54]

dsinclair@chromium.org changed reviewers:
+ dmazzoni@chromium.org, jchaffraix@chromium.org, mkwst@chromium.org

[dsinclair, 2015-03-04 16:38:56]

PTAL.

jchaffraix@ for Source/core
dmazzoni@ for Source/modules/accessibility
mkwst@ for Source/web

[dmazzoni, 2015-03-04 18:46:18]

lgtm

not sure if you're waiting for my review on these, please feel free to TBR me
for future renames in modules/accessibility rather than waiting for my approval

[leviw_travelin_and_unemployed, 2015-03-04 20:15:28]

leviw@chromium.org changed reviewers:
+ leviw@chromium.org

[leviw_travelin_and_unemployed, 2015-03-04 20:15:31]

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h
File Source/core/dom/Node.h (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h#n...
Source/core/dom/Node.h:103: void setRenderer(LayoutObject* renderer) {
m_renderer = renderer; }
Man this is weird :p

[dsinclair, 2015-03-04 20:37:56]

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h
File Source/core/dom/Node.h (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h#n...
Source/core/dom/Node.h:103: void setRenderer(LayoutObject* renderer) {
m_renderer = renderer; }
On 2015/03/04 20:15:30, leviw wrote:
> Man this is weird :p

Yea. Doing everything in one CL, I think, would be insane. But, it means we have
wacky intermediate state.

[dsinclair, 2015-03-05 14:33:25]

dsinclair@chromium.org changed reviewers:
+ esprehn@chromium.org

[dsinclair, 2015-03-05 14:33:27]

ping.

[Julien - ping for review, 2015-03-05 16:19:11]

jchaffraix@chromium.org changed reviewers:
- esprehn@chromium.org

[Julien - ping for review, 2015-03-05 16:19:16]

rs lgtm

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Document...
File Source/core/dom/DocumentMarkerController.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Document...
Source/core/dom/DocumentMarkerController.cpp:576: if (LayoutObject* renderer =
iterator->key->layoutObject())
s/renderer/layoutObject/

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Document...
Source/core/dom/DocumentMarkerController.cpp:606: if (LayoutObject* renderer =
node->layoutObject()) {
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Element.cpp
File Source/core/dom/Element.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Element....
Source/core/dom/Element.cpp:558: if (LayoutObject* renderer =
this->layoutObject())
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Element....
Source/core/dom/Element.cpp:923: LayoutObject* elementRenderer = layoutObject();
s/Renderer/LayoutObject/

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Element....
Source/core/dom/Element.cpp:1658: if (LayoutObject* renderer =
this->layoutObject()) {
s/renderer/layoutObject/

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/FirstLet...
File Source/core/dom/FirstLetterPseudoElement.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/FirstLet...
Source/core/dom/FirstLetterPseudoElement.cpp:104: parentRenderer =
element.parentOrShadowHostElement()->layoutObject();
s/parentRenderer/parentLayoutObject/

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/FirstLet...
Source/core/dom/FirstLetterPseudoElement.cpp:295: LayoutObject* renderer =
this->layoutObject();
s/renderer/layoutObject/

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Fullscre...
File Source/core/dom/Fullscreen.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Fullscre...
Source/core/dom/Fullscreen.cpp:432: LayoutObject* renderer =
m_fullScreenElement->layoutObject();
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
File Source/core/dom/LayoutTreeBuilder.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
Source/core/dom/LayoutTreeBuilder.cpp:54: m_renderingParent =
containerNode->layoutObject();
We should probably renaming this one too: m_layoutObjectParent?

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
Source/core/dom/LayoutTreeBuilder.cpp:135: LayoutObject* nextRenderer =
this->nextRenderer();
We need to rename nextRenderer et al., I didn't catch this when we renamed
LayoutTreeBuilderForElement.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
Source/core/dom/LayoutTreeBuilder.cpp:137: newRenderer->setStyle(&style); //
setStyle() can depend on layoutObject() already being set.
s/newRenderer/newLayoutObject/

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp
File Source/core/dom/Node.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp...
Source/core/dom/Node.cpp:373: LayoutObject* renderer =
m_data.m_rareData->layoutObject();
s/renderer/layoutObject/

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp...
Source/core/dom/Node.cpp:373: LayoutObject* renderer =
m_data.m_rareData->layoutObject();
s/renderer/layoutObject/

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp...
Source/core/dom/Node.cpp:605: LayoutObject* renderer = this->layoutObject();
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp...
Source/core/dom/Node.cpp:612: return renderer && renderer->isBoxModelObject() ?
toLayoutBoxModelObject(renderer) : nullptr;
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h
File Source/core/dom/Node.h (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h#n...
Source/core/dom/Node.h:503: LayoutObject* layoutObject() const { return
hasRareData() ? m_data.m_rareData->layoutObject() : m_data.m_renderer; };
It's weird that layoutObject is in rare data.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeLayo...
File Source/core/dom/NodeLayoutStyle.h (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeLayo...
Source/core/dom/NodeLayoutStyle.h:38: if (LayoutObject* renderer =
this->layoutObject())
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeRend...
File Source/core/dom/NodeRenderingTraversal.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeRend...
Source/core/dom/NodeRenderingTraversal.cpp:231: LayoutObject* renderer =
sibling->layoutObject();
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeRend...
Source/core/dom/NodeRenderingTraversal.cpp:241: LayoutObject* renderer =
sibling->layoutObject();
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeRend...
Source/core/dom/NodeRenderingTraversal.cpp:256: if (LayoutObject* renderer =
topLayerElements[i]->layoutObject())
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Position...
File Source/core/dom/Position.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Position...
Source/core/dom/Position.cpp:57: LayoutObject* renderer = node->layoutObject();
Ditto.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Position...
Source/core/dom/Position.cpp:71: LayoutObject* renderer = node->layoutObject();
Ditto (tired of repeating this so no more ditto).

https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLPlu...
File Source/core/html/HTMLPlugInElement.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLPlu...
Source/core/html/HTMLPlugInElement.cpp:377: LayoutObject* r = layoutObject();
Note that s/renderer/layoutObject/ applies to |r| too.

https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
File Source/core/html/HTMLTextAreaElement.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
Source/core/html/HTMLTextAreaElement.cpp:174:
layoutObject()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
It seems like it could be rewritten (as we're touching those lines):

if (LayoutObject layoutObject = this->layoutObject())
    layoutObject->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();

https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
Source/core/html/HTMLTextAreaElement.cpp:189:
layoutObject()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
Ditto.

[dsinclair, 2015-03-05 16:20:59]

On 2015/03/05 at 16:19:16, jchaffraix wrote:
> rs lgtm
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Document...
> File Source/core/dom/DocumentMarkerController.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Document...
> Source/core/dom/DocumentMarkerController.cpp:576: if (LayoutObject* renderer =
iterator->key->layoutObject())
> s/renderer/layoutObject/
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Document...
> Source/core/dom/DocumentMarkerController.cpp:606: if (LayoutObject* renderer =
node->layoutObject()) {
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Element.cpp
> File Source/core/dom/Element.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Element....
> Source/core/dom/Element.cpp:558: if (LayoutObject* renderer =
this->layoutObject())
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Element....
> Source/core/dom/Element.cpp:923: LayoutObject* elementRenderer =
layoutObject();
> s/Renderer/LayoutObject/
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Element....
> Source/core/dom/Element.cpp:1658: if (LayoutObject* renderer =
this->layoutObject()) {
> s/renderer/layoutObject/
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/FirstLet...
> File Source/core/dom/FirstLetterPseudoElement.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/FirstLet...
> Source/core/dom/FirstLetterPseudoElement.cpp:104: parentRenderer =
element.parentOrShadowHostElement()->layoutObject();
> s/parentRenderer/parentLayoutObject/
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/FirstLet...
> Source/core/dom/FirstLetterPseudoElement.cpp:295: LayoutObject* renderer =
this->layoutObject();
> s/renderer/layoutObject/
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Fullscre...
> File Source/core/dom/Fullscreen.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Fullscre...
> Source/core/dom/Fullscreen.cpp:432: LayoutObject* renderer =
m_fullScreenElement->layoutObject();
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
> File Source/core/dom/LayoutTreeBuilder.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
> Source/core/dom/LayoutTreeBuilder.cpp:54: m_renderingParent =
containerNode->layoutObject();
> We should probably renaming this one too: m_layoutObjectParent?
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
> Source/core/dom/LayoutTreeBuilder.cpp:135: LayoutObject* nextRenderer =
this->nextRenderer();
> We need to rename nextRenderer et al., I didn't catch this when we renamed
LayoutTreeBuilderForElement.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
> Source/core/dom/LayoutTreeBuilder.cpp:137: newRenderer->setStyle(&style); //
setStyle() can depend on layoutObject() already being set.
> s/newRenderer/newLayoutObject/
> 
> https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp
> File Source/core/dom/Node.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp...
> Source/core/dom/Node.cpp:373: LayoutObject* renderer =
m_data.m_rareData->layoutObject();
> s/renderer/layoutObject/
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp...
> Source/core/dom/Node.cpp:373: LayoutObject* renderer =
m_data.m_rareData->layoutObject();
> s/renderer/layoutObject/
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp...
> Source/core/dom/Node.cpp:605: LayoutObject* renderer = this->layoutObject();
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.cpp...
> Source/core/dom/Node.cpp:612: return renderer && renderer->isBoxModelObject()
? toLayoutBoxModelObject(renderer) : nullptr;
> Ditto.
> 
> https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h
> File Source/core/dom/Node.h (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h#n...
> Source/core/dom/Node.h:503: LayoutObject* layoutObject() const { return
hasRareData() ? m_data.m_rareData->layoutObject() : m_data.m_renderer; };
> It's weird that layoutObject is in rare data.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeLayo...
> File Source/core/dom/NodeLayoutStyle.h (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeLayo...
> Source/core/dom/NodeLayoutStyle.h:38: if (LayoutObject* renderer =
this->layoutObject())
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeRend...
> File Source/core/dom/NodeRenderingTraversal.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeRend...
> Source/core/dom/NodeRenderingTraversal.cpp:231: LayoutObject* renderer =
sibling->layoutObject();
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeRend...
> Source/core/dom/NodeRenderingTraversal.cpp:241: LayoutObject* renderer =
sibling->layoutObject();
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/NodeRend...
> Source/core/dom/NodeRenderingTraversal.cpp:256: if (LayoutObject* renderer =
topLayerElements[i]->layoutObject())
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Position...
> File Source/core/dom/Position.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Position...
> Source/core/dom/Position.cpp:57: LayoutObject* renderer =
node->layoutObject();
> Ditto.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Position...
> Source/core/dom/Position.cpp:71: LayoutObject* renderer =
node->layoutObject();
> Ditto (tired of repeating this so no more ditto).
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLPlu...
> File Source/core/html/HTMLPlugInElement.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLPlu...
> Source/core/html/HTMLPlugInElement.cpp:377: LayoutObject* r = layoutObject();
> Note that s/renderer/layoutObject/ applies to |r| too.
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
> File Source/core/html/HTMLTextAreaElement.cpp (right):
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
> Source/core/html/HTMLTextAreaElement.cpp:174:
layoutObject()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
> It seems like it could be rewritten (as we're touching those lines):
> 
> if (LayoutObject layoutObject = this->layoutObject())
>     layoutObject->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
> 
>
https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
> Source/core/html/HTMLTextAreaElement.cpp:189:
layoutObject()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
> Ditto.


As mentioned in the description, I'm explicitly not renaming the variables at
this point as the CL will become too hard to manage. This just does renderer().
I'll cleanup the variables with a secondary pass. Variables can be cleaned up
with out having to edit every file at the same time.

[dsinclair, 2015-03-05 16:40:02]

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
File Source/core/dom/LayoutTreeBuilder.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
Source/core/dom/LayoutTreeBuilder.cpp:54: m_renderingParent =
containerNode->layoutObject();
On 2015/03/05 at 16:19:15, Julien Chaffraix - PST wrote:
> We should probably renaming this one too: m_layoutObjectParent?

Done.

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/LayoutTr...
Source/core/dom/LayoutTreeBuilder.cpp:135: LayoutObject* nextRenderer =
this->nextRenderer();
On 2015/03/05 at 16:19:15, Julien Chaffraix - PST wrote:
> We need to rename nextRenderer et al., I didn't catch this when we renamed
LayoutTreeBuilderForElement.

Yes, they're on my list to cleanup (along with about 20 other methods we have
that have Renderer in them.)

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h
File Source/core/dom/Node.h (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/dom/Node.h#n...
Source/core/dom/Node.h:503: LayoutObject* layoutObject() const { return
hasRareData() ? m_data.m_rareData->layoutObject() : m_data.m_renderer; };
On 2015/03/05 at 16:19:16, Julien Chaffraix - PST wrote:
> It's weird that layoutObject is in rare data.

crbug.com/464392

https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
File Source/core/html/HTMLTextAreaElement.cpp (right):

https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
Source/core/html/HTMLTextAreaElement.cpp:174:
layoutObject()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
On 2015/03/05 at 16:19:16, Julien Chaffraix - PST wrote:
> It seems like it could be rewritten (as we're touching those lines):
> 
> if (LayoutObject layoutObject = this->layoutObject())
>     layoutObject->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();

Done.

https://codereview.chromium.org/977113003/diff/20001/Source/core/html/HTMLTex...
Source/core/html/HTMLTextAreaElement.cpp:189:
layoutObject()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
On 2015/03/05 at 16:19:16, Julien Chaffraix - PST wrote:
> Ditto.

Done.

[dsinclair, 2015-03-06 12:58:25]

dsinclair@chromium.org changed reviewers:
+ esprehn@chromium.org

[dsinclair, 2015-03-06 12:58:27]

+esprehn for source/web

[chrishtr, 2015-03-06 16:18:53]

chrishtr@chromium.org changed reviewers:
+ chrishtr@chromium.org

[chrishtr, 2015-03-06 16:18:55]

lgtm

[dsinclair, 2015-03-06 16:26:48]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2015-03-06 16:26:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org, jchaffraix@chromium.org
Link to the patchset: https://codereview.chromium.org/977113003/#ps80001 (title:
"Rebase to master")

[commit-bot: I haz the power, 2015-03-06 16:27:10]

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

[commit-bot: I haz the power, 2015-03-06 16:54:31]

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

[commit-bot: I haz the power, 2015-03-06 16:54:34]

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

[dsinclair, 2015-03-06 17:46:14]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2015-03-06 17:46:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org, dmazzoni@chromium.org, jchaffraix@chromium.org
Link to the patchset: https://codereview.chromium.org/977113003/#ps100001
(title: "Rebase to master")

[commit-bot: I haz the power, 2015-03-06 17:48:00]

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

[commit-bot: I haz the power, 2015-03-06 20:08:04]

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

[commit-bot: I haz the power, 2015-03-06 20:08:08]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/46317)

[dsinclair, 2015-03-06 20:49:40]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2015-03-06 20:49:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org, dmazzoni@chromium.org, jchaffraix@chromium.org
Link to the patchset: https://codereview.chromium.org/977113003/#ps120001
(title: " ")

[commit-bot: I haz the power, 2015-03-06 20:50:16]

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

[commit-bot: I haz the power, 2015-03-07 00:31:55]

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