Add UMA metrics to phishguard pings

components/safe_browsing/password_protection/password_protection_request.h

 // Copyright 2017 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 COMPONENTS_SAFE_BROWSING_PASSWORD_PROTECTION_PASSWORD_PROTECTION_REQUEST_H_
 #define COMPONENTS_SAFE_BROWSING_PASSWORD_PROTECTION_PASSWORD_PROTECTION_REQUEST_H_
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/task/cancelable_task_tracker.h"
@@ skipping 22 matching lines @@
 // (5) |   UI   | If verdict cached, quit request; else prepare request proto.
 // (6) |   UI   | Start a timeout task, and send network request.
 // (7) |   UI   | On receiving response, handle response and finish.
 //     |        | On request timeout, cancel request.
 //     |        | On deletion of |password_protection_service_|, cancel request.
 class PasswordProtectionRequest : public base::RefCountedThreadSafe<
                                       PasswordProtectionRequest,
                                       content::BrowserThread::DeleteOnUIThread>,
                                   public net::URLFetcherDelegate {
  public:
-  // The outcome of the request. These values are used for UMA.
-  // DO NOT CHANGE THE ORDERING OF THESE VALUES.
-  enum RequestOutcome {
-    UNKNOWN = 0,
-    SUCCEEDED = 1,
-    CANCELED = 2,
-    TIMEDOUT = 3,
-    MATCHED_WHITELIST = 4,
-    RESPONSE_ALREADY_CACHED = 5,
-    DEPRECATED_NO_EXTENDED_REPORTING = 6,
-    DEPRECATED_INCOGNITO = 7,
-    REQUEST_MALFORMED = 8,
-    FETCH_FAILED = 9,
-    RESPONSE_MALFORMED = 10,
-    SERVICE_DESTROYED = 11,
-    MAX_OUTCOME
-  };
-
   PasswordProtectionRequest(const GURL& main_frame_url,
                             const GURL& password_form_action,
                             const GURL& password_form_frame_url,
                             LoginReputationClientRequest::TriggerType type,
                             PasswordProtectionService* pps,
                             int request_timeout_in_ms);
 
   base::WeakPtr<PasswordProtectionRequest> GetWeakPtr() {
     return weakptr_factory_.GetWeakPtr();
   }
@@ skipping 37 matching lines @@
   // Fill |request_proto_| with appropriate values.
   void FillRequestProto();
 
   // Initiates network request to Safe Browsing backend.
   void SendRequest();
 
   // Start a timer to cancel the request if it takes too long.
   void StartTimeout();
 
   // |this| will be destroyed after calling this function.
-  void Finish(RequestOutcome outcome,
+  void Finish(PasswordProtectionService::RequestOutcome outcome,
               std::unique_ptr<LoginReputationClientResponse> response);
 
   // Main frame URL of the login form.
   const GURL main_frame_url_;
 
   // The action URL of the password form.
   const GURL password_form_action_;
 
   // Frame url of the detected password form.
   const GURL password_form_frame_url_;
@@ skipping 24 matching lines @@
   // Needed for canceling tasks posted to different threads.
   base::CancelableTaskTracker tracker_;
 
   base::WeakPtrFactory<PasswordProtectionRequest> weakptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(PasswordProtectionRequest);
 };
 
 }  // namespace safe_browsing
 
 #endif  // COMPONENTS_SAFE_BROWSING_PASSWORD_PROTECTION_PASSWORD_PROTECTION_REQUEST_H_