Closed Bug 1173388 Opened 9 years ago Closed 9 years ago

Cookies settings are not always kept after browser restart

Categories

(Firefox for Android Graveyard :: Settings and Preferences, defect)

39 Branch
ARM
Android
defect
Not set
normal

Tracking

(firefox38.0.5 unaffected, firefox39+ fixed, firefox40 unaffected, firefox41 unaffected, fennec39+)

RESOLVED DUPLICATE of bug 1118818
Tracking Status
firefox38.0.5 --- unaffected
firefox39 + fixed
firefox40 --- unaffected
firefox41 --- unaffected
fennec 39+ ---

People

(Reporter: cos_flaviu, Assigned: liuche)

References

Details

Attachments

(1 file)

Attached file cookies settings logs.txt (deleted) —
Environment: Device: Asus Transformer Pad (Android 4.2.1); Build: Firefox 39 Beta 4 build 2; Steps to reproduce: 1. Go to Settings -> Privacy; 2. Disable Cookies; 3. Restart fennec. 1. Go back to Settings -> Privacy; Expected result: After restart the Cookies are disabled. Actual result: After restart the Cookies are still enabled. Notes: Attached are the logs from logcat; Looks like if I wait like 30 second after I close Fennec and then open it again the bug will not reproduce. Can not reproduce it at all on Aurora or Nightly.
Summary: Cookies settings are not allways kept after browser restart → Cookies settings are not always kept after browser restart
tracking-fennec: --- → ?
[Tracking Requested - why for this release]: regression Ni to Margaret for an assignee
tracking-fennec: ? → 39+
Flags: needinfo?(margaret.leibovic)
Sounds like we need to flush gecko preferences to disk after they change. liuche, can you look at this?
Flags: needinfo?(margaret.leibovic) → needinfo?(liuche)
Since bug 1118818 landed in 40 and fixes that issue, it looks like we just need to uplift that to 39. We're nearing the end of the cycle though, so rnewman, can you think of any reason we shouldn't request uplift?
Flags: needinfo?(liuche) → needinfo?(rnewman)
If you can verify that this doesn't repro in 40+, then uplifting that fix is safe IMO. Run with it!
Assignee: nobody → liuche
Status: NEW → ASSIGNED
Depends on: 1118818
Flags: needinfo?(rnewman)
Hm, I tried this on fresh Betas from the Play Store (6/12) installed on the following devices, and could not repro this. Flaviu, can you repro this on a device that matches one of the ones that I tried? Nexus 7 (5.0.1) Nexus 9 (5.0.1) Asus Transformer TF101 (4.0.3) Nexus 4 (4.4.4)
My STR: - Open Settings > Privacy > Cookies - Switch from "Enabled" to "Disabled" - Swipe close from recent apps - Re-open, go to Settings > Privacy > Cookies
(In reply to Chenxia Liu [:liuche] from comment #6) > Hm, I tried this on fresh Betas from the Play Store (6/12) installed on the > following devices, and could not repro this. Flaviu, can you repro this on a > device that matches one of the ones that I tried? > > Nexus 7 (5.0.1) > Nexus 9 (5.0.1) > Asus Transformer TF101 (4.0.3) > Nexus 4 (4.4.4) I tested on Asus Transformer TF101 (4.0.3) and Nexus 7 (5.1.1) and could not reproduce. It may be specific to device or android version.
Flaviu, does this build fix the issue on the affected devices? http://people.mozilla.org/~liuche/bug-1173388/flush-beta.apk This is a build of the patch from bug 1118818 on top of a pull from beta this morning.
Flags: needinfo?(flaviu.cos)
(In reply to Chenxia Liu [:liuche] from comment #9) > Flaviu, does this build fix the issue on the affected devices? > > http://people.mozilla.org/~liuche/bug-1173388/flush-beta.apk > > This is a build of the patch from bug 1118818 on top of a pull from beta > this morning. I can not reproduce the issue using this build.
Flags: needinfo?(flaviu.cos)
The patch from bug 1118818 is landed in beta, and that should fix this, so I'm duping this.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
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: