Don't register or use the DelegateExecute verb handler if it isn't present.

chrome/installer/setup/uninstall.cc

Index: chrome/installer/setup/uninstall.cc
diff --git a/chrome/installer/setup/uninstall.cc b/chrome/installer/setup/uninstall.cc
index ba2abb5a98341b0d5863137f0f103f0a81af6324..8dc12c6d8db77cf2d7e0c499ca19567d2423e69a 100644
--- a/chrome/installer/setup/uninstall.cc
+++ b/chrome/installer/setup/uninstall.cc
@@ -701,13 +701,12 @@ const wchar_t kChromeExtProgId[] = L"ChromiumExt";
 // work items for |product|.  This will be a noop for products whose
 // corresponding BrowserDistribution implementations do not publish
 // DelegateExecute data via an implementation of GetDelegateExecuteHandlerData.
-bool ProcessDelegateExecuteWorkItems(const InstallationState& original_state,
-                                     const InstallerState& installer_state,
+bool ProcessDelegateExecuteWorkItems(const InstallerState& installer_state,
                                      const FilePath& setup_path,

[gab, 2012/04/25 05:18:29]

|setup_path| is now unused.

[grt (UTC plus 2), 2012/04/25 13:05:11]

Good catch.  Done.

                                      const Product& product) {
   scoped_ptr<WorkItemList> item_list(WorkItem::CreateNoRollbackWorkItemList());
-  AddDelegateExecuteWorkItems(original_state, installer_state, setup_path,
-                              Version(), product, item_list.get());
+  AddDelegateExecuteWorkItems(installer_state, FilePath(), Version(), product,

[gab, 2012/04/25 05:18:29]

I don't like having to create empty FilePath and Version objects here, but I
realize there is no such thing as passing a null reference in C++. What about
making the interface use pointers instead of references? I guess that puts the
burden in AddDelegateExecuteWorkItems now having to check for null-ptrs..?

Just looking for C++ coding insights :)

[grt (UTC plus 2), 2012/04/25 13:05:11]

For the most part, pointers are used for out/inout params in Chromium.  Although
the style guide permits const Foo*, it discourages its use.  I think it
generally reduces readability to do so since it's so rare.

An alternative would be to create an AddRemoveDelegateExecuteWorkItems function,
or somesuch, that only takes the params required to remove the registration. 
I'm hesitant to do this, though, since the remove code is hit during installs
and because it separates the implementation of the two.  Also, there's precedent
in the installer for funcs to do double-duty like this.

+                              item_list.get());
   return item_list->Do();
 }
 
@@ -830,8 +829,7 @@ InstallStatus UninstallProduct(const InstallationState& original_state,
                                  suffix, installer_state.target_path(), &ret);
   }
 
-  ProcessDelegateExecuteWorkItems(original_state, installer_state, setup_path,
-                                  product);
+  ProcessDelegateExecuteWorkItems(installer_state, setup_path, product);
 
   if (!is_chrome) {
     ProcessChromeFrameWorkItems(original_state, installer_state, setup_path,