Give SxS distribution its own registration GUIDs.

chrome/installer/util/chrome_frame_distribution.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.
 //
 // This file defines a specific implementation of BrowserDistribution class for
 // Chrome Frame. It overrides the bare minimum of methods necessary to get a
 // Chrome Frame installer that does not interact with Google Chrome or
 // Chromium installations.
 
 #include "chrome/installer/util/chrome_frame_distribution.h"
@@ skipping 103 matching lines @@
 int ChromeFrameDistribution::GetIconIndex(ShortcutType shortcut_type) {
   switch (shortcut_type) {
     case SHORTCUT_CHROME:
       return 0;
     default:
       NOTREACHED();
       return 0;
   }
 }
 
+bool ChromeFrameDistribution::IsSetAsDefaultSupported() {
+  return false;
+}
+
 bool ChromeFrameDistribution::CanSetAsDefault() {
   return false;
 }
 
 bool ChromeFrameDistribution::CanCreateDesktopShortcuts() {
   return false;
 }
 
+bool ChromeFrameDistribution::GetCommandExecuteImplClsidString(
+    string16* handler_class_uuid) {
+  return false;

[gab, 2013/09/06 18:40:08]

I find it sad to have to duplicate this logic from GetCommandExecuteImplClsid()
everywhere... I remember we talked about this and determined this was probably
the easiest way; but what was the reason again for not being able to only store
the GUID as a constant and have this method return the string conversion of each
distribution's GetCommandExecuteImplClsid()?

[zturner, 2013/09/06 20:53:16]

StringFromCLSID() and UuidToString() both internally allocate memory for the
string, which the caller is required to free.  (CoTaskMemFree and RpcStringFree,
respectively).  We could still provide the helper and to the memory freeing
there, but it just seemed ugly.  We decided not to do the reverse (store the
string and convert to GUID) because the opposite function, UuidFromString, isn't
const correct and would require both a const_cast<> and a reinterpret_cast<> in
order to call the function.  So that was also ugly.

However, Greg pointed out CLSIDFromString(), which is const-correct, so that's
probably the best solution.

At some point, someone (I wouldn't mind doing it) should build out base::guid so
that it's more useful and does all this for us. 

+}
+
 bool ChromeFrameDistribution::GetCommandExecuteImplClsid(
-    string16* handler_class_uuid) {
+    CLSID* handler_class_uuid) {
   return false;
 }
 
 void ChromeFrameDistribution::UpdateInstallStatus(bool system_install,
     installer::ArchiveType archive_type,
     installer::InstallStatus install_status) {
 #if defined(GOOGLE_CHROME_BUILD)
   GoogleUpdateSettings::UpdateInstallStatus(system_install,
       archive_type, InstallUtil::GetInstallReturnCode(install_status),
       kChromeFrameGuid);
 #endif
 }