Closed Bug 1276577 Opened 8 years ago Closed 8 years ago

(DLC) SyncAction: Use Switchboard.isInExperiment() instead of Experiments.isInExperimentLocal()

Categories

(Firefox for Android Graveyard :: General, defect)

All
Android
defect
Not set
normal

Tracking

(firefox49 fixed)

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: sebastian, Assigned: k.krish, Mentored)

References

Details

Attachments

(1 file, 1 obsolete file)

From bug 1201059 comment 24.

SyncAction uses Experiments.isInExperimentLocal() to test if it is enabled:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/dlc/SyncAction.java#237

Instead we should have used Switchboard.isInExperiment() like we do here in BrowserApp:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/BrowserApp.java#2318

Background: We use Switchboard to enable the synchronization of the downloadable content catalog for 50% of our Nightly users. isInExperimentLocal() exists just for the first run tour where we need to decide if the user is in the experiment before the first start of the app (=> We can't ask the servers first and delay app start).
Attachment #8758002 - Attachment is obsolete: true
Comment on attachment 8758003 [details]
Bug1276577 Using Switchboard.isInExperiment()

https://reviewboard.mozilla.org/r/56358/#review52952

This looks good. Thank you!

It seems like you accidentally pushed two commits. Mercurial's bookmarks are helpful here (They are pretty much like git branches). For every new bug create a new bookmark from fx-team or mozilla-central (depending what repository you are working on). Then just push this new bookmark to reviewboard.

Here are some helpful links:
* https://www.mercurial-scm.org/wiki/Bookmarks
* http://mercurial.aragost.com/kick-start/en/bookmarks/
* http://mozilla-version-control-tools.readthedocs.io/en/latest/hgmozilla/bookmarks.html
Attachment #8758003 - Flags: review+
Yes, I was trying out the review board for the first time. Accidentally pushed two commits. 
Thanks for the guidance.
Krish: Can you get this landed?
Comment on attachment 8758003 [details]
Bug1276577 Using Switchboard.isInExperiment()

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/56358/diff/1-2/
Attachment #8758003 - Attachment description: MozReview Request: Bug1276577 Using Switchboard.isInExperiment() → Bug1276577 Using Switchboard.isInExperiment()
Keywords: checkin-needed
Pushed by kcambridge@mozilla.com:
https://hg.mozilla.org/integration/fx-team/rev/d1c3952c299d
Using Switchboard.isInExperiment() r=sebastian
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/d1c3952c299d
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: