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

Unified Diff: chrome/browser/extensions/api/file_system/consent_provider.h

Issue 2934143002: Move chrome.fileSystem implementation to //extensions (Closed)
Patch Set: benwells, test fix 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.h
diff --git a/chrome/browser/extensions/api/file_system/consent_provider.h b/chrome/browser/extensions/api/file_system/consent_provider.h
index 3a517b6f68d8296547812c32ab50f57d2c7d023f..b677b2f38c61f1223653e5a89304c65f70c5437b 100644
--- a/chrome/browser/extensions/api/file_system/consent_provider.h
+++ b/chrome/browser/extensions/api/file_system/consent_provider.h
@@ -42,6 +42,7 @@ class ConsentProvider {
public:
// Shows a dialog for granting permissions.
virtual void ShowDialog(const Extension& extension,
+ content::RenderFrameHost* host,
const base::WeakPtr<file_manager::Volume>& volume,
bool writable,
const ShowDialogCallback& callback) = 0;
@@ -66,6 +67,7 @@ class ConsentProvider {
// volume by the |extension|. Must be called only if the extension is
// grantable, which can be checked with IsGrantable().
void RequestConsent(const Extension& extension,
+ content::RenderFrameHost* host,
const base::WeakPtr<file_manager::Volume>& volume,
bool writable,
const ConsentCallback& callback);
@@ -83,7 +85,7 @@ class ConsentProvider {
// context of running extensions) for ConsentProvider.
class ConsentProviderDelegate : public ConsentProvider::DelegateInterface {
public:
- ConsentProviderDelegate(Profile* profile, content::RenderFrameHost* host);
+ explicit ConsentProviderDelegate(Profile* profile);
~ConsentProviderDelegate();
private:
@@ -95,6 +97,7 @@ class ConsentProviderDelegate : public ConsentProvider::DelegateInterface {
// ConsentProvider::DelegateInterface overrides:
void ShowDialog(const Extension& extension,
+ content::RenderFrameHost* host,
const base::WeakPtr<file_manager::Volume>& volume,
bool writable,
const file_system_api::ConsentProvider::ShowDialogCallback&
@@ -106,7 +109,6 @@ class ConsentProviderDelegate : public ConsentProvider::DelegateInterface {
bool IsWhitelistedComponent(const Extension& extension) override;
Profile* const profile_;
- content::RenderFrameHost* const host_;
DISALLOW_COPY_AND_ASSIGN(ConsentProviderDelegate);
};

Powered by Google App Engine
This is Rietveld 408576698