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

Issue 1850123002: Rename WebFrameLoadType::ReloadFromOrigin to ReloadBypassingCache (Closed)

Created:
4 years, 8 months ago by Takashi Toyoshima
Modified:
4 years, 8 months ago
CC:
apavlov+blink_chromium.org, blink-reviews, blink-reviews-api_chromium.org, caseq+blink_chromium.org, chromium-reviews, creis+watch_chromium.org, darin-cc_chromium.org, devtools-reviews_chromium.org, dglazkov+blink, gavinp+loader_chromium.org, jam, Nate Chapin, kozyatinskiy+blink_chromium.org, loading-reviews_chromium.org, lushnikov+blink_chromium.org, mkwst+moarreviews-renderer_chromium.org, mlamouri+watch-content_chromium.org, nasko+codewatch_chromium.org, pfeldman+blink_chromium.org, sergeyv+blink_chromium.org, tyoshino+watch_chromium.org, Yoav Weiss
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Rename WebFrameLoadType::ReloadFromOrigin to ReloadBypassingCache Other types, e.g. CachePolicy, uses 'BypassingCache', and WebFrameLoadType and FrameLoadType is only places to use 'FromOrigin'. To use consistent term in chrome/blink, rename it to ReloadBypassingCache. There are several type conversion code between WebFrameLoadType, FrameLoadType, CachePolicy, and boolean to specify the reload type. This cleanup is important to avoid unnecessary confusion, and eventually remove reload() API to migrate it to load(). Also we use a word 'ReloadOriginalRequest' in content to express a different thing. So, 'Origin' is a bit confusing. BUG=599364 Committed: https://crrev.com/4988d88eb8c53b454d454cc9bceb8128cacc1fab Cr-Commit-Position: refs/heads/master@{#384526}

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -23 lines) Patch
M content/renderer/render_frame_impl.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/core/frame/LocalFrame.cpp View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/core/inspector/InspectorPageAgent.h View 1 chunk +1 line, -0 lines 1 comment Download
M third_party/WebKit/Source/core/inspector/InspectorPageAgent.cpp View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/core/loader/FrameFetchContext.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/core/loader/FrameFetchContextTest.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/core/loader/FrameLoader.cpp View 6 chunks +7 lines, -7 lines 0 comments Download
M third_party/WebKit/Source/core/loader/FrameLoaderTypes.h View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/core/testing/Internals.h View 1 chunk +1 line, -1 line 1 comment Download
M third_party/WebKit/Source/core/testing/Internals.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/web/AssertMatchingEnums.cpp View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/web/RemoteFrameClientImpl.cpp View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/web/WebLocalFrameImpl.cpp View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/public/web/WebFrameLoadType.h View 1 chunk +1 line, -1 line 1 comment Download

Dependent Patchsets:

Messages

Total messages: 15 (8 generated)
Takashi Toyoshima
Jochen, can you take a look? I split this one mechanical change from https://codereview.chromium.org/1846653004/. https://codereview.chromium.org/1850123002/diff/1/third_party/WebKit/Source/core/inspector/InspectorPageAgent.h ...
4 years, 8 months ago (2016-04-01 07:22:31 UTC) #3
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1850123002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1850123002/1
4 years, 8 months ago (2016-04-01 07:22:55 UTC) #5
jochen (gone - plz use gerrit)
lgtm
4 years, 8 months ago (2016-04-01 08:36:56 UTC) #7
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 8 months ago (2016-04-01 09:00:49 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1850123002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1850123002/1
4 years, 8 months ago (2016-04-01 09:07:56 UTC) #11
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 8 months ago (2016-04-01 09:13:17 UTC) #13
commit-bot: I haz the power
4 years, 8 months ago (2016-04-01 09:14:30 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/4988d88eb8c53b454d454cc9bceb8128cacc1fab
Cr-Commit-Position: refs/heads/master@{#384526}

Powered by Google App Engine
This is Rietveld 408576698