Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(187)

Issue 1325613002: Remove StaleWhileRevalidateCacheControl feature. (Closed)

Created:
5 years, 3 months ago by Adam Rice
Modified:
5 years, 3 months ago
Reviewers:
tkent, yhirano
CC:
blink-reviews, dglazkov+blink
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

Remove StaleWhileRevalidateCacheControl feature. This feature will not be used in Blink in the immediate future, so remove it. There are no callers or users. BUG=348877 TEST=compiled Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201546

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -8 lines) Patch
M Source/platform/RuntimeEnabledFeatures.in View 1 chunk +0 lines, -1 line 0 comments Download
M Source/web/WebRuntimeFeatures.cpp View 1 chunk +0 lines, -5 lines 0 comments Download
M public/web/WebRuntimeFeatures.h View 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 13 (5 generated)
Adam Rice
5 years, 3 months ago (2015-08-31 00:32:46 UTC) #2
yhirano
LGTM.
5 years, 3 months ago (2015-08-31 05:13:44 UTC) #3
Adam Rice
+tkent for WebKit OWNERS.
5 years, 3 months ago (2015-09-01 02:24:49 UTC) #5
tkent
lgtm
5 years, 3 months ago (2015-09-01 02:26:29 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1325613002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1325613002/1
5 years, 3 months ago (2015-09-01 02:43:40 UTC) #8
commit-bot: I haz the power
Try jobs failed on following builders: linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_rel_ng/builds/63155)
5 years, 3 months ago (2015-09-01 06:05:10 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1325613002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1325613002/1
5 years, 3 months ago (2015-09-01 09:57:12 UTC) #12
commit-bot: I haz the power
5 years, 3 months ago (2015-09-01 10:02:17 UTC) #13
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201546

Powered by Google App Engine
This is Rietveld 408576698