New issue
Advanced search Search tips

Issue 714574 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Reorganize ResourceFetcher::DetermineRevalidationPolicy

Project Member Reported by yhirano@chromium.org, Apr 24 2017

Issue description

...in order to make it more understandable than the current state, which is a mere random list of conditions.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 25 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/58197789bfafee408d8fc728c0ed2f1da81bbbed

commit 58197789bfafee408d8fc728c0ed2f1da81bbbed
Author: yhirano <yhirano@chromium.org>
Date: Tue Apr 25 09:03:39 2017

Move |is_static_data| check down in DetermineRevalidationPolicy

This CL moves

 if (is_static_data)
   return kUse;

section down in DetermineRevalidationPolicy as a preparation to factor out some
|return kReload;| statements.

BUG= 652228 , 714574
R=hiroshige@chromium.org

Review-Url: https://codereview.chromium.org/2829043002
Cr-Commit-Position: refs/heads/master@{#466931}

[modify] https://crrev.com/58197789bfafee408d8fc728c0ed2f1da81bbbed/third_party/WebKit/Source/platform/loader/fetch/ResourceFetcher.cpp

Project Member

Comment 2 by sheriffbot@chromium.org, Apr 25 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Available (was: Untriaged)

Sign in to add a comment