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

Unified Diff: chrome/browser/renderer_host/pepper/pepper_flash_browser_host.h

Issue 11705003: Change PepperFlashBrowserHost to use CookieSettings to get the LSO settings (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years 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/renderer_host/pepper/pepper_flash_browser_host.h
diff --git a/chrome/browser/renderer_host/pepper/pepper_flash_browser_host.h b/chrome/browser/renderer_host/pepper/pepper_flash_browser_host.h
index 6b38c368941fb9eb22dad05f7e4e2daadd88f8d9..4bb3b9756395a69c06af46d1e50a97bd94dcb434 100644
--- a/chrome/browser/renderer_host/pepper/pepper_flash_browser_host.h
+++ b/chrome/browser/renderer_host/pepper/pepper_flash_browser_host.h
@@ -6,7 +6,9 @@
#define CHROME_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FLASH_BROWSER_HOST_H_
#include "base/basictypes.h"
+#include "base/memory/ref_counted.h"
#include "base/memory/weak_ptr.h"
+#include "ppapi/c/private/ppb_flash.h"
#include "ppapi/host/host_message_context.h"
#include "ppapi/host/resource_host.h"
@@ -14,9 +16,12 @@ namespace base {
class Time;
}
+namespace chrome {
+class FlashLSOSettingsHelper;
yzshen1 2012/12/29 02:04:05 nit: you could merge the namespace with the one be
raymes 2012/12/29 21:15:40 Done.
+}
+
namespace content {
class BrowserPpapiHost;
-class ResourceContext;
}
class GURL;
@@ -43,16 +48,13 @@ class PepperFlashBrowserHost : public ppapi::host::ResourceHost {
int32_t OnMsgGetLocalDataRestrictions(
ppapi::host::HostMessageContext* context);
- void GetLocalDataRestrictions(ppapi::host::ReplyMessageContext reply_context,
- const GURL& document_url,
- const GURL& plugin_url,
- content::ResourceContext* resource_context);
+ void OnGetLocalDataRestrictions(
+ ppapi::host::ReplyMessageContext reply_context,
yzshen1 2012/12/29 02:04:05 Any reason to pass a value instead of a ref?
raymes 2012/12/29 21:15:40 Done.
+ PP_FlashLSORestrictions restrictions);
content::BrowserPpapiHost* host_;
- int render_process_id_;
- // For fetching the Flash LSO settings.
- content::ResourceContext* resource_context_;
base::WeakPtrFactory<PepperFlashBrowserHost> weak_factory_;
+ scoped_refptr<FlashLSOSettingsHelper> lso_settings_helper_;
DISALLOW_COPY_AND_ASSIGN(PepperFlashBrowserHost);
};

Powered by Google App Engine
This is Rietveld 408576698