Add error handling to inline signin flow

chrome/browser/ui/sync/one_click_signin_helper.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_UI_SYNC_ONE_CLICK_SIGNIN_HELPER_H_
 #define CHROME_BROWSER_UI_SYNC_ONE_CLICK_SIGNIN_HELPER_H_
 
 #include <string>
 
 #include "base/gtest_prod_util.h"
@@ skipping 102 matching lines @@
                                   ProfileIOData* io_data);
 
   // Looks for the Google-Accounts-SignIn response header, and if found,
   // tries to display an infobar in the tab contents identified by the
   // child/route id.
   static void ShowInfoBarIfPossible(net::URLRequest* request,
                                     ProfileIOData* io_data,
                                     int child_id,
                                     int route_id);
 
+  static void ShowSigninErrorBubble(Browser* browser, const std::string& error);
+
   // Remove the item currently at the top of the history list if it's
   // the Gaia redirect URL. Due to limitations of the NavigationController
   // this cannot be done until a new page becomes "current".
   static void RemoveSigninRedirectURLHistoryItem(
       content::WebContents* web_contents);
 
   static void LogConfirmHistogramValue(int action);
 
  private:
   friend class content::WebContentsUserData<OneClickSigninHelper>;
@@ skipping 68 matching lines @@
   // entered on the Gaia sign in page (for explicit sign ins).
   static void ShowInfoBarUIThread(const std::string& session_index,
                                   const std::string& email,
                                   AutoAccept auto_accept,
                                   signin::Source source,
                                   const GURL& continue_url,
                                   int child_id,
                                   int route_id);
 
   void RedirectToSignin();
-  void ShowSigninErrorBubble(Browser* browser, const std::string& error);
 
   // Clear all data member of the helper, except for the error.
   void CleanTransientState();
 
   // Unitests that use a TestingProfile should call this.
   // Otherwise, clearing the pending e-mail crashes because the code expects
   // a real ResourceContext rather than the MockResourceContext a
   // TestingProfile provides.
   void SetDoNotClearPendingEmailForTesting();
 
@@ skipping 59 matching lines @@
 
   // Allows unittest to avoid starting sync for real.
   bool do_not_start_sync_for_testing_;
 
   base::WeakPtrFactory<OneClickSigninHelper> weak_pointer_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(OneClickSigninHelper);
 };
 
 #endif  // CHROME_BROWSER_UI_SYNC_ONE_CLICK_SIGNIN_HELPER_H_