Bug 59108

Summary:

Web Inspector: group content scripts in the scripts panel drop down.

Product:

WebKit

Reporter:

Pavel Feldman <pfeldman>

Component:

Web Inspector (Deprecated)

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	yurys: review+

Description Pavel Feldman 2011-04-21 09:35:03 PDT

Patch to follow.

Comment 1 Pavel Feldman 2011-04-21 09:40:02 PDT

Created attachment 90547 [details]
Patch

Comment 2 Yury Semikhatsky 2011-04-22 05:37:39 PDT

Comment on attachment 90547 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=90547&action=review

> Source/WebCore/inspector/front-end/ScriptsPanel.js:292
> +            if (a.isContentScript && !b.isContentScript)

you could use difference between two boolean values instead of these two statements:

var d = !!a.isContentScript - !!b.isContentScript;
if (!d) return d;

Comment 3 Pavel Feldman 2011-04-22 06:18:29 PDT

Committed r84623: <http://trac.webkit.org/changeset/84623>