[App Shell] Add dev tools for debugging

apps/shell/app/shell_main_delegate.cc

Index: apps/shell/app/shell_main_delegate.cc
diff --git a/apps/shell/app/shell_main_delegate.cc b/apps/shell/app/shell_main_delegate.cc
index 964c4c543f866ef5fbb5aa8671634e38f179357f..279ce34ff7acccac6a7d97c2a88fad7571b484ea 100644
--- a/apps/shell/app/shell_main_delegate.cc
+++ b/apps/shell/app/shell_main_delegate.cc
@@ -95,8 +95,13 @@ void ShellMainDelegate::InitializeResourceBundle() {
   // TODO(jamescook): app_shell needs its own manifest data file.
   base::FilePath resources_pack_path;
   PathService::Get(chrome::FILE_RESOURCES_PACK, &resources_pack_path);
+  base::FilePath pak_file, pak_dir;
+  PathService::Get(base::DIR_MODULE, &pak_dir);
+  pak_file = pak_dir.Append(FILE_PATH_LITERAL("content_shell.pak"));

[James Cook, 2014/02/10 20:31:23]

Why do we need content_shell's PAK file?  Isn't Chrome's sufficient?

If we need both pak files please comment in the source why we need both.

nit: I would put these lines below the AddDataPackFromPath line below.

[Haojian Wu, 2014/02/11 03:24:36]

Done.

Yes, we need content_shell.pak here.
We use content's DevTools rather than chrome's, so we need
shell_devtools_discovery_page.html in content.pak instead of
devtools_discovery_page.html in chrome resource pak file.

   ResourceBundle::GetSharedInstance().AddDataPackFromPath(
       resources_pack_path, ui::SCALE_FACTOR_NONE);
+  ResourceBundle::GetSharedInstance().AddDataPackFromPath(
+      pak_file, ui::SCALE_FACTOR_NONE);
 }
 
 }  // namespace apps