Change PepperFlashBrowserHost to use CookieSettings to get the LSO settings

chrome/browser/renderer_host/pepper/pepper_flash_browser_host.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FLASH_BROWSER_HOST_H_
 #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"
 
 namespace base {
 class Time;
 }
 
+namespace chrome {
+class FlashLSOSettingsHelper;

[yzshen1, 2012/12/29 02:04:05]

nit: you could merge the namespace with the one below.

[raymes, 2012/12/29 21:15:40]

Done.

+}
+
 namespace content {
 class BrowserPpapiHost;
-class ResourceContext;
 }
 
 class GURL;
 
 namespace chrome {
 
 class PepperFlashBrowserHost : public ppapi::host::ResourceHost {
  public:
   PepperFlashBrowserHost(content::BrowserPpapiHost* host,
                          PP_Instance instance,
                          PP_Resource resource);
   virtual ~PepperFlashBrowserHost();
 
   // ppapi::host::ResourceHost override.
   virtual int32_t OnResourceMessageReceived(
       const IPC::Message& msg,
       ppapi::host::HostMessageContext* context) OVERRIDE;
 
  private:
   int32_t OnMsgUpdateActivity(ppapi::host::HostMessageContext* host_context);
   int32_t OnMsgGetLocalTimeZoneOffset(
       ppapi::host::HostMessageContext* host_context,
       const base::Time& t);
   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);
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FLASH_BROWSER_HOST_H_