"Redirecting URLs to Packaged Apps" implementation: revised

chrome/common/extensions/chrome_manifest_handlers.cc

Index: chrome/common/extensions/chrome_manifest_handlers.cc
diff --git a/chrome/common/extensions/chrome_manifest_handlers.cc b/chrome/common/extensions/chrome_manifest_handlers.cc
index 1881868a265f49b5f08cb4c4fa9d06eaf4d29102..61f1b3be3585b801df551c11a78b0f70d4260e3a 100644
--- a/chrome/common/extensions/chrome_manifest_handlers.cc
+++ b/chrome/common/extensions/chrome_manifest_handlers.cc
@@ -23,6 +23,7 @@
 #include "chrome/common/extensions/api/speech/tts_engine_manifest_handler.h"
 #include "chrome/common/extensions/api/spellcheck/spellcheck_handler.h"
 #include "chrome/common/extensions/api/system_indicator/system_indicator_handler.h"
+#include "chrome/common/extensions/api/url_handlers/url_handlers_parser.h"
 #include "chrome/common/extensions/background_info.h"
 #include "chrome/common/extensions/csp_handler.h"
 #include "chrome/common/extensions/incognito_handler.h"
@@ -63,6 +64,7 @@ void RegisterChromeManifestHandlers() {
   (new ExternallyConnectableHandler)->Register();
   (new FileBrowserHandlerParser)->Register();
   (new FileHandlersParser)->Register();
+  (new UrlHandlersParser)->Register();

[benwells, 2013/09/03 01:10:56]

Nit: keep this list aphabetically sorted.
Also nitty: is Url the typical case? Below there is URLOverridesHandler.

[sergeygs, 2013/09/03 06:15:24]

Ah, an automated renaming artifact. Done. Regarding Url vs URL: I've talked to
the team, and the prevalent and now recommended usage is Url. This class comes
from a different module, which already has Url's (ShellWindow::SetAppIconUrl),
but no URL's (except for GURL).

   (new HomepageURLHandler)->Register();
   (new IconsHandler)->Register();
   (new IncognitoHandler)->Register();