Removed AttemptResult enum in favor of using DefaultWebClientState

chrome/browser/shell_integration.h

 // 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.
 
 #ifndef CHROME_BROWSER_SHELL_INTEGRATION_H_
 #define CHROME_BROWSER_SHELL_INTEGRATION_H_
 
 #include <string>
 
 #include "base/callback.h"
@@ skipping 193 matching lines @@
   void StartCheckIsDefault();
 
   // Sets Chrome as the default web client application. Once done, it will
   // trigger a check for the default state using StartCheckIsDefault() to return
   // the default state to the caller.
   void StartSetAsDefault();
 
  protected:
   friend class base::RefCountedThreadSafe<DefaultWebClientWorker>;
 
-  // Possible result codes for a set-as-default operation.
-  // Do not modify the ordering as it is important for UMA.
-  enum AttemptResult {
-    // Chrome was set as the default web client.
-    SUCCESS = 0,
-    // Chrome was already the default web client. This counts as a successful
-    // attempt.
-    // Note: This result is no longer used since the removal of the Win10
-    // default browser experiment.
-    // ALREADY_DEFAULT = 1,
-    // Chrome was not set as the default web client.
-    FAILURE = 2,
-    // The attempt was abandoned because the observer was destroyed.
-    // Note: This result is no longer used since the removal of
-    // DefaultWebClientObserver.
-    // ABANDONED = 3,
-    // Failed to launch the process to set Chrome as the default web client
-    // asynchronously.
-    // Note: This result is no longer used since the removal of the Win10
-    // default browser experiment.
-    // LAUNCH_FAILURE = 4,
-    // Another worker is already in progress to make Chrome the default web
-    // client.
-    // Note: This result is no longer used since the removal of the Win10
-    // default browser experiment.
-    // OTHER_WORKER = 5,
-    // The user initiated another attempt while the asynchronous operation was
-    // already in progress.
-    RETRY = 6,
-    // No errors were encountered yet Chrome is still not the default web
-    // client.
-    NO_ERRORS_NOT_DEFAULT = 7,
-    NUM_ATTEMPT_RESULT_TYPES
-  };
-
   virtual ~DefaultWebClientWorker();
 
-  // Communicates the result via the |callback_|. In contrast to
-  // OnSetAsDefaultAttemptComplete(), this should not be called multiple
-  // times.
-  void OnCheckIsDefaultComplete(DefaultWebClientState state);
-
-  // Called when the set as default operation is completed. This then invokes
-  // FinalizeSetAsDefault() and starts the check is default process.
-  // It is safe to call this multiple times. Only the first call is processed
-  // after StartSetAsDefault() is invoked.
-  void OnSetAsDefaultAttemptComplete(AttemptResult result);
+  // Communicates the result via the |callback_|. When
+  // |is_following_set_as_default| is true, |state| will be reported to UMA as
+  // the result of the set-as-default operation.
+  void OnCheckIsDefaultComplete(DefaultWebClientState state,
+                                bool is_following_set_as_default);
 
   // When false, the operation to set as default will fail for interactive
   // flows.
   bool interactive_permitted_ = true;
 
  private:
   // Checks whether Chrome is the default web client. Always called on the
   // FILE thread. Subclasses are responsible for calling
-  // OnCheckIsDefaultComplete() on the UI thread.
-  virtual void CheckIsDefault() = 0;
+  // OnCheckIsDefaultComplete() on the UI thread. When
+  // |is_following_set_as_default| is true, the default state will be reported
+  // to UMA as the result of the set-as-default operation.
+  virtual void CheckIsDefault(bool is_following_set_as_default) = 0;
 
   // Sets Chrome as the default web client. Always called on the FILE thread.
-  // Subclasses are responsible for calling OnSetAsDefaultAttemptComplete() on
-  // the UI thread.
+  // Subclasses are responsible for calling CheckIsDefault().
   virtual void SetAsDefault() = 0;
 
   // Returns the prefix used for metrics to differentiate UMA metrics for
   // setting the default browser and setting the default protocol client.
   virtual const char* GetHistogramPrefix() = 0;
 
-  // Reports the result and duration for one set-as-default attempt.
-  void ReportAttemptResult(AttemptResult result);
+  // Reports the result for the set-as-default operation.
+  void ReportSetDefaultResult(DefaultWebClientState state);
 
   // Updates the UI in our associated view with the current default web
   // client state.
   void UpdateUI(DefaultWebClientState state);
 
   // Called with the default state after the worker is done.
   DefaultWebClientWorkerCallback callback_;
 
-  // Wait until Chrome has been confirmed as the default browser before
-  // reporting a successful attempt.
-  bool check_default_should_report_success_ = false;
-
   DISALLOW_COPY_AND_ASSIGN(DefaultWebClientWorker);
 };
 
 // Worker for checking and setting the default browser.
 class DefaultBrowserWorker : public DefaultWebClientWorker {
  public:
   explicit DefaultBrowserWorker(const DefaultWebClientWorkerCallback& callback);
 
  private:
   ~DefaultBrowserWorker() override;
 
   // Check if Chrome is the default browser.
-  void CheckIsDefault() override;
+  void CheckIsDefault(bool is_following_set_as_default) override;
 
   // Set Chrome as the default browser.
   void SetAsDefault() override;
 
   // Returns the histogram prefix for DefaultBrowserWorker.
   const char* GetHistogramPrefix() override;
 
   DISALLOW_COPY_AND_ASSIGN(DefaultBrowserWorker);
 };
 
 // Worker for checking and setting the default client application
 // for a given protocol. A different worker instance is needed for each
 // protocol you are interested in, so to check or set the default for
 // multiple protocols you should use multiple worker objects.
 class DefaultProtocolClientWorker : public DefaultWebClientWorker {
  public:
   DefaultProtocolClientWorker(const DefaultWebClientWorkerCallback& callback,
                               const std::string& protocol);
 
   const std::string& protocol() const { return protocol_; }
 
  protected:
   ~DefaultProtocolClientWorker() override;
 
  private:
   // Check is Chrome is the default handler for this protocol.
-  void CheckIsDefault() override;
+  void CheckIsDefault(bool is_following_set_as_default) override;
 
   // Set Chrome as the default handler for this protocol.
   void SetAsDefault() override;
 
   // Returns the histogram prefix for DefaultProtocolClientWorker.
   const char* GetHistogramPrefix() override;
 
   std::string protocol_;
 
   DISALLOW_COPY_AND_ASSIGN(DefaultProtocolClientWorker);
 };
 
 }  // namespace shell_integration
 
 #endif  // CHROME_BROWSER_SHELL_INTEGRATION_H_