Devtools: Do not greedily collapse URLs

Devtools: Do not greedily collapse URLs

A longstanding bug resulted in a property value containing multiple URLs
was being matched as a single URL and collapsed. When re-expanding this,
the original text was then not restored correctly. This then resulted in
the incorrect style being applied upon any edit.

BUG=590172
Committed: https://crrev.com/5ac5f0c65ec60fa465e378b295d8aca067986234
Cr-Commit-Position: refs/heads/master@{#378982}

[samli, 2016-03-01 06:11:57]

samli@chromium.org changed reviewers:
+ dgozman@chromium.org

[dgozman, 2016-03-01 06:17:26]

dgozman@chromium.org changed reviewers:
+ lushnikov@chromium.org

[dgozman, 2016-03-01 06:17:27]

Over to Andrey.

[kozy, 2016-03-01 22:05:40]

kozyatinskiy@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org

[kozy, 2016-03-01 22:05:40]

Can we add a test?

[samli, 2016-03-03 03:17:30]

On 2016/03/01 at 22:05:40, kozyatinskiy wrote:
> Can we add a test?

Done.

[lushnikov, 2016-03-03 05:05:41]

lgtm % nit

https://codereview.chromium.org/1750263002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/elements/styles/url-multiple-collapsing.html
(right):

https://codereview.chromium.org/1750263002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/elements/styles/url-multiple-collapsing.html:52:
Tests that multiple URLs which are long are not squashed into a single URL. Bug
<a href="http://bugs.webkit.org/show_bug.cgi?id=590172">590172</a>.
bugs.webkit.org?..

[samli, 2016-03-03 05:10:40]

The CQ bit was checked by samli@chromium.org

[samli, 2016-03-03 05:10:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
lushnikov@chromium.org
Link to the patchset: https://codereview.chromium.org/1750263002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2016-03-03 05:10:58]

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

[commit-bot: I haz the power, 2016-03-03 08:12:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-03 08:13:59]

Description was changed from

==========
Devtools: Do not greedily collapse URLs

A longstanding bug resulted in a property value containing multiple URLs
was being matched as a single URL and collapsed. When re-expanding this,
the original text was then not restored correctly. This then resulted in
the incorrect style being applied upon any edit.

BUG=590172
==========

to

==========
Devtools: Do not greedily collapse URLs

A longstanding bug resulted in a property value containing multiple URLs
was being matched as a single URL and collapsed. When re-expanding this,
the original text was then not restored correctly. This then resulted in
the incorrect style being applied upon any edit.

BUG=590172

Committed: https://crrev.com/5ac5f0c65ec60fa465e378b295d8aca067986234
Cr-Commit-Position: refs/heads/master@{#378982}
==========

[commit-bot: I haz the power, 2016-03-03 08:14:01]

Patchset 3 (id:??) landed as
https://crrev.com/5ac5f0c65ec60fa465e378b295d8aca067986234
Cr-Commit-Position: refs/heads/master@{#378982}