Enable builtin Mojo JS modules in layout tests

content/public/browser/content_browser_client.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 CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 
 #include <map>
 #include <string>
 #include <utility>
@@ skipping 702 matching lines @@
   // to control the flow of a navigation on the UI thread. The embedder is
   // guaranteed that the throttles will be executed in the order they were
   // provided.
   virtual ScopedVector<NavigationThrottle> CreateThrottlesForNavigation(
       NavigationHandle* navigation_handle);
 
   // Creates and returns a factory used for creating CDM instances for playing
   // protected content.
   virtual scoped_ptr<media::CdmFactory> CreateCdmFactory();
 
+  // Allows the embedder to override the default bindings available to a render
+  // view. This must return a bitwise OR of BindingsPolicy values.
+  virtual int GetDefaultEnabledBindingsForView(RenderViewHost* view);

[Charlie Reis, 2016/01/15 01:26:51]

Hmm, no one outside content/ is using this.  John, does content shell count as
"outside content" in terms of adding public APIs like this?

[jam, 2016/01/15 02:25:15]

correct. content/shell can only use content/public (so that it can be a good
example of embedding content)

[jam, 2016/01/15 02:25:15]

Why isn't this on WebContentsDelegate? ContentBrowserClient is a last-resort
place to put methods, to avoid making it even larger, where a more focused
delegate interface exists.

[Ken Rockot(use gerrit already), 2016/01/24 23:03:51]

Actually that's not even necessary either. I've gone an even simpler (and IMHO
safer) route and explicitly add the bindings from content_shell's
WebContentsDelegate if and only if --run-layout-test is set.

+
   // Populates |mappings| with all files that need to be mapped before launching
   // a child process.
 #if defined(OS_ANDROID)
   virtual void GetAdditionalMappedFilesForChildProcess(
       const base::CommandLine& command_line,
       int child_process_id,
       content::FileDescriptorInfo* mappings,
       std::map<int, base::MemoryMappedFile::Region>* regions) {}
 #elif defined(OS_POSIX) && !defined(OS_MACOSX)
   virtual void GetAdditionalMappedFilesForChildProcess(
@@ skipping 22 matching lines @@
   // Allows an embedder to provide its own ExternalVideoSurfaceContainer
   // implementation.  Return nullptr to disable external surface video.
   virtual ExternalVideoSurfaceContainer*
   OverrideCreateExternalVideoSurfaceContainer(WebContents* web_contents);
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_