[Layout API] Add LayoutItem.isBR() and use from editing/VisibleUnits

[Layout API] Add LayoutItem.isBR() and use from editing/VisibleUnits

As far as I can tell, we don't need a whole LayoutBRItem class to proxy
specific methods from LayoutBR, but we do need an isBR() method on the
ancestor LayoutItem so we can call it from places that currently call
LayoutObject*->isBR(). One example of this is in editing/VisibleUnits.

There are no functional changes.

BUG=499321
TBR=leviw@chromium.org

Committed: https://crrev.com/943ae70d508af56a2aaafeaf855e7f0e6c717403
Cr-Commit-Position: refs/heads/master@{#381970}

[pilgrim_google, 2016-03-18 14:02:23]

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

[pilgrim_google, 2016-03-18 14:02:35]

Description was changed from

==========
[Layout API] Add LayoutItem.isBR() and use from editing/VisibleUnits

As far as I can tell, we don't need a whole LayoutBRItem class to proxy
specific methods from LayoutBR, but we do need an isBR() method on the
ancestor LayoutItem so we can call it from places that currently call
LayoutObject*->isBR(). One example of this is in editing/VisibleUnits.

There are no functional changes.

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

to

==========
[Layout API] Add LayoutItem.isBR() and use from editing/VisibleUnits

As far as I can tell, we don't need a whole LayoutBRItem class to proxy
specific methods from LayoutBR, but we do need an isBR() method on the
ancestor LayoutItem so we can call it from places that currently call
LayoutObject*->isBR(). One example of this is in editing/VisibleUnits.

There are no functional changes.

BUG=499321
TBR=leviw@chromium.org
==========

[pilgrim_google, 2016-03-18 14:02:38]

The CQ bit was checked by pilgrim@chromium.org

[commit-bot: I haz the power, 2016-03-18 14:02:52]

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

[commit-bot: I haz the power, 2016-03-18 15:09:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-18 15:11:24]

Description was changed from

==========
[Layout API] Add LayoutItem.isBR() and use from editing/VisibleUnits

As far as I can tell, we don't need a whole LayoutBRItem class to proxy
specific methods from LayoutBR, but we do need an isBR() method on the
ancestor LayoutItem so we can call it from places that currently call
LayoutObject*->isBR(). One example of this is in editing/VisibleUnits.

There are no functional changes.

BUG=499321
TBR=leviw@chromium.org
==========

to

==========
[Layout API] Add LayoutItem.isBR() and use from editing/VisibleUnits

As far as I can tell, we don't need a whole LayoutBRItem class to proxy
specific methods from LayoutBR, but we do need an isBR() method on the
ancestor LayoutItem so we can call it from places that currently call
LayoutObject*->isBR(). One example of this is in editing/VisibleUnits.

There are no functional changes.

BUG=499321
TBR=leviw@chromium.org

Committed: https://crrev.com/943ae70d508af56a2aaafeaf855e7f0e6c717403
Cr-Commit-Position: refs/heads/master@{#381970}
==========

[commit-bot: I haz the power, 2016-03-18 15:11:25]

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