Pass along Blink's visual rect for display items.

Pass along Blink's visual rect for display items.

Resubmit of http://crrev.com/1518223002 now that we have a fix in for
http://crbug.com/569643.

Work still pending per http://crbug.com/569211 to use correct rects for certain
items.

BUG=529938
TBR=christr

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

[wkorman, 2016-01-28 20:00:22]

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

[wkorman, 2016-01-28 21:14:46]

Description was changed from

==========
Pass along Blink's visual rect for display items.

Resubmit of http://crrev.com/1518223002 now that we have a fix in for
http://crbug.com/569643.

Work still pending per http://crbug.com/569211 to use correct rects for certain
items.

BUG=529938
==========

to

==========
Pass along Blink's visual rect for display items.

Resubmit of http://crrev.com/1518223002 now that we have a fix in for
http://crbug.com/569643.

Work still pending per http://crbug.com/569211 to use correct rects for certain
items.

BUG=529938
TBR=christr
==========

[wkorman, 2016-01-28 21:14:46]

wkorman@chromium.org changed reviewers:
+ chrishtr@chromium.org
- pdr@chromium.org

[wkorman, 2016-01-28 21:14:59]

The CQ bit was checked by wkorman@chromium.org

[wkorman, 2016-01-28 21:14:59]

This was approved previously and is just a resubmit.

[commit-bot: I haz the power, 2016-01-28 21:15:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1648023002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1648023002/1

[pdr., 2016-01-28 23:07:44]

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

[pdr., 2016-01-28 23:07:45]

LGTM

https://codereview.chromium.org/1648023002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/graphics/paint/PaintArtifact.cpp
(right):

https://codereview.chromium.org/1648023002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/graphics/paint/PaintArtifact.cpp:101:
displayItem.appendToWebDisplayItemList(visualRect, list);
Mind wrapping this all up in one line?

for (const DisplayItem& displayItem : m_displayItemList)
    displayItem.appendToWebDisplayItemList(displayItem.client().visualRect(),
list);

[pdr., 2016-01-28 23:08:16]

On 2016/01/28 at 23:07:45, pdr wrote:
> LGTM
> 
>
https://codereview.chromium.org/1648023002/diff/1/third_party/WebKit/Source/p...
> File third_party/WebKit/Source/platform/graphics/paint/PaintArtifact.cpp
(right):
> 
>
https://codereview.chromium.org/1648023002/diff/1/third_party/WebKit/Source/p...
> third_party/WebKit/Source/platform/graphics/paint/PaintArtifact.cpp:101:
displayItem.appendToWebDisplayItemList(visualRect, list);
> Mind wrapping this all up in one line?
> 
> for (const DisplayItem& displayItem : m_displayItemList)
>     displayItem.appendToWebDisplayItemList(displayItem.client().visualRect(),
list);

(or followup is okay too, or not at all is also ok)

[commit-bot: I haz the power, 2016-01-28 23:14:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-28 23:15:49]

Description was changed from

==========
Pass along Blink's visual rect for display items.

Resubmit of http://crrev.com/1518223002 now that we have a fix in for
http://crbug.com/569643.

Work still pending per http://crbug.com/569211 to use correct rects for certain
items.

BUG=529938
TBR=christr
==========

to

==========
Pass along Blink's visual rect for display items.

Resubmit of http://crrev.com/1518223002 now that we have a fix in for
http://crbug.com/569643.

Work still pending per http://crbug.com/569211 to use correct rects for certain
items.

BUG=529938
TBR=christr

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

[commit-bot: I haz the power, 2016-01-28 23:15:50]

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