New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 808816 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Files loaded "from disk cache" although "disable cache" is set

Reported by t...@tobireif.com, Feb 3 2018

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.140 Safari/537.36

Steps to reproduce the problem:
1. With the Network tab open (and "disable cache") load https://tobireif.com/demos/grid/ .
2. Navigate away, eg by clicking on "tobi​reif​.com".
3. Click on Chrome's back button.

What is the expected behavior?
When "disable cache" is set, no file should get loaded from cache .

What went wrong?
In the Network tab there are several files listed as loaded "from disk cache", although "disable cache" is set.

Did this work before? N/A 

Chrome version: 64.0.3282.140  Channel: stable
OS Version: OS X 10.13.3
Flash Version:
 
screenshot.png
180 KB View Download
Labels: Needs-Triage-M64
Able to reproduce the issue on reported version 64.0.3282.140 and the same is not seen on latest canary 66.0.3340.0 using Windows 10, Ubuntu 14.04, Mac 10.13.1 hence providing reverse bisect info

Reverse Bisect Info:
================
Last Bad build: 65.0.3323.0
First Good build: 65.0.3324.0

You are probably looking for a change made after 529589 (known good), but no later than 529590 (first known bad).

https://chromium.googlesource.com/chromium/src/+log/a27d2ed81f77433719db21e6cd7678c89758664d..2c7c1f3f6cdd3926f1074ad69b23537402daeef4

Reviewed on: https://chromium-review.googlesource.com/865157

@Andrey Kosyakov: Please confirm the issue and help in re-assigning if it is not related to your change, please merge it to M-64 and M-65
Adding ReleaseBlock-Stable as it seems recent break, feel free to remove it if not applicable.

Thanks!
Cc: viswatej...@techmahindra.com
Labels: -Type-Bug -Pri-2 hasbisect-per-revision ReleaseBlock-Stable Triaged-ET RegressedIn-65 M-64 M-65 FoundIn-65 FoundIn-64 Target-65 Target-64 OS-Linux OS-Windows Pri-1 Type-Bug-Regression
Owner: caseq@chromium.org
Status: Assigned (was: Unconfirmed)
Labels: OS-Chrome
Able to reproduce the issue on Chrome 64.0.3282.144/ CrOS 10176.68.0 -Peppy

Comment 5 by caseq@chromium.org, Feb 6 2018

Labels: -Needs-Triage-M64 -RegressedIn-65 -Target-64 Merge-Request-65
I'm sorry, but how is this an RB-S if the bug is already reported in stable? I don't think the merge to M-64 is justified considering moderate severity of the bug and the fact that we've had it since PlzNavigate was enabled. This can be merged into m65, though, I think.

Comment 6 by caseq@chromium.org, Feb 6 2018

Labels: -ReleaseBlock-Stable

Comment 7 by caseq@chromium.org, Feb 6 2018

Status: Fixed (was: Assigned)
Marking as Fixed since it's not present on ToT
Project Member

Comment 8 by sheriffbot@chromium.org, Feb 7 2018

Labels: -Merge-Request-65 Hotlist-Merge-Approved Merge-Approved-65
Your change meets the bar and is auto-approved for M65. Please go ahead and merge the CL to branch 3325 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Pls merge your change to M65 branch 3325 before 2:00 PM PT tomorrow, Wednesday (02/07/18). Thank you.
Labels: -M-65 -Merge-Approved-65 -Target-65
Ouch -- upon a closer look, I realized this is already part of m65 (i.e. was landed before the branch point), so no merge is necessary (#2 states this as well). Apologies for confusion!

Sign in to add a comment