Reformat comments in core/layout up until LayoutMultiColumnFlowThread

Reformat comments in core/layout up until LayoutMultiColumnFlowThread

BUG=563793
R=pdr@chromium.org
NOTRY=true

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

[eae, 2016-10-07 01:45:05]

The CQ bit was checked by eae@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-07 01:45:29]

Dry run: 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-10-07 01:54:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-07 01:54:02]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[pdr., 2016-10-07 02:49:40]

LGTM to submit as-is, all comments optional.

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

https://codereview.chromium.org/2394263004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutBoxModelObject.h:158: //
(LayoutFlow) to return the remaining width on a given line (and the height
I think this should be LayoutInline now.

https://codereview.chromium.org/2394263004/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/LayoutInline.cpp (right):

https://codereview.chromium.org/2394263004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutInline.cpp:298: // FIXME: the call
to roundedLayoutPoint() below is temporary and should be
I think this comment can be removed now.

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

https://codereview.chromium.org/2394263004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutMultiColumnFlowThread.h:142: //
dev/developers/design-documents/multi-column-layout
Should this be
https://www.chromium.org/developers/design-documents/multi-column-layout?

[eae, 2016-10-07 16:38:13]

Thanks pdr!

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

https://codereview.chromium.org/2394263004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutBoxModelObject.h:158: //
(LayoutFlow) to return the remaining width on a given line (and the height
On 2016/10/07 02:49:40, pdr. wrote:
> I think this should be LayoutInline now.

Good catch, thanks!

https://codereview.chromium.org/2394263004/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/LayoutInline.cpp (right):

https://codereview.chromium.org/2394263004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutInline.cpp:298: // FIXME: the call
to roundedLayoutPoint() below is temporary and should be
On 2016/10/07 02:49:40, pdr. wrote:
> I think this comment can be removed now.

Yay!

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

https://codereview.chromium.org/2394263004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutMultiColumnFlowThread.h:142: //
dev/developers/design-documents/multi-column-layout
On 2016/10/07 02:49:40, pdr. wrote:
> Should this be
> https://www.chromium.org/developers/design-documents/multi-column-layout

Yes, I tried to change it from sites.google.com to chromium.org but must have
pasted the wrong thing, good catch!

[eae, 2016-10-07 16:51:12]

The CQ bit was checked by eae@chromium.org

[eae, 2016-10-07 16:51:12]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/2394263004/#ps20001
(title: "Rebase w/HEAD")

[commit-bot: I haz the power, 2016-10-07 16:51:35]

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-10-07 16:58:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-07 17:02:17]

Description was changed from

==========
Reformat comments in core/layout up until LayoutMultiColumnFlowThread

BUG=563793
R=pdr@chromium.org
NOTRY=true
==========

to

==========
Reformat comments in core/layout up until LayoutMultiColumnFlowThread

BUG=563793
R=pdr@chromium.org
NOTRY=true

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

[commit-bot: I haz the power, 2016-10-07 17:02:18]

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