Reland: Adjust ChromeToMobile error logging and mitigation.

chrome/browser/chrome_to_mobile_service.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_CHROME_TO_MOBILE_SERVICE_H_
 #define CHROME_BROWSER_CHROME_TO_MOBILE_SERVICE_H_
 
 #include <map>
 #include <queue>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
 #include "base/memory/weak_ptr.h"
+#include "base/memory/scoped_vector.h"

[Raghu Simha, 2012/09/07 17:15:55]

nit: This goes above weak_ptr.h.

 #include "base/string16.h"
 #include "base/timer.h"
 #include "base/values.h"
 #include "chrome/browser/profiles/profile_keyed_service.h"
 #include "chrome/browser/sessions/session_id.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "google_apis/gaia/oauth2_access_token_consumer.h"
 #include "googleurl/src/gurl.h"
 #include "net/url_request/url_fetcher_delegate.h"
@@ skipping 33 matching lines @@
     DEVICES_REQUESTED = 0,  // Cloud print was contacted to list devices.
     DEVICES_AVAILABLE,      // Cloud print returned 1+ compatible devices.
     BUBBLE_SHOWN,           // The page action bubble was shown.
     SNAPSHOT_GENERATED,     // A snapshot was successfully generated.
     SNAPSHOT_ERROR,         // An error occurred during snapshot generation.
     SENDING_URL,            // Send was invoked (with or without a snapshot).
     SENDING_SNAPSHOT,       // A snapshot was sent along with the page URL.
     SEND_SUCCESS,           // Cloud print responded with success on send.
     SEND_ERROR,             // Cloud print responded with failure on send.
     LEARN_MORE_CLICKED,     // The "Learn more" help article link was clicked.
+    BAD_TOKEN,              // The cloud print access token could not be minted.
+    BAD_SEARCH_AUTH,        // The cloud print search request failed (auth).
+    BAD_SEARCH_OTHER,       // The cloud print search request failed (other).
+    BAD_SEND_407,           // The cloud print send response was errorCode==407.
+                            // "Print job added but failed to notify printer..."
+    BAD_SEND_ERROR,         // The cloud print send response was errorCode!=407.
+    BAD_SEND_AUTH,          // The cloud print send request failed (auth).
+    BAD_SEND_OTHER,         // The cloud print send request failed (other).
     NUM_METRICS
   };
 
   // The supported mobile device operating systems.
   enum MobileOS {
     ANDROID = 0,
     IOS,
   };
 
   // The cloud print job types.
@@ skipping 136 matching lines @@
   std::string access_token_;
 
   // The set of snapshots currently available.
   std::set<FilePath> snapshots_;
 
   // Map URLFetchers to observers for reporting OnSendComplete.
   typedef std::map<const net::URLFetcher*, base::WeakPtr<Observer> >
       RequestObserverMap;
   RequestObserverMap request_observer_map_;
 
-  // The pending OAuth access token request and a timer for retrying on failure.
+  // The list of active URLFetcher requests owned by the service.
+  ScopedVector<net::URLFetcher> url_fetchers_;
+
+  // The pending OAuth access token request and timers for retrying on failure.
   scoped_ptr<OAuth2AccessTokenFetcher> access_token_fetcher_;
   base::OneShotTimer<ChromeToMobileService> auth_retry_timer_;
+  base::OneShotTimer<ChromeToMobileService> search_retry_timer_;
 
   // A queue of tasks to perform after an access token is lazily initialized.
   std::queue<base::Closure> task_queue_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeToMobileService);
 };
 
 #endif  // CHROME_BROWSER_CHROME_TO_MOBILE_SERVICE_H_