Last Comment Bug 705974 - Remove files deprecated in Fx7/Fx8 when updating to Fx9
: Remove files deprecated in Fx7/Fx8 when updating to Fx9
Product: Firefox
Classification: Client Software
Component: Build Config (show other bugs)
: 9 Branch
: x86 All
-- normal (vote)
: Firefox 11
Assigned To: Nobody; OK to take it and work on it
: Gregory Szorc [:gps] (away until 2017-03-20)
Depends on: 504645 670880
Blocks: 714609
  Show dependency treegraph
Reported: 2011-11-28 19:41 PST by Nick Thomas [:nthomas]
Modified: 2012-01-02 04:49 PST (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Adjust (1.11 KB, patch)
2011-11-28 19:41 PST, Nick Thomas [:nthomas]
khuey: review+
akeybl: approval‑mozilla‑aurora-
akeybl: approval‑mozilla‑beta-
Details | Diff | Splinter Review

Description User image Nick Thomas [:nthomas] 2011-11-28 19:41:02 PST
Created attachment 577469 [details] [diff] [review]

Bug 670880 removed modules/SpatialNavigation.js in Fx8 for all platforms
Bug 504645 removed icons/document.png in Fx7 for Linux

Both are not removed when applying an update from earlier versions, eg 3.6.24, and this fixes that up.
Comment 1 User image Nick Thomas [:nthomas] 2011-12-04 16:12:37 PST
Comment on attachment 577469 [details] [diff] [review]
Comment 2 User image Matt Brubeck (:mbrubeck) 2011-12-05 10:32:19 PST
Comment 3 User image Nick Thomas [:nthomas] 2011-12-05 19:29:40 PST
Comment on attachment 577469 [details] [diff] [review]

Drivers, this patch removes two files when users apply an update, which we stopped shipping in Fx7 or 8 "installers". The result is an old install + updates will be the same as a new install (excluding old empty directories we don't remove with the updater).  Both files were minor ones that were unused or little used before we stopped shipping them in "installers".

I'd like to at least get this on Aurora, and beta if it's an appropriate change at this point in the cycle, but it's not a problem if that is not problem.
Comment 4 User image Nick Thomas [:nthomas] 2011-12-05 19:30:46 PST
Ahem, .... it's not a problem that isn't possible.
Comment 5 User image Alex Keybl [:akeybl] 2011-12-06 14:55:04 PST
Comment on attachment 577469 [details] [diff] [review]

[Triage Comment]
Let's let this ride the train. Doesn't appear to have major user benefit, and QA said this doesn't affect their testing.
Comment 6 User image Nick Thomas [:nthomas] 2011-12-06 14:56:40 PST
Ok, all done here then.

Note You need to log in before you can comment on or make changes to this bug.