Add inlineOverflow and blockOverflow methods to NGFragment.

Add inlineOverflow and blockOverflow methods to NGFragment and make it garbage-collected.

BUG=635619
NOTRY=true

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

[Gleb Lanbin, 2016-08-09 16:14:33]

Description was changed from

==========
Add inlineOverflow and blockOverflow methods to NGFragment.

BUG=635619
==========

to

==========
Add inlineOverflow and blockOverflow methods to NGFragment and make it
garbage-collected.

BUG=635619
==========

[Gleb Lanbin, 2016-08-09 16:15:09]

glebl@chromium.org changed reviewers:
+ cbiesinger@chromium.org, ikilpatrick@chromium.org

[ikilpatrick, 2016-08-09 16:23:29]

ikilpatrick@chromium.org changed reviewers:
+ eae@chromium.org

[ikilpatrick, 2016-08-09 16:23:29]

lgtm

[eae, 2016-08-09 16:26:02]

LGTM w/nit

https://codereview.chromium.org/2229663005/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/ng/NGFragment.h (right):

https://codereview.chromium.org/2229663005/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/ng/NGFragment.h:24: // Returns the size of
the contents which are outside the border-box.
This is a little misleading, how about:

Returns the total size, including the contents outside of the border-box.

[Gleb Lanbin, 2016-08-09 16:37:46]

https://codereview.chromium.org/2229663005/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/ng/NGFragment.h (right):

https://codereview.chromium.org/2229663005/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/ng/NGFragment.h:24: // Returns the size of
the contents which are outside the border-box.
On 2016/08/09 16:26:01, eae wrote:
> This is a little misleading, how about:
> 
> Returns the total size, including the contents outside of the border-box.

Done.

[Gleb Lanbin, 2016-08-09 16:37:51]

The CQ bit was checked by glebl@chromium.org

[Gleb Lanbin, 2016-08-09 16:37:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
ikilpatrick@chromium.org, eae@chromium.org
Link to the patchset: https://codereview.chromium.org/2229663005/#ps20001
(title: "fix comments")

[commit-bot: I haz the power, 2016-08-09 16:38:32]

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

[eae, 2016-08-09 19:16:04]

The CQ bit was unchecked by eae@chromium.org

[eae, 2016-08-09 19:16:10]

Description was changed from

==========
Add inlineOverflow and blockOverflow methods to NGFragment and make it
garbage-collected.

BUG=635619
==========

to

==========
Add inlineOverflow and blockOverflow methods to NGFragment and make it
garbage-collected.

BUG=635619
NOTRY=true
==========

[eae, 2016-08-09 19:16:16]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2016-08-09 19:16:52]

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

[commit-bot: I haz the power, 2016-08-09 19:49:12]

Description was changed from

==========
Add inlineOverflow and blockOverflow methods to NGFragment and make it
garbage-collected.

BUG=635619
NOTRY=true
==========

to

==========
Add inlineOverflow and blockOverflow methods to NGFragment and make it
garbage-collected.

BUG=635619
NOTRY=true
==========

[commit-bot: I haz the power, 2016-08-09 19:49:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-09 19:50:52]

Description was changed from

==========
Add inlineOverflow and blockOverflow methods to NGFragment and make it
garbage-collected.

BUG=635619
NOTRY=true
==========

to

==========
Add inlineOverflow and blockOverflow methods to NGFragment and make it
garbage-collected.

BUG=635619
NOTRY=true

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

[commit-bot: I haz the power, 2016-08-09 19:50:54]

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