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)
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)
(deleted),
text/plain
|
Details |
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.
Reporter | ||
Updated•9 years ago
|
Summary: Cookies settings are not allways kept after browser restart → Cookies settings are not always kept after browser restart
Reporter | ||
Updated•9 years ago
|
status-firefox38.0.5:
--- → unaffected
status-firefox39:
--- → affected
status-firefox40:
--- → unaffected
status-firefox41:
--- → unaffected
Updated•9 years ago
|
tracking-fennec: --- → ?
Comment 1•9 years ago
|
||
[Tracking Requested - why for this release]: regression
Ni to Margaret for an assignee
Comment 2•9 years ago
|
||
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)
Comment 3•9 years ago
|
||
See also Bug 1118818, Bug 1025560.
Assignee | ||
Comment 4•9 years ago
|
||
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)
Comment 5•9 years ago
|
||
If you can verify that this doesn't repro in 40+, then uplifting that fix is safe IMO. Run with it!
Assignee | ||
Comment 6•9 years ago
|
||
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)
Assignee | ||
Comment 7•9 years ago
|
||
My STR:
- Open Settings > Privacy > Cookies
- Switch from "Enabled" to "Disabled"
- Swipe close from recent apps
- Re-open, go to Settings > Privacy > Cookies
Reporter | ||
Comment 8•9 years ago
|
||
(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.
Assignee | ||
Comment 9•9 years ago
|
||
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)
Reporter | ||
Comment 10•9 years ago
|
||
(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)
Assignee | ||
Comment 11•9 years ago
|
||
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
Marking this as fixed based on bug 1118818.
Updated•4 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
•