Refactoring: Remove addDisplayInline parameter of StyledMarkupAccumulator::appendElement

Refactoring: Remove addDisplayInline parameter of StyledMarkupAccumulator::appendElement

This CL removes an boolean parameter |addDisplayInline| from
StyledMarkupAccumulator::appendElement to improve the code health.
|addDisplayInline| is used to determine whether an inline style is added
to an element node, and this CL splits appendElement into appendElement
and appendElementWithInlineStyle for the respective cases.

BUG=n/a
TEST=n/a; behavior change

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

[hajimehoshi, 2015-06-11 08:31:45]

hajimehoshi@chromium.org changed reviewers:
+ tkent@chromium.org

[hajimehoshi, 2015-06-11 08:31:46]

tkent: I created this CL with yosin@ in pair-programming, so please take a look.

[tkent, 2015-06-11 08:46:09]

lgtm

https://codereview.chromium.org/1173223003/diff/1/Source/core/editing/StyledM...
File Source/core/editing/StyledMarkupAccumulator.cpp (right):

https://codereview.chromium.org/1173223003/diff/1/Source/core/editing/StyledM...
Source/core/editing/StyledMarkupAccumulator.cpp:187: void
StyledMarkupAccumulator::appendElement(StringBuilder& out, Element& element)
optional: Can we make |element| argument |const Element&|?

[hajimehoshi, 2015-06-11 08:56:14]

Thank you!

https://codereview.chromium.org/1173223003/diff/1/Source/core/editing/StyledM...
File Source/core/editing/StyledMarkupAccumulator.cpp (right):

https://codereview.chromium.org/1173223003/diff/1/Source/core/editing/StyledM...
Source/core/editing/StyledMarkupAccumulator.cpp:187: void
StyledMarkupAccumulator::appendElement(StringBuilder& out, Element& element)
On 2015/06/11 08:46:08, tkent wrote:
> optional: Can we make |element| argument |const Element&|?

Done.

[hajimehoshi, 2015-06-11 08:56:18]

The CQ bit was checked by hajimehoshi@chromium.org

[hajimehoshi, 2015-06-11 08:56:18]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1173223003/#ps20001
(title: "tkent's review")

[commit-bot: I haz the power, 2015-06-11 08:56:49]

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

[commit-bot: I haz the power, 2015-06-11 10:12:05]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196934