Packaged (CRX) extensions shouldn't be able to get 'dev' interfaces in NaCl.

ppapi/native_client/tests/ppapi_browser/ppb_dev/ppapi_ppb_dev.cc

 // Copyright (c) 2011 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.
 //
 // Test cases for PPAPI Dev interfaces.
 //
 
 #include <string.h>
 
 #include "native_client/src/include/nacl_macros.h"
@@ skipping 32 matching lines @@
 #include "ppapi/c/dev/ppb_zoom_dev.h"
 
 
 namespace {
 
 ////////////////////////////////////////////////////////////////////////////////
 // Test Cases
 ////////////////////////////////////////////////////////////////////////////////
 
 void TestGetDevInterfaces() {
-  // This test is run only w/ NACL_ENABLE_PPAPI_DEV=0, which should
-  // turn off and disable the PPAPI developer interfaces. When they are
-  // disabled, the interface should return NULL.
-  CHECK(GetBrowserInterface(PPB_BUFFER_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_CHAR_SET_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_CONSOLE_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_CONTEXT_3D_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_CRYPTO_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_CURSOR_CONTROL_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_DIRECTORYREADER_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_FILECHOOSER_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_FIND_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_FONT_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_FULLSCREEN_DEV_INTERFACE) == NULL);
+  // This test is run as a simple embedded .nexe with --enable-nacl. It should
+  // have access to all dev interfaces.

[sehr, 2011/10/14 21:38:32]

Shouldn't we have a corresponding test that these are not available from
appstore nexes?

[bbudge, 2011/10/14 22:03:27]

Yes, but it would be very difficult to set this up in this test. Ideas?
On 2011/10/14 21:38:32, sehr1 wrote:

+  CHECK(GetBrowserInterface(PPB_BUFFER_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_CHAR_SET_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_CONSOLE_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_CONTEXT_3D_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_CRYPTO_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_CURSOR_CONTROL_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_DIRECTORYREADER_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_FILECHOOSER_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_FIND_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_FONT_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_FULLSCREEN_DEV_INTERFACE) != NULL);
   CHECK(GetBrowserInterface(
-      PPB_GLES_CHROMIUM_TEXTURE_MAPPING_DEV_INTERFACE) == NULL);
+      PPB_GLES_CHROMIUM_TEXTURE_MAPPING_DEV_INTERFACE) != NULL);
   CHECK(GetBrowserInterface(PPB_LAYER_COMPOSITOR_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_MEMORY_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_MOUSELOCK_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_SCROLLBAR_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_SURFACE_3D_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_TESTING_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_TRANSPORT_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_URLUTIL_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_VAR_DEPRECATED_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_VIDEODECODER_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_WIDGET_DEV_INTERFACE) == NULL);
-  CHECK(GetBrowserInterface(PPB_ZOOM_DEV_INTERFACE) == NULL);
+  CHECK(GetBrowserInterface(PPB_MEMORY_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_MOUSELOCK_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_SCROLLBAR_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_SURFACE_3D_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_TESTING_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_TRANSPORT_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_URLUTIL_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_VAR_DEPRECATED_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_VIDEODECODER_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_WIDGET_DEV_INTERFACE) != NULL);
+  CHECK(GetBrowserInterface(PPB_ZOOM_DEV_INTERFACE) != NULL);
 
   TEST_PASSED;
 }
 
 }  // namespace
 
 void SetupTests() {
   RegisterTest("TestGetDevInterfaces", TestGetDevInterfaces);
 }
 
 void SetupPluginInterfaces() {
   // none
 }