Index: chrome/renderer/chrome_content_renderer_client.cc |
=================================================================== |
--- chrome/renderer/chrome_content_renderer_client.cc (revision 162033) |
+++ chrome/renderer/chrome_content_renderer_client.cc (working copy) |
@@ -586,6 +586,19 @@ |
return GURL(); |
} |
+// TODO(dschuff): remove this when ARM ABI is stable |
+#if defined(__arm__) && defined(OS_CHROMEOS) |
+static bool IsWhiteListedARM(const GURL& url) { |
+ return |
+ // QuickOffice |
+ url.host() == "gbkeegbaiigmenfmjfclcdgdpimamgkj" || |
+ // ssh dev |
+ url.host() == "okddffdblfhhnmhodogpojmfkjmhinfp" || |
+ // ssh stable |
+ url.host() == "pnhechapfaindjhompbnflcldabbghjo"; |
+} |
+#endif |
+ |
// static |
bool ChromeContentRendererClient::IsNaClAllowed( |
const GURL& manifest_url, |
@@ -606,15 +619,15 @@ |
// under development, invocations from whitelisted URLs, and all invocations |
// if --enable-nacl is set. |
bool is_nacl_allowed = |
-#if defined(__arm__) |
+#if defined(__arm__) && defined(OS_CHROMEOS) |
// The ARM ABI is not quite stable, so only allow NaCl for |
// unrestricted extensions (i.e. built-in and under development), |
- // and for the QuickOffice webstore app. |
+ // and for certain whitelisted webstore apps. |
// See http://crbug.com/145694 |
// TODO(dschuff): remove this when the ABI is stable |
(is_extension_from_webstore && |
manifest_url.SchemeIs("chrome-extension") && |
- manifest_url.host() == "gbkeegbaiigmenfmjfclcdgdpimamgkj") || |
+ IsWhiteListedARM(manifest_url) || |
#else |
is_extension_from_webstore || |
is_whitelisted_url || |