[Layout API] Convert (most of) FrameSelection to use Document::layoutViewItem()

[Layout API] Convert (most of) FrameSelection to use Document::layoutViewItem()

There are several references to Document::layoutView(), which is deprecated
by the new layout API. I converted them to use layoutViewItem() instead. I
added LayoutViewItem::invalidatePaintForSelection() to the API.

There are no functional changes.

BUG=499321
Committed: https://crrev.com/89a4287c0abb8a7816c079de9435b8ff22b133fa
Cr-Commit-Position: refs/heads/master@{#388593}

[pilgrim_google, 2016-04-15 16:31:11]

pilgrim@chromium.org changed reviewers:
+ eae@chromium.org, jsbell@chromium.org, leviw@chromium.org

[eae, 2016-04-17 18:17:50]

https://codereview.chromium.org/1885373005/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/editing/FrameSelection.cpp (right):

https://codereview.chromium.org/1885373005/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/editing/FrameSelection.cpp:936: if
(LayoutViewItem view = document->layoutViewItem())
Can this still be null?

[pilgrim_google, 2016-04-20 21:35:47]

On 2016/04/17 at 18:17:50, eae wrote:
>
https://codereview.chromium.org/1885373005/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/editing/FrameSelection.cpp (right):
> 
>
https://codereview.chromium.org/1885373005/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/editing/FrameSelection.cpp:936: if
(LayoutViewItem view = document->layoutViewItem())
> Can this still be null?

Good point. Fixed in patch 2.

[leviw_travelin_and_unemployed, 2016-04-20 21:53:51]

The CQ bit was checked by leviw@chromium.org

[leviw_travelin_and_unemployed, 2016-04-20 21:53:52]

lgtm

[commit-bot: I haz the power, 2016-04-20 21:54:32]

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

[commit-bot: I haz the power, 2016-04-20 23:27:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-22 19:27:13]

Description was changed from

==========
[Layout API] Convert (most of) FrameSelection to use Document::layoutViewItem()

There are several references to Document::layoutView(), which is deprecated
by the new layout API. I converted them to use layoutViewItem() instead. I
added LayoutViewItem::invalidatePaintForSelection() to the API.

There are no functional changes.

BUG=499321
==========

to

==========
[Layout API] Convert (most of) FrameSelection to use Document::layoutViewItem()

There are several references to Document::layoutView(), which is deprecated
by the new layout API. I converted them to use layoutViewItem() instead. I
added LayoutViewItem::invalidatePaintForSelection() to the API.

There are no functional changes.

BUG=499321

Committed: https://crrev.com/89a4287c0abb8a7816c079de9435b8ff22b133fa
Cr-Commit-Position: refs/heads/master@{#388593}
==========

[commit-bot: I haz the power, 2016-04-22 19:27:14]

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