Setting chrome as the default browser is now fixed on Windows 10

chrome/browser/ui/startup/startup_browser_creator.h

Index: chrome/browser/ui/startup/startup_browser_creator.h
diff --git a/chrome/browser/ui/startup/startup_browser_creator.h b/chrome/browser/ui/startup/startup_browser_creator.h
index 3ace91f511bafa1979f41660970ba677019aef30..15f81a0d971f37cf98ef608d1311567b13b66a2b 100644
--- a/chrome/browser/ui/startup/startup_browser_creator.h
+++ b/chrome/browser/ui/startup/startup_browser_creator.h
@@ -5,6 +5,7 @@
 #ifndef CHROME_BROWSER_UI_STARTUP_STARTUP_BROWSER_CREATOR_H_
 #define CHROME_BROWSER_UI_STARTUP_STARTUP_BROWSER_CREATOR_H_
 
+#include <map>

[grt (UTC plus 2), 2015/09/24 18:24:12]

unused

[Patrick Monette, 2015/09/25 20:06:25]

Removed.

 #include <string>
 #include <vector>
 
@@ -103,6 +104,40 @@ class StartupBrowserCreator {
 
   static void RegisterLocalStatePrefs(PrefRegistrySimple* registry);
 
+#if defined(OS_WIN)
+  // Setting Chrome as the default browser in Windows 10+ requires a specific
+  // url to be opened through openwith.exe. We intercept this url in
+  // ProcessCmdLineImpl when the interceptor is set. See DefaultBrowserWorker in
+  // shell_integration.h for more details.
+  static void SetAsyncSetAsDefaultFilter(base::Closure callback);

[Peter Kasting, 2015/09/23 23:01:39]

I find the consistent use of the phrase "AsyncSetAsDefault" everywhere
confusing, since I can barely parse it, it leads to multiple verbs in a row
("set async set"), and it doesn't say "default what".  Something like this:

SetDefaultBrowserCallback();
GetDefaultBrowserURL();

...might be clearer.

[grt (UTC plus 2), 2015/09/24 18:24:12]

nit: pass callbacks by constref

[Patrick Monette, 2015/09/25 20:06:25]

Done.

[Patrick Monette, 2015/09/25 20:06:25]

Done.

+
+  // Clears the filter when it isn't needed anymore.
+  static void ClearAsyncSetAsDefaultFilter();
+
+  // Returns the url used to set Chrome as the default browser asynchronously.
+  static const wchar_t* GetAsyncSetAsDefaultUrl();
+#endif  // defined(OS_WIN)
+
+  // All the filters functions should be called on the UI thread.

[Peter Kasting, 2015/09/23 23:01:39]

Looks like all this should have been removed?

[Patrick Monette, 2015/09/25 20:06:25]

Yes. Removed.

+  /*
+  // Adds a new filter that will execute the |callback| when the |url| is opened
+  // through the command line.
+  static void AddUrlFilter(const GURL& url,
+                           const base::Callback<void(void)>& callback);
+
+  // Removes the filter associated the the |url|.
+  static void RemoveUrlFilter(const GURL& url);
+
+  // Returns true if at least one url filter exists.
+  static bool HasUrlFilters();
+
+  // Executes the callback associated to the |url| filter. Returns false if the
+  // |url| isn't associated to any filters.
+  static bool ExecuteFilterCallback(const GURL& url);
+
+  // Returns true if a filter is associated to the |url|.
+  static bool UrlFilterExists(const GURL& url);*/
+
  private:
   friend class CloudPrintProxyPolicyTest;
   friend class CloudPrintProxyPolicyStartupTest;