Revert of Don't access DisplayItemClient::VisualRect() for cached display items.

Revert of Don't access DisplayItemClient::VisualRect() for cached display items. (patchset #3 id:40001 of https://codereview.chromium.org/2894093002/ )

Reason for revert:
BUG=724920, 724263, 725041, 724936
Original issue's description:
> Don't access DisplayItemClient::VisualRect() for cached display items.
> 
> - Access DisplayItemClient::VisualRect() when a new DisplayItem is
>   created, and save it in the new DisplayItem.
> - When a cached display item is copied, don't access DisplayItemClient::
>   VisualRect() but fetch from the cached DisplayItem.
> - Remove DisplayItemList::visual_rects_.
> - Save original visual rects (LayoutRect) and apply GraphicsLayer offset
>   with EnclosingIntRect() when the display items are added into
>   WebDisplayItemList.
> 
> Please see the bug for the reason.
> 
> BUG=724263
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2894093002
> Cr-Commit-Position: refs/heads/master@{#473371}
> Committed: https://chromium.googlesource.com/chromium/src/+/3c7f1c4ecd1421d074daa5a103ea3f895c1c8724

TBR=chrishtr@chromium.org,wkorman@chromium.org,pdr@chromium.org,michaelpg@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=724263
Review-Url: https://codereview.chromium.org/2899793002
Cr-Commit-Position: refs/heads/master@{#473640}
Committed: https://chromium.googlesource.com/chromium/src/+/86c54d1018d43154254a9b479f43621bd8f6cad6

[Xianzhu, 2017-05-22 16:57:14]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2017-05-22 16:57:15]

Created Revert of Don't access DisplayItemClient::VisualRect() for cached
display items.

[commit-bot: I haz the power, 2017-05-22 16:57:39]

CQ is trying da patch.

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

[michaelpg, 2017-05-22 17:12:01]

revert lgtm

[commit-bot: I haz the power, 2017-05-22 18:50:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495472234274230, "parent_rev":
"cabeb0d9ab1b39fc25a1647d38afade5bea2e70e", "commit_rev":
"86c54d1018d43154254a9b479f43621bd8f6cad6"}

[commit-bot: I haz the power, 2017-05-22 18:50:13]

Description was changed from

==========
Revert of Don't access DisplayItemClient::VisualRect() for cached display items.
(patchset #3 id:40001 of https://codereview.chromium.org/2894093002/ )

Reason for revert:
BUG=724920,724263,725041,724936

Original issue's description:
> Don't access DisplayItemClient::VisualRect() for cached display items.
> 
> - Access DisplayItemClient::VisualRect() when a new DisplayItem is
>   created, and save it in the new DisplayItem.
> - When a cached display item is copied, don't access DisplayItemClient::
>   VisualRect() but fetch from the cached DisplayItem.
> - Remove DisplayItemList::visual_rects_.
> - Save original visual rects (LayoutRect) and apply GraphicsLayer offset
>   with EnclosingIntRect() when the display items are added into
>   WebDisplayItemList.
> 
> Please see the bug for the reason.
> 
> BUG=724263
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2894093002
> Cr-Commit-Position: refs/heads/master@{#473371}
> Committed:
https://chromium.googlesource.com/chromium/src/+/3c7f1c4ecd1421d074daa5a103ea...

TBR=chrishtr@chromium.org,wkorman@chromium.org,pdr@chromium.org,michaelpg@chr...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=724263
==========

to

==========
Revert of Don't access DisplayItemClient::VisualRect() for cached display items.
(patchset #3 id:40001 of https://codereview.chromium.org/2894093002/ )

Reason for revert:
BUG=724920,724263,725041,724936

Original issue's description:
> Don't access DisplayItemClient::VisualRect() for cached display items.
> 
> - Access DisplayItemClient::VisualRect() when a new DisplayItem is
>   created, and save it in the new DisplayItem.
> - When a cached display item is copied, don't access DisplayItemClient::
>   VisualRect() but fetch from the cached DisplayItem.
> - Remove DisplayItemList::visual_rects_.
> - Save original visual rects (LayoutRect) and apply GraphicsLayer offset
>   with EnclosingIntRect() when the display items are added into
>   WebDisplayItemList.
> 
> Please see the bug for the reason.
> 
> BUG=724263
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> 
> Review-Url: https://codereview.chromium.org/2894093002
> Cr-Commit-Position: refs/heads/master@{#473371}
> Committed:
https://chromium.googlesource.com/chromium/src/+/3c7f1c4ecd1421d074daa5a103ea...

TBR=chrishtr@chromium.org,wkorman@chromium.org,pdr@chromium.org,michaelpg@chr...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=724263

Review-Url: https://codereview.chromium.org/2899793002
Cr-Commit-Position: refs/heads/master@{#473640}
Committed:
https://chromium.googlesource.com/chromium/src/+/86c54d1018d43154254a9b479f43...
==========

[commit-bot: I haz the power, 2017-05-22 18:50:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/86c54d1018d43154254a9b479f43...