Closed
Bug 1267195
Opened 8 years ago
Closed 8 years ago
Another instance of 'indication of amount of content cache does not update when click [Clear Now] button preferences'
Categories
(Core :: Networking: Cache, defect)
Tracking
()
RESOLVED
FIXED
mozilla49
People
(Reporter: abhilashmandaliya, Assigned: mayhemer)
References
Details
(Whiteboard: [necko-active])
Attachments
(1 file)
1.04 KB,
patch
|
michal
:
review+
ritu
:
approval-mozilla-aurora+
ritu
:
approval-mozilla-beta-
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Firefox/45.0 Build ID: 20160407164938 Steps to reproduce: tried to clear the cache using options->advanced->network->clear now. I am using version 45.0.2. Actual results: I had to click several time to clear the cache data. Expected results: it should clear all data in single click.
Comment 1•8 years ago
|
||
User Agent Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:49.0) Gecko/20100101 Firefox/49.0 Build ID 20160428030218 Reproduced on Firefox 45.0.2, Firefox 46 and Nightly 49 on Windows 10.
Status: UNCONFIRMED → NEW
Component: Untriaged → Networking: Cache
Ever confirmed: true
OS: Unspecified → Windows 10
Product: Firefox → Core
Hardware: Unspecified → x86_64
Assignee | ||
Comment 2•8 years ago
|
||
(In reply to Liviu Cirdei [:liviucirdei] from comment #1) > User Agent Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:49.0) > Gecko/20100101 Firefox/49.0 > Build ID 20160428030218 > > Reproduced on Firefox 45.0.2, Firefox 46 and Nightly 49 on Windows 10. Can you tell me how exactly you reproduced this? What did you expect and what happened?
Flags: needinfo?(liviu.cirdei)
Comment 3•8 years ago
|
||
I clicked "Clear Now" button and I expected to see all the cache being cleared at the first click. For example if I had 120 Mb of cache, on a click I expect to have 0 Mb of cache. Instead at first click the cache value decreases to another number (e.g 80mb), at a second click decreases again and so on until I get to 0 Mb of cache.
Flags: needinfo?(liviu.cirdei)
Assignee | ||
Comment 4•8 years ago
|
||
(In reply to Liviu Cirdei [:liviucirdei] from comment #3) > I clicked "Clear Now" button and I expected to see all the cache being > cleared at the first click. For example if I had 120 Mb of cache Where do you see this number? Generally speaking, where/how do you check how much space the cache occupies? > , on a click > I expect to have 0 Mb of cache. Instead at first click the cache value > decreases to another number (e.g 80mb), at a second click decreases again > and so on until I get to 0 Mb of cache.
Flags: needinfo?(liviu.cirdei)
Assignee | ||
Comment 5•8 years ago
|
||
Also a note that deletion is made asynchronously in background. We first rename the cache2/entries dir to cache2/trash.nnnn and delete the trash dir in background.
Comment 6•8 years ago
|
||
I see the number of cache data going to Options->Advanced->Network --> Cached Web content.
Flags: needinfo?(liviu.cirdei)
Assignee | ||
Comment 7•8 years ago
|
||
(In reply to Liviu Cirdei [:liviucirdei] from comment #6) > I see the number of cache data going to Options->Advanced->Network --> > Cached Web content. Aha, thanks. OK, few more questions: - I presume you refresh the Options page to update this number? - how long after you pressed the Clear Now button did you do that? - how many times? (if more than once) - does it ever show "Calculating..." instead of a number? - after the number drops to say those 80MB, does it ever start dropping again by itself? - how long did you wait before pressing the Clear Now button again? - are there any web heavier pages (like facebook,gmail,news pages) being open during the deletion process? - can you reproduce with a clear profile (or at least with no add-ons installed?) Disregard comment 5 please. We no longer do that for user initiated cache clear, but only for clear-cache-on-shutdown option turned on, I realized.
Flags: needinfo?(liviu.cirdei)
Comment 8•8 years ago
|
||
This is what I do: 1. Using a clean profile with no add-ons installed I open Firefox and open some pages to have some data in the cache. I use heavy pages like Facebook, gmail, youtube, etc 2. I open Options>Advanced Network page --> I see the cache number (if I refresh the page at this point I have the same number or sometimes the number increases). 4. I click "Clear Now" --> The number decreases instantly (I don't have to refresh the page) e.g from 100 mb to 80mb. 5. If I wait 30s-1 min the number remains the same (80 mb) and if I click again "Clear Now" it decreases again or in some cases it goes to 0 mb. Every time at least two clicks have to be made in order to have 0 mb of data cache. Also I can see "Calculating..." sometimes if I go to Options>Advanced>Network immediately after I opened the pages. And the pages are opened during the deletion process.
Flags: needinfo?(liviu.cirdei)
Assignee | ||
Comment 9•8 years ago
|
||
What happens when you: - have data in the cache - go to the Options/../Networking page (optionally wait for "Calculating..." text to become an actual number) - click the Clear Now button - close the Options page - open Options/../Networking again ?
Flags: needinfo?(liviu.cirdei)
Assignee | ||
Comment 10•8 years ago
|
||
We just update the UI too soon. This problem has recently been fixed and is now broken again. Have to investigate what's exactly wrong.
Assignee: nobody → honzab.moz
Flags: needinfo?(liviu.cirdei)
Summary: cache can't be cleared in one click. It needs several clicks to clear 350 MB of cache data → Another instance of 'indication of amount of content cache does not update when click [Clear Now] button preferences'
Updated•8 years ago
|
Whiteboard: [necko-active]
Assignee | ||
Comment 11•8 years ago
|
||
Cause: on one place we were missing a check on the block flag, added.
Status: NEW → ASSIGNED
Assignee | ||
Comment 12•8 years ago
|
||
- we often call CacheIndex::ChangeState -> WRITING during deletion (a bug?) which was causing trigger of the notification to UI too early - added check for the mAsyncGetDiskConsumptionBlocked flag also on that place
Attachment #8751777 -
Flags: review?(michal.novotny)
Updated•8 years ago
|
Attachment #8751777 -
Flags: review?(michal.novotny) → review+
Comment 13•8 years ago
|
||
(In reply to Honza Bambas (:mayhemer) from comment #12) > - we often call CacheIndex::ChangeState -> WRITING during deletion (a bug?) That's not a bug, but we could postpone writing index until eviction finishes.
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 14•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8adc9b314e78
Keywords: checkin-needed
Comment 15•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8adc9b314e78
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Comment 16•8 years ago
|
||
This is still an active problem with 47.0b7. It has been for more than a month. With a cache value of 1GB, it will usually take more than 2 dozen clicks of the <Clear Cache> button to fully clear the cache.
Assignee | ||
Comment 19•8 years ago
|
||
(In reply to Patrick McManus [:mcmanus] from comment #17) > uplifts? Yep.
Flags: needinfo?(honzab.moz)
Assignee | ||
Comment 20•8 years ago
|
||
Comment on attachment 8751777 [details] [diff] [review] 1267195-cache-delete-block-notify-2.patch Approval Request Comment [Feature/regressing bug #]: HTTP cache v2 [User impact if declined]: Incorrect report of cache size when clearing cache from preferences (people falsely believe the mechanism doesn't work at all) ; bad UX [Describe test coverage new/current, TreeHerder]: I think none [Risks and why]: zero :) the patch is super simple [String/UUID change made/needed]: none
Attachment #8751777 -
Flags: approval-mozilla-beta?
Attachment #8751777 -
Flags: approval-mozilla-aurora?
Comment 21•8 years ago
|
||
When you say RESOLVED, I take it to mean that you have a fix in mind but just haven't pushed it out to users yet? I'm still seeing the problem on my end.
Comment 22•8 years ago
|
||
(In reply to Mike Aguilar from comment #21) > When you say RESOLVED, I take it to mean that you have a fix in mind but > just haven't pushed it out to users yet? I'm still seeing the problem on my > end. mike - its resolved on the target milestoe (which is firefox 49 nightly). are you testing with the latest nightly? under "tracking-flags" on this bug you can see what other channels the fix has been ported to. (so far none, but there are requests in comment 20 to start that process.)
Comment 23•8 years ago
|
||
Patrick, I'm in Firefox 47. Nightly wasn't working with the CMS front-end that I was using for most of my work last I checked. When I get back from paying my bills, I'll see if it works with the WP CMS I use currently. I hope it does, since I actually prefer Nightly because it's faster.
Comment on attachment 8751777 [details] [diff] [review] 1267195-cache-delete-block-notify-2.patch While this is not a recent regression (since Fx45), the fix seems simple enough, Aurora48+, Beta47+
Attachment #8751777 -
Flags: approval-mozilla-beta?
Attachment #8751777 -
Flags: approval-mozilla-beta+
Attachment #8751777 -
Flags: approval-mozilla-aurora?
Attachment #8751777 -
Flags: approval-mozilla-aurora+
Hi Mike, this fix should be in 47.0b9, which will be pushed to Beta channel on Friday 5/27. Could you please verify it then and let us know if this is fixed as expected? Thanks!
Flags: needinfo?(rivermikerat)
Comment 26•8 years ago
|
||
Yep. No problem.
Comment 27•8 years ago
|
||
Just did the 0b8 update and now it's sticking at 46k. Still requiring multiple clicks, but now won't clear all the way to zero. I'll update this thread when the 0b9 update comes out. I'm going to close FF and check to see if Nightly works correctly with my primary CMS platform.
This is hitting conflicts on uplift to beta. Can we get a rebased patch for it? I'll be landing the aurora uplift momentarily.
Flags: needinfo?(honzab.moz)
Comment 29•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/15264dfb8421
Comment 30•8 years ago
|
||
My WP CMS seems to work in Nightly 49.0a1, but the clear cache is getting stuck as 46k, no matter how many times I click it. First click causes the "Calculating....." to show and then it quickly drops to 46 from whatever it was.
Flags: needinfo?(rivermikerat)
Assignee | ||
Comment 31•8 years ago
|
||
Re beta conflicts: we need to uplift bug 1066970 first.
Flags: needinfo?(honzab.moz)
Updated•8 years ago
|
Comment on attachment 8751777 [details] [diff] [review] 1267195-cache-delete-block-notify-2.patch Since I decided not to uplift bug 1066970, I am changing my decision on this one too. It is an old regression and now that we are entering RC week in 2 days, it's safer not to uplift this to Beta47.
Attachment #8751777 -
Flags: approval-mozilla-beta+ → approval-mozilla-beta-
Comment 33•8 years ago
|
||
So you're just going to leave us hanging with this problem?
Assignee | ||
Comment 34•8 years ago
|
||
(In reply to Mike Aguilar from comment #33) > So you're just going to leave us hanging with this problem? There is actually no critical problem there. It's just that the UI is incorrectly updated, your cache clears anyway.
Comment 35•8 years ago
|
||
How can we be sure of that? Not being able to trust what the UI is displaying to us seems pretty critical to me. If it's displaying that information incorrectly, how do we know it's not displaying other things correctly? 0b9 it's still an issue.
You need to log in
before you can comment on or make changes to this bug.
Description
•