Bug 62537 - Web Inspector: no way to distinguish between two scripts with the same name.
Summary: Web Inspector: no way to distinguish between two scripts with the same name.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Pavel Podivilov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-13 04:57 PDT by Pavel Podivilov
Modified: 2011-06-14 09:26 PDT (History)
11 users (show)

See Also:


Attachments
Patch. (2.65 KB, patch)
2011-06-13 04:59 PDT, Pavel Podivilov
no flags Details | Formatted Diff | Diff
Fix style. (2.71 KB, patch)
2011-06-13 06:07 PDT, Pavel Podivilov
eric: review-
Details | Formatted Diff | Diff
Added test. (8.23 KB, patch)
2011-06-14 02:37 PDT, Pavel Podivilov
pfeldman: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Podivilov 2011-06-13 04:57:22 PDT
Web Inspector: no way to distinguish between two scripts with the same name.

We currently display only the "last path component" of script url in scripts file select. There is no way to see the full url.
Comment 1 Pavel Podivilov 2011-06-13 04:59:05 PDT
Created attachment 96946 [details]
Patch.

Web Inspector: add tooltip to file select options in scripts panel.
Comment 2 WebKit Review Bot 2011-06-13 05:03:22 PDT
Attachment 96946 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1

Source/WebCore/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Pavel Podivilov 2011-06-13 06:07:48 PDT
Created attachment 96950 [details]
Fix style.
Comment 4 Eric Seidel (no email) 2011-06-13 14:39:18 PDT
Comment on attachment 96950 [details]
Fix style.

Looks reasonable to me.  How do we test this?
Comment 5 Pavel Podivilov 2011-06-14 02:37:49 PDT
Created attachment 97091 [details]
Added test.
Comment 6 Pavel Podivilov 2011-06-14 09:26:11 PDT
Committed r88808: <http://trac.webkit.org/changeset/88808>