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

Unified Diff: chrome/renderer/chrome_content_renderer_client.h

Issue 13409003: Hide ContentClient getters from embedders so that they they don't reuse content's embedder API. The… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: sync Created 7 years, 8 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
« no previous file with comments | « chrome/common/chrome_content_client.cc ('k') | chrome/renderer/chrome_content_renderer_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/chrome_content_renderer_client.h
===================================================================
--- chrome/renderer/chrome_content_renderer_client.h (revision 192622)
+++ chrome/renderer/chrome_content_renderer_client.h (working copy)
@@ -115,17 +115,13 @@
const GURL& url,
const GURL& first_party_for_cookies,
const std::string& value) OVERRIDE;
- virtual bool AllowBrowserPlugin(WebKit::WebPluginContainer* container) const
- OVERRIDE;
+ virtual bool AllowBrowserPlugin(
+ WebKit::WebPluginContainer* container) const OVERRIDE;
+ virtual void RegisterPPAPIInterfaceFactories(
+ webkit::ppapi::PpapiInterfaceFactoryManager* factory_manager) OVERRIDE;
+ virtual bool IsRequestOSFileHandleAllowedForURL(
+ const GURL& url) const OVERRIDE;
- // TODO(mpcomplete): remove after we collect histogram data.
- // http://crbug.com/100411
- bool IsAdblockInstalled();
- bool IsAdblockPlusInstalled();
- bool IsAdblockWithWebRequestInstalled();
- bool IsAdblockPlusWithWebRequestInstalled();
- bool IsOtherExtensionWithWebRequestInstalled();
-
// For testing.
void SetExtensionDispatcher(extensions::Dispatcher* extension_dispatcher);
@@ -137,17 +133,19 @@
// and start over.
void OnPurgeMemory();
- virtual void RegisterPPAPIInterfaceFactories(
- webkit::ppapi::PpapiInterfaceFactoryManager* factory_manager) OVERRIDE;
-
- WebKit::WebPlugin* CreatePlugin(
+ static WebKit::WebPlugin* CreatePlugin(
content::RenderView* render_view,
WebKit::WebFrame* frame,
const WebKit::WebPluginParams& params,
const ChromeViewHostMsg_GetPluginInfo_Output& output);
- virtual bool IsRequestOSFileHandleAllowedForURL(
- const GURL& url) const OVERRIDE;
+ // TODO(mpcomplete): remove after we collect histogram data.
+ // http://crbug.com/100411
+ static bool IsAdblockInstalled();
+ static bool IsAdblockPlusInstalled();
+ static bool IsAdblockWithWebRequestInstalled();
+ static bool IsAdblockPlusWithWebRequestInstalled();
+ static bool IsOtherExtensionWithWebRequestInstalled();
private:
FRIEND_TEST_ALL_PREFIXES(ChromeContentRendererClientTest, NaClRestriction);
« no previous file with comments | « chrome/common/chrome_content_client.cc ('k') | chrome/renderer/chrome_content_renderer_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698