Closed
Bug 728936
Opened 12 years ago
Closed 12 years ago
Drop jquery*.min.js from test pilot
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(thunderbird12 fixed)
RESOLVED
FIXED
Thunderbird 13.0
Tracking | Status | |
---|---|---|
thunderbird12 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
123.88 KB,
patch
|
Irving
:
review+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
Doing a few searches on the code for Test Pilot shows that we don't actually need the *.min.js files. These actually come from the shipped version of the "flot" extension to jquery which includes both the min and non-min files, so we should really just package the ones we need.
Attachment #598937 -
Flags: review?(jdicarlo)
Assignee | ||
Updated•12 years ago
|
Attachment #598937 -
Flags: review?(jdicarlo) → review?(squibblyflabbetydoo)
Assignee | ||
Updated•12 years ago
|
Attachment #598937 -
Flags: review?(irving)
Comment 1•12 years ago
|
||
Comment on attachment 598937 [details] [diff] [review] The fix Review of attachment 598937 [details] [diff] [review]: ----------------------------------------------------------------- Can't see any reason why we need these files.
Attachment #598937 -
Flags: review?(irving) → review+
Assignee | ||
Comment 2•12 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/c04a8a9ed0f3
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
Comment 3•12 years ago
|
||
Comment on attachment 598937 [details] [diff] [review] The fix Clearing out my review, since this is already checked in. Sorry about dropping the ball on this...
Attachment #598937 -
Flags: review?(squibblyflabbetydoo)
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 598937 [details] [diff] [review] The fix [Triage Comment] Taking to beta as I want to get the version of test pilot we're distributing, synced up with the version on AMO.
Attachment #598937 -
Flags: approval-comm-aurora+
Assignee | ||
Updated•12 years ago
|
Attachment #598937 -
Flags: approval-comm-aurora+ → approval-comm-beta+
Assignee | ||
Comment 5•12 years ago
|
||
Checked in: http://hg.mozilla.org/releases/comm-beta/rev/7196a9fd07f4
status-thunderbird12:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•