Split a CHECK in 2 in MiscellaneousBindings::DispatchOnConnect to help

chrome/renderer/extensions/dispatcher.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 CHROME_RENDERER_EXTENSIONS_DISPATCHER_H_
 #define CHROME_RENDERER_EXTENSIONS_DISPATCHER_H_
 
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 156 matching lines @@
       bool adblock,
       bool adblock_plus,
       bool other_webrequest);
   void OnShouldUnload(const std::string& extension_id, int sequence_id);
   void OnUnload(const std::string& extension_id);
   void OnCancelUnload(const std::string& extension_id);
 
   // Update the list of active extensions that will be reported when we crash.
   void UpdateActiveExtensions();
 
-  // Calls RenderThread's RegisterExtension and keeps tracks of which v8
-  // extension is for Chrome Extensions only.
-  void RegisterExtension(v8::Extension* extension, bool restrict_to_extensions);
-
   // Sets up the host permissions for |extension|.
   void InitOriginPermissions(const Extension* extension);
   void AddOrRemoveOriginPermissions(
       UpdatedExtensionPermissionsInfo::Reason reason,
       const Extension* extension,
       const URLPatternSet& origins);
 
   void RegisterNativeHandlers(ModuleSystem* module_system,
                               ChromeV8Context* context);
 
@@ skipping 28 matching lines @@
   // All the bindings contexts that are currently loaded for this renderer.
   // There is zero or one for each v8 context.
   ChromeV8ContextSet v8_context_set_;
 
   scoped_ptr<UserScriptSlave> user_script_slave_;
 
   // Same as above, but on a longer timer and will run even if the process is
   // not idle, to ensure that IdleHandle gets called eventually.
   base::RepeatingTimer<content::RenderThread> forced_idle_timer_;
 
   // The v8 extensions which are restricted to extension-related contexts.

[Mihai Parparita -not on Chrome, 2012/08/22 23:36:32]

I think you can remove this too.

[Matt Perry, 2012/08/23 00:22:25]

Done.

   std::set<std::string> restricted_v8_extensions_;
 
   // All declared function names.
   std::set<std::string> function_names_;
 
   // The extensions and apps that are active in this process.
   std::set<std::string> active_extension_ids_;
 
   // True once WebKit has been initialized (and it is therefore safe to poke).
   bool is_webkit_initialized_;
@@ skipping 15 matching lines @@
 
   // Sends API requests to the extension host.
   scoped_ptr<RequestSender> request_sender_;
 
   DISALLOW_COPY_AND_ASSIGN(Dispatcher);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_RENDERER_EXTENSIONS_DISPATCHER_H_