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

Unified Diff: chrome/browser/extensions/api/file_system/consent_provider_unittest.cc

Issue 2934143002: Move chrome.fileSystem implementation to //extensions (Closed)
Patch Set: devlin Created 3 years, 6 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/file_system/consent_provider_unittest.cc
diff --git a/chrome/browser/extensions/api/file_system/consent_provider_unittest.cc b/chrome/browser/extensions/api/file_system/consent_provider_unittest.cc
index 1576915349798dd46734be67ddf3fcffd3b2178f..23782c4dfb9ffca8d16098acaf13980483dfef13 100644
--- a/chrome/browser/extensions/api/file_system/consent_provider_unittest.cc
+++ b/chrome/browser/extensions/api/file_system/consent_provider_unittest.cc
@@ -60,6 +60,7 @@ class TestingConsentProviderDelegate
// ConsentProvider::DelegateInterface overrides:
void ShowDialog(
const extensions::Extension& extension,
+ content::RenderFrameHost* host,
const base::WeakPtr<Volume>& volume,
bool writable,
const ConsentProvider::ShowDialogCallback& callback) override {
@@ -153,8 +154,8 @@ TEST_F(FileSystemApiConsentProviderTest, ForNonKioskApps) {
EXPECT_TRUE(provider.IsGrantable(*whitelisted_component_extension));
ConsentProvider::Consent result = ConsentProvider::CONSENT_IMPOSSIBLE;
- provider.RequestConsent(*whitelisted_component_extension.get(), volume_,
- true /* writable */,
+ provider.RequestConsent(*whitelisted_component_extension.get(), nullptr,
+ volume_, true /* writable */,
base::Bind(&OnConsentReceived, &result));
base::RunLoop().RunUntilIdle();
@@ -196,7 +197,7 @@ TEST_F(FileSystemApiConsentProviderTest, ForKioskApps) {
EXPECT_TRUE(provider.IsGrantable(*auto_launch_kiosk_app));
ConsentProvider::Consent result = ConsentProvider::CONSENT_IMPOSSIBLE;
- provider.RequestConsent(*auto_launch_kiosk_app.get(), volume_,
+ provider.RequestConsent(*auto_launch_kiosk_app.get(), nullptr, volume_,
true /* writable */,
base::Bind(&OnConsentReceived, &result));
base::RunLoop().RunUntilIdle();
@@ -226,7 +227,7 @@ TEST_F(FileSystemApiConsentProviderTest, ForKioskApps) {
EXPECT_TRUE(provider.IsGrantable(*manual_launch_kiosk_app));
ConsentProvider::Consent result = ConsentProvider::CONSENT_IMPOSSIBLE;
- provider.RequestConsent(*manual_launch_kiosk_app.get(), volume_,
+ provider.RequestConsent(*manual_launch_kiosk_app.get(), nullptr, volume_,
true /* writable */,
base::Bind(&OnConsentReceived, &result));
base::RunLoop().RunUntilIdle();
@@ -245,7 +246,7 @@ TEST_F(FileSystemApiConsentProviderTest, ForKioskApps) {
EXPECT_TRUE(provider.IsGrantable(*manual_launch_kiosk_app));
ConsentProvider::Consent result = ConsentProvider::CONSENT_IMPOSSIBLE;
- provider.RequestConsent(*manual_launch_kiosk_app.get(), volume_,
+ provider.RequestConsent(*manual_launch_kiosk_app.get(), nullptr, volume_,
true /* writable */,
base::Bind(&OnConsentReceived, &result));
base::RunLoop().RunUntilIdle();

Powered by Google App Engine
This is Rietveld 408576698