Closed Bug 1109000 Opened 7 years ago Closed 7 years ago

Refactor statics out of BrowserLocaleManager

Categories

(Firefox for Android Graveyard :: Locale switching and selection, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 37

People

(Reporter: rnewman, Assigned: rnewman)

References

Details

Attachments

(1 file)

No description provided.
Comment on attachment 8533660 [details] [diff] [review]
Split statics out of BrowserLocaleManager. v1

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

I have no problem with this.
Attachment #8533660 - Flags: review?(nalexander) → review+
Comment on attachment 8533660 [details] [diff] [review]
Split statics out of BrowserLocaleManager. v1

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

::: mobile/android/base/Locales.java
@@ +15,5 @@
>  import android.os.StrictMode;
>  import android.support.v4.app.FragmentActivity;
>  
>  /**
> + * This is a helper class to do typical locale switching operations without

There are some trailing whitespace issues throughout here, but nbd.
https://hg.mozilla.org/mozilla-central/rev/e2759f8d7232
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 37
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.