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

Unified Diff: chrome/renderer/chrome_content_renderer_client.h

Issue 9368046: Allow NaCl for non web store apps if their top level URL is whitelisted. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 8 years, 10 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/renderer/chrome_content_renderer_client.h
===================================================================
--- chrome/renderer/chrome_content_renderer_client.h (revision 121347)
+++ chrome/renderer/chrome_content_renderer_client.h (working copy)
@@ -135,6 +135,8 @@
const std::string& actual_mime_type);
private:
+ FRIEND_TEST_ALL_PREFIXES(ChromeContentRendererClientTest, NaClRestriction);
+
// Returns true if the frame is navigating to an URL either into or out of an
// extension app's extent.
bool CrossesExtensionExtents(WebKit::WebFrame* frame,
@@ -143,14 +145,14 @@
bool is_extension_url,
bool is_initial_navigation);
- // Returns true if the NaCl plugin can be created. If it returns true, as a
- // side effect, it may add special attributes to params.
- bool IsNaClAllowed(const webkit::WebPluginInfo& plugin,
- const GURL& url,
- const std::string& actual_mime_type,
- bool is_nacl_mime_type,
- bool enable_nacl,
- WebKit::WebPluginParams& params);
+ static GURL GetNaClContentHandlerURL(const std::string& actual_mime_type,
+ const webkit::WebPluginInfo& plugin);
+ static bool IsNaClAllowed(const GURL& manifest_url,
+ const GURL& top_url,
+ bool is_nacl_unrestricted,
+ bool is_extension_unrestricted,
+ bool is_extension_from_webstore,
+ WebKit::WebPluginParams& params);
scoped_ptr<ChromeRenderProcessObserver> chrome_observer_;
scoped_ptr<ExtensionDispatcher> extension_dispatcher_;

Powered by Google App Engine
This is Rietveld 408576698