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

Unified Diff: third_party/WebKit/Source/modules/serviceworkers/InstallEvent.cpp

Issue 2810073002: Blink Rename follow-up: URLs --big-rename--> _ur_ls --this-fix--> _urls. (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/serviceworkers/InstallEvent.cpp
diff --git a/third_party/WebKit/Source/modules/serviceworkers/InstallEvent.cpp b/third_party/WebKit/Source/modules/serviceworkers/InstallEvent.cpp
index e91c724a80dad29e39737c2261ef3d64fd1e1256..acc152e350dd8c01f2a7d542364c47b0bb9376b3 100644
--- a/third_party/WebKit/Source/modules/serviceworkers/InstallEvent.cpp
+++ b/third_party/WebKit/Source/modules/serviceworkers/InstallEvent.cpp
@@ -68,27 +68,27 @@ void InstallEvent::registerForeignFetch(ScriptState* script_state,
return;
}
const Vector<String>& sub_scopes = options.scopes();
- Vector<KURL> sub_scope_ur_ls(sub_scopes.size());
+ Vector<KURL> sub_scope_urls(sub_scopes.size());
for (size_t i = 0; i < sub_scopes.size(); ++i) {
- sub_scope_ur_ls[i] = execution_context->CompleteURL(sub_scopes[i]);
- if (!sub_scope_ur_ls[i].IsValid()) {
+ sub_scope_urls[i] = execution_context->CompleteURL(sub_scopes[i]);
+ if (!sub_scope_urls[i].IsValid()) {
exception_state.ThrowTypeError("Invalid subscope URL: " + sub_scopes[i]);
return;
}
- sub_scope_ur_ls[i].RemoveFragmentIdentifier();
- if (!origin->CanRequest(sub_scope_ur_ls[i])) {
+ sub_scope_urls[i].RemoveFragmentIdentifier();
+ if (!origin->CanRequest(sub_scope_urls[i])) {
exception_state.ThrowTypeError("Subscope URL is not within scope: " +
sub_scopes[i]);
return;
}
- String sub_scope_path = sub_scope_ur_ls[i].GetPath();
+ String sub_scope_path = sub_scope_urls[i].GetPath();
if (!sub_scope_path.StartsWith(scope_path)) {
exception_state.ThrowTypeError("Subscope URL is not within scope: " +
sub_scopes[i]);
return;
}
}
- client->RegisterForeignFetchScopes(sub_scope_ur_ls, parsed_origins);
+ client->RegisterForeignFetchScopes(sub_scope_urls, parsed_origins);
}
const AtomicString& InstallEvent::InterfaceName() const {

Powered by Google App Engine
This is Rietveld 408576698