Componentize TranslateUIDelegate

chrome/browser/translate/translate_tab_helper.h

 // Copyright 2011 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_TRANSLATE_TRANSLATE_TAB_HELPER_H_
 #define CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 76 matching lines @@
   virtual TranslateDriver* GetTranslateDriver() OVERRIDE;
   virtual PrefService* GetPrefs() OVERRIDE;
   virtual scoped_ptr<TranslatePrefs> GetTranslatePrefs() OVERRIDE;
   virtual TranslateAcceptLanguages* GetTranslateAcceptLanguages() OVERRIDE;
   virtual void ShowTranslateUI(translate::TranslateStep step,
                                const std::string source_language,
                                const std::string target_language,
                                TranslateErrors::Type error_type,
                                bool triggered_from_menu) OVERRIDE;
   virtual bool IsTranslatableURL(const GURL& url) OVERRIDE;
-  virtual void ShowReportLanguageDetectionErrorUI(
-      const GURL& report_url) OVERRIDE;
+  virtual void ShowReportLanguageDetectionErrorUI(const GURL& report_url)
+      OVERRIDE;

[droger, 2014/04/09 08:31:24]

Nit: break after opening bracket.

[blundell, 2014/04/09 08:49:27]

Done.

 
  private:
   explicit TranslateTabHelper(content::WebContents* web_contents);
   friend class content::WebContentsUserData<TranslateTabHelper>;
 
   // content::WebContentsObserver implementation.
   virtual void NavigationEntryCommitted(
       const content::LoadCommittedDetails& load_details) OVERRIDE;
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
   virtual void DidNavigateAnyFrame(
@@ skipping 58 matching lines @@
 
   // Necessary for binding the callback to HandleCLDDataRequest on the blocking
   // pool and for delaying translation initialization until the page has
   // finished loading on a reload.
   base::WeakPtrFactory<TranslateTabHelper> weak_pointer_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateTabHelper);
 };
 
 #endif  // CHROME_BROWSER_TRANSLATE_TRANSLATE_TAB_HELPER_H_