Add a class comment to LayoutText

Add a class comment to LayoutText

The class was sorely missing some documentation
about the design decisions and its purpose.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201360

[Julien - ping for review, 2015-08-27 22:16:06]

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

[eae, 2015-08-27 22:22:25]

https://codereview.chromium.org/1323443002/diff/1/Source/core/layout/LayoutTe...
File Source/core/layout/LayoutText.h (right):

https://codereview.chromium.org/1323443002/diff/1/Source/core/layout/LayoutTe...
Source/core/layout/LayoutText.h:51: // See e.g.
LayoutBlockFlow::layoutInlineChildren.
It is probably worth pointing out that this class does contain the logic for
prefs width calculation so it's more than just a placeholder.

[eae, 2015-08-28 00:32:33]

Awesome, LGTM with nits. Please commit directly once addressed, no need for
another round of reviews.

https://codereview.chromium.org/1323443002/diff/20001/Source/core/layout/Layo...
File Source/core/layout/LayoutText.h (right):

https://codereview.chromium.org/1323443002/diff/20001/Source/core/layout/Layo...
Source/core/layout/LayoutText.h:50: // Bidirectional Algorithm algorithm to
break the text into actual lines.
s/Algorithm algorithm/algorithm/

https://codereview.chromium.org/1323443002/diff/20001/Source/core/layout/Layo...
Source/core/layout/LayoutText.h:51: // The result of layout is the line box
tree, which represents text lines
s/text lines/lines - as lines can contain inline boxes (non-text) as well.

[Julien - ping for review, 2015-08-28 02:18:41]

The CQ bit was checked by jchaffraix@chromium.org

[Julien - ping for review, 2015-08-28 02:18:41]

Updated thanks for the review!

https://codereview.chromium.org/1323443002/diff/20001/Source/core/layout/Layo...
File Source/core/layout/LayoutText.h (right):

https://codereview.chromium.org/1323443002/diff/20001/Source/core/layout/Layo...
Source/core/layout/LayoutText.h:50: // Bidirectional Algorithm algorithm to
break the text into actual lines.
On 2015/08/28 at 00:32:33, eae wrote:
> s/Algorithm algorithm/algorithm/

Done.

https://codereview.chromium.org/1323443002/diff/20001/Source/core/layout/Layo...
Source/core/layout/LayoutText.h:51: // The result of layout is the line box
tree, which represents text lines
On 2015/08/28 at 00:32:33, eae wrote:
> s/text lines/lines - as lines can contain inline boxes (non-text) as well.

Done.

[Julien - ping for review, 2015-08-28 02:18:42]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org
Link to the patchset: https://codereview.chromium.org/1323443002/#ps40001
(title: "Updated after Emil's comments.")

[commit-bot: I haz the power, 2015-08-28 02:18:49]

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

[commit-bot: I haz the power, 2015-08-28 03:03:24]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201360