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

Unified Diff: chrome/browser/extensions/api/web_request/web_request_apitest.cc

Issue 1739183003: Make extensions::DictionaryBuilder and extensions::ListValue unmovable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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: chrome/browser/extensions/api/web_request/web_request_apitest.cc
diff --git a/chrome/browser/extensions/api/web_request/web_request_apitest.cc b/chrome/browser/extensions/api/web_request/web_request_apitest.cc
index 375fe6f1392760df45495a751f7325936f74b607..5f510f94fb1e016d8d824450855399d0994abac0 100644
--- a/chrome/browser/extensions/api/web_request/web_request_apitest.cc
+++ b/chrome/browser/extensions/api/web_request/web_request_apitest.cc
@@ -458,15 +458,18 @@ IN_PROC_BROWSER_TEST_F(ExtensionWebRequestApiTest, HostedAppRequest) {
"/extensions/api_test/webrequest_hosted_app/index.html"));
scoped_refptr<Extension> hosted_app =
ExtensionBuilder()
- .SetManifest(std::move(
+ .SetManifest(
DictionaryBuilder()
.Set("name", "Some hosted app")
.Set("version", "1")
.Set("manifest_version", 2)
- .Set("app",
- std::move(DictionaryBuilder().Set(
- "launch", std::move(DictionaryBuilder().Set(
- "web_url", hosted_app_url.spec())))))))
+ .Set("app", DictionaryBuilder()
+ .Set("launch", DictionaryBuilder()
+ .Set("web_url",
+ hosted_app_url.spec())
+ .Build())
+ .Build())
+ .Build())
.Build();
ExtensionSystem::Get(browser()->profile())
->extension_service()

Powered by Google App Engine
This is Rietveld 408576698