Get chrome:// dev tool urls hooked up in content_shell.

content/app/content_main_runner.cc

 // 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.
 
 #include "content/public/app/content_main_runner.h"
 
 #include "base/at_exit.h"
 #include "base/command_line.h"
 #include "base/debug/debugger.h"
 #include "base/debug/trace_event.h"
 #include "base/file_path.h"
 #include "base/i18n/icu_util.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/metrics/stats_table.h"
 #include "base/path_service.h"
 #include "base/process_util.h"
 #include "base/stringprintf.h"
 #include "base/string_number_conversions.h"
 #include "content/browser/browser_main.h"
 #include "content/common/set_process_title.h"
 #include "content/public/app/content_main_delegate.h"
 #include "content/public/app/startup_helper_win.h"
 #include "content/public/common/content_client.h"
 #include "content/public/common/content_constants.h"
 #include "content/public/common/content_paths.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/main_function_params.h"
 #include "content/public/common/sandbox_init.h"
+#include "content/public/common/url_constants.h"
 #include "crypto/nss_util.h"
 #include "ipc/ipc_switches.h"
 #include "media/base/media.h"
 #include "sandbox/src/sandbox_types.h"
 #include "ui/base/ui_base_switches.h"
 #include "ui/base/ui_base_paths.h"
 #include "webkit/glue/webkit_glue.h"
 
 #if defined(OS_WIN)
 #include <atlbase.h>
@@ skipping 419 matching lines @@
 #elif defined(OS_POSIX)
       // On linux, we're in the zygote here; so we need the parent process' id.
       browser_pid = base::GetParentProcessId(base::GetCurrentProcId());
 #endif
     }
 
     InitializeStatsTable(browser_pid, command_line);
 
     if (delegate)
       delegate->PreSandboxStartup();
+    else

[jam, 2012/04/02 16:19:25]

you're assuming that the presence of a delegate means they'll call
content::RegisterContentSchemes which might not always be true

the model we've been gradually switching to, after adding the Content API, is
that content is the one that owns and is responsible for startup. it calls the
embedder for them to optionally customize things. this allows more code sharing
with other embedders of content

so can you make content's RegisterContentSchemes call the embedder through
ContentClient::RegisterSchemes?

[michaeln, 2012/04/02 19:37:22]

Sure, is this what you intend...

* add an unconditional call to content::RegisterContentSchemes() here
* have that function call the 'clients' method
* remove the calls to content::RegisterSchemes() from the existing
PreSandboxStartup impls

[jam, 2012/04/02 20:23:13]

yep

+      content::RegisterContentSchemes(NULL);
 
     if (!process_type.empty())
       CommonSubprocessInit(process_type);
 
 #if defined(OS_WIN)
     CHECK(content::InitializeSandbox(sandbox_info));
 #elif defined(OS_MACOSX)
     if (process_type == switches::kRendererProcess ||
         process_type == switches::kPpapiPluginProcess ||
         (delegate && delegate->DelaySandboxInitialization(process_type))) {
@@ skipping 88 matching lines @@
 }  // namespace
 
 namespace content {
 
 // static
 ContentMainRunner* ContentMainRunner::Create() {
   return new ContentMainRunnerImpl();
 }
 
 }  // namespace content