Closed
Bug 895670
Opened 11 years ago
Closed 11 years ago
Remove SetupScreen and associated assets
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 25
People
(Reporter: nalexander, Assigned: nalexander)
References
Details
Attachments
(1 file)
13.43 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
This is part of removing the profile migrator (Bug 886587). I want to fold some related mobile/android/base/Makefile.in changes into this part, because with the death of setup_screen.xml comes the death of logo.png, which means I can finally kill some long-standing Makefile-badness.
Assignee | ||
Comment 1•11 years ago
|
||
There are two parts to this. One is removing SetupScreen and its assets, and that includes removing part of the profile migration code. This is not comprehensive; Bug 886587 will remove the remaining profile migration code. The second is deleting $(MOZ_BRANDING_DIRECTORY)/**/android-resources.mn and removing references to those manifests from mobile/android/base/Makefile.in; this already got positive review under the name of Part 4 of Bug 892603.
Attachment #778137 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 2•11 years ago
|
||
http://tbpl.mozilla.org/?tree=Try&rev=1822bbccf491
Comment 3•11 years ago
|
||
Comment on attachment 778137 [details] [diff] [review] Remove SetupScreen and associated assets. r=mfinkle Kill these strings too: splash_settingup splash_bookmarks_history Can we delete?: branding/*/content/logo.png branding/*/content/favicon32.png
Attachment #778137 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 4•11 years ago
|
||
> Kill these strings too: > splash_settingup > splash_bookmarks_history Good catch! I had so many trees open, I killed these in the wrong buffer. > Can we delete?: > branding/*/content/logo.png mxr says yes. > branding/*/content/favicon32.png Not trivially. We could, with a little mucking about with the built in bookmark icon sizes and some testing code. I'm going to land with these changes.
Assignee | ||
Comment 5•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/e9ef70363066
Target Milestone: --- → Firefox 25
Comment 6•11 years ago
|
||
Touched the clobber file because of R.java build errors: https://tbpl.mozilla.org/php/getParsedLog.php?id=25513336&tree=Mozilla-Inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/ac4f748c92cf
Comment 7•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e9ef70363066 https://hg.mozilla.org/mozilla-central/rev/ac4f748c92cf
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•