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

Unified Diff: chrome/browser/extensions/extension_disabled_ui_browsertest.cc

Issue 2314363002: extensions: Change ScopedTempDir::path() to GetPath() (Closed)
Patch Set: Comment addressed Created 4 years, 3 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/extension_disabled_ui_browsertest.cc
diff --git a/chrome/browser/extensions/extension_disabled_ui_browsertest.cc b/chrome/browser/extensions/extension_disabled_ui_browsertest.cc
index 7b4fff1e8a3f93d6e16544a894859adfcf4f876f..0e11967e0d7df31664847d142095106c15fd45b0 100644
--- a/chrome/browser/extensions/extension_disabled_ui_browsertest.cc
+++ b/chrome/browser/extensions/extension_disabled_ui_browsertest.cc
@@ -63,18 +63,15 @@ class ExtensionDisabledGlobalErrorTest : public ExtensionBrowserTest {
const base::FilePath pem_path = test_dir.AppendASCII("permissions.pem");
path_v1_ = PackExtensionWithOptions(
test_dir.AppendASCII("v1"),
- scoped_temp_dir_.path().AppendASCII("permissions1.crx"),
- pem_path,
+ scoped_temp_dir_.GetPath().AppendASCII("permissions1.crx"), pem_path,
base::FilePath());
path_v2_ = PackExtensionWithOptions(
test_dir.AppendASCII("v2"),
- scoped_temp_dir_.path().AppendASCII("permissions2.crx"),
- pem_path,
+ scoped_temp_dir_.GetPath().AppendASCII("permissions2.crx"), pem_path,
base::FilePath());
path_v3_ = PackExtensionWithOptions(
test_dir.AppendASCII("v3"),
- scoped_temp_dir_.path().AppendASCII("permissions3.crx"),
- pem_path,
+ scoped_temp_dir_.GetPath().AppendASCII("permissions3.crx"), pem_path,
base::FilePath());
}
@@ -242,7 +239,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionDisabledGlobalErrorTest,
.AppendASCII("updates.xml"));
interceptor.SetResponseIgnoreQuery(
GURL("http://localhost/autoupdate/v2.crx"),
- scoped_temp_dir_.path().AppendASCII("permissions2.crx"));
+ scoped_temp_dir_.GetPath().AppendASCII("permissions2.crx"));
sync_service->MergeDataAndStartSyncing(
syncer::EXTENSIONS, syncer::SyncDataList(),
@@ -282,7 +279,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionDisabledGlobalErrorTest, RemoteInstall) {
.AppendASCII("updates.xml"));
interceptor.SetResponseIgnoreQuery(
GURL("http://localhost/autoupdate/v2.crx"),
- scoped_temp_dir_.path().AppendASCII("permissions2.crx"));
+ scoped_temp_dir_.GetPath().AppendASCII("permissions2.crx"));
sync_pb::EntitySpecifics specifics;
specifics.mutable_extension()->set_id(extension_id);

Powered by Google App Engine
This is Rietveld 408576698