DevTools: support sourceMappingURL for the inline stylesheets

DevTools: support sourceMappingURL for the inline stylesheets

This patch properly linkifies inline stylesheet URLs if they have attached
sourcemap.

BUG=649679
R=dgozman

Committed: https://crrev.com/67acf0bd70e456cf893140802c1eda5813ab33c5
Cr-Commit-Position: refs/heads/master@{#436416}

[lushnikov, 2016-12-03 00:26:04]

please, take a look

[lushnikov, 2016-12-03 00:26:08]

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

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

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-12-03 02:25:16]

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

[commit-bot: I haz the power, 2016-12-03 02:25:17]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-12-04 03:13:14]

Nice one! lgtm

https://codereview.chromium.org/2545883005/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js
(right):

https://codereview.chromium.org/2545883005/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js:631:
this.navigable = !!rule.resourceURL() || hasSourceMap;
Note: you can replace rule.resourceURL() with header.resourceURL(), and then
introduce StyleSheetHeader.isAnonymousInlineStyleSheet() similarly to debugger
script.

https://codereview.chromium.org/2545883005/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js:676:
if (ruleLocation && rule.styleSheetId && header && (header.resourceURL() ||
!!header.sourceMap())) {
... and use it here.

[lushnikov, 2016-12-05 19:56:39]

Thanks!

https://codereview.chromium.org/2545883005/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js
(right):

https://codereview.chromium.org/2545883005/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js:631:
this.navigable = !!rule.resourceURL() || hasSourceMap;
On 2016/12/04 03:13:13, dgozman wrote:
> Note: you can replace rule.resourceURL() with header.resourceURL(), and then
> introduce StyleSheetHeader.isAnonymousInlineStyleSheet() similarly to debugger
> script.

Done.

https://codereview.chromium.org/2545883005/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js:676:
if (ruleLocation && rule.styleSheetId && header && (header.resourceURL() ||
!!header.sourceMap())) {
On 2016/12/04 03:13:13, dgozman wrote:
> ... and use it here.

Done.

[lushnikov, 2016-12-05 19:56:42]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2016-12-05 19:56:42]

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/2545883005/#ps40001
(title: "address comments")

[commit-bot: I haz the power, 2016-12-05 19:57:28]

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-12-05 21:34:59]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480967802293240,
"parent_rev": "261f198f086ed86108ca16a3b8d538bfefa083df", "commit_rev":
"b3a7768bb2ea897410e37f2f2ed0f123b5893436"}

[commit-bot: I haz the power, 2016-12-05 21:35:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-05 21:37:54]

Description was changed from

==========
DevTools: support sourceMappingURL for the inline stylesheets

This patch properly linkifies inline stylesheet URLs if they have attached
sourcemap.

BUG=649679
R=dgozman
==========

to

==========
DevTools: support sourceMappingURL for the inline stylesheets

This patch properly linkifies inline stylesheet URLs if they have attached
sourcemap.

BUG=649679
R=dgozman

Committed: https://crrev.com/67acf0bd70e456cf893140802c1eda5813ab33c5
Cr-Commit-Position: refs/heads/master@{#436416}
==========

[commit-bot: I haz the power, 2016-12-05 21:37:55]

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