DevTools: fix css pretty-print.

DevTools: fix css pretty-print.

The CSS pretty-printing was regressed back in the days when all formatters
were moved to a single content builder.

The content builder assumes that added tokens are "words" - and makes sure
those do not stick together. However, CSS formatter violates this contract.

This patch is a workaround which could be merged.

BUG=614678
R=dgozman

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

[lushnikov, 2016-05-25 23:27:01]

please, take a look

[lushnikov, 2016-05-25 23:27:09]

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

[commit-bot: I haz the power, 2016-05-25 23:27:43]

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

[commit-bot: I haz the power, 2016-05-26 03:04:38]

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

[commit-bot: I haz the power, 2016-05-26 03:04:39]

Dry run: This issue passed the CQ dry run.

[commit-bot: I haz the power, 2016-05-26 03:04:52]

Dry run: None

[lushnikov, 2016-05-26 17:51:35]

ping

[dgozman, 2016-05-26 19:37:43]

lgtm

https://codereview.chromium.org/2012953002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/sources/pretty-print-css-1-expected.txt
(right):

https://codereview.chromium.org/2012953002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/sources/pretty-print-css-1-expected.txt:86:
a.b.c:hover,.d.e.f.g::before,h.i {
Let's have a test with this selector inside html.

[lushnikov, 2016-05-26 19:44:58]

https://codereview.chromium.org/2012953002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/sources/pretty-print-css-1-expected.txt
(right):

https://codereview.chromium.org/2012953002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/sources/pretty-print-css-1-expected.txt:86:
a.b.c:hover,.d.e.f.g::before,h.i {
On 2016/05/26 19:37:43, dgozman wrote:
> Let's have a test with this selector inside html.

Done.

[lushnikov, 2016-05-26 19:45:04]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2016-05-26 19:45:04]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2012953002/#ps20001
(title: "address comments")

[commit-bot: I haz the power, 2016-05-26 19:45:34]

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

[commit-bot: I haz the power, 2016-05-27 00:39:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-27 00:40:45]

Description was changed from

==========
DevTools: fix css pretty-print.

The CSS pretty-printing was regressed back in the days when all formatters
were moved to a single content builder.

The content builder assumes that added tokens are "words" - and makes sure
those do not stick together. However, CSS formatter violates this contract.

This patch is a workaround which could be merged.

BUG=614678
R=dgozman
==========

to

==========
DevTools: fix css pretty-print.

The CSS pretty-printing was regressed back in the days when all formatters
were moved to a single content builder.

The content builder assumes that added tokens are "words" - and makes sure
those do not stick together. However, CSS formatter violates this contract.

This patch is a workaround which could be merged.

BUG=614678
R=dgozman

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

[commit-bot: I haz the power, 2016-05-27 00:40:47]

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