1159978 - Strip out old StartPane activity

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Chenxia Liu [:liuche]]

We switched from a StartPane activity to using a Fragment - this bug is for stripping out the old references to StartPane that are no longer used. (Note, there are some references to forms of "startpane" that are still used, but the activity declaration in AndroidManifest.xml.in is definitely obsolete.)

Comment 1

[Karim Benhmida [:karim]]

Attachment: Strip out old StartPane activity

Comment 2

[Chenxia Liu [:liuche]]

Comment on attachment 8608272 [details] [diff] [review]
Strip out old StartPane activity

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

Just one suggestion and you're good. Upload another version, obsolete the previous patch, and flag me for review :)

::: mobile/android/base/resources/values-large-v11/themes.xml
@@ +7,1 @@
>      

Actually, since there are no more styles in the file, you can just remove it entirely :)

(Also, if you weren't removing the file, you could at least remove these trailing spaces. We don't want to make whitespace-only changes, because that changes the hg history and can be confusing, but if you're already making changes in a place and you see trailing spaces [which are shown in red here] you can just remove them.)

Comment 3

[Karim Benhmida [:karim]]

Attachment: Strip out old StartPane activity

Comment 4

[Chenxia Liu [:liuche]]

Comment on attachment 8608286 [details] [diff] [review]
Strip out old StartPane activity

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

Nice, I'll land this for you.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/200691b33953

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/200691b33953

Comment 7

[Karim Benhmida [:karim]]

Attachment: MozReview Request: bz://1159978/karim

/r/9287 - Bug 1159978 - Strip out old StartPane activity. r=liuche
/r/9289 - Bug 1164301 - Voice input in url activates on click and release. r=liuche
/r/9291 - try: -b o -f -p android-x86,android-api-9,android-api-11 -u all -t none

Pull down these commits:

hg pull -r e3da3cad76c26c8697549e4f2cbf80b2fe1cbca0 https://reviewboard-hg.mozilla.org/gecko/

Comment 9

[Karim Benhmida [:karim]]

Attachment: MozReview Request: Bug 1164301 - Voice input in url activates on click and release. r=liuche

Comment 10

[Karim Benhmida [:karim]]

Attachment: MozReview Request: try: -b o -f -p android-x86,android-api-9,android-api-11 -u all -t none

Comment 11

[Karim Benhmida [:karim]]

Attachment: MozReview Request: Bug 1159978 - Strip out old StartPane activity. r=liuche