Too large script labels in various pages

RESOLVED FIXED in Firefox 15

Status

P2
normal
RESOLVED FIXED
7 years ago
8 months ago

People

(Reporter: past, Assigned: vporof)

Tracking

Trunk
Firefox 15

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

STR:
1) Visit cnn.com
2) Open the script debugger
3) Observe the huge labels in the script list

In SS__getScriptLabel we trim the query string before deciding on a unique, short label for the script. In this case, however, the query string is what differentiates the script URLs.

The best mitigation I came up with is using the path info as usual, but instead of appending the whole query string, just add a unique subset of it, probably with ellipsis characters before and after.
(Assignee)

Updated

7 years ago
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Duplicate of this bug: 759629
(Assignee)

Comment 3

7 years ago
(In reply to Panos Astithas [:past] from comment #2)
> I'm not even sure such URLs are valid

Sometimes I'm surprised the whole internet thing even works...
(Assignee)

Updated

7 years ago
Summary: Too large script labels in cnn.com → Too large script labels in various pages
(Assignee)

Comment 4

7 years ago
Created attachment 628299 [details] [diff] [review]
v1

Works.
Attachment #628299 - Flags: review?(past)
(Assignee)

Comment 5

7 years ago
Created attachment 628301 [details] [diff] [review]
v1.1

I just realized that until now, the whole "for (let i = 0; i < vs._scripts.itemCount; i++)" check for script urls was incorrect, because there was no 1:1 relationship between the test urls array and the actual added urls in the dropdown. This means that, although it worked and the order wasn't important, not all the urls were tested. Fixed in this version.
Attachment #628299 - Attachment is obsolete: true
Attachment #628299 - Flags: review?(past)
Attachment #628301 - Flags: review?(past)
Comment on attachment 628301 [details] [diff] [review]
v1.1

Review of attachment 628301 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #628301 - Flags: review?(past) → review+
Whiteboard: [land-in-fx-team]
https://hg.mozilla.org/integration/fx-team/rev/fd76266dc5ca
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Target Milestone: --- → Firefox 15
https://hg.mozilla.org/mozilla-central/rev/fd76266dc5ca
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]

Updated

8 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.