Refactoring: Create TranslateScript from TranslateManager

chrome/browser/translate/translate_url_fetcher.h

Index: chrome/browser/translate/translate_url_fetcher.h
diff --git a/chrome/browser/translate/translate_url_fetcher.h b/chrome/browser/translate/translate_url_fetcher.h
index 6654ab87bba560ad329e8b842c8e4874abe1f2f4..0d85db8731afc71b176d42898522a9769a593797 100644
--- a/chrome/browser/translate/translate_url_fetcher.h
+++ b/chrome/browser/translate/translate_url_fetcher.h
@@ -26,6 +26,20 @@ class TranslateURLFetcher : public net::URLFetcherDelegate {
   explicit TranslateURLFetcher(int id);
   virtual ~TranslateURLFetcher();
 
+  int retry_count() {

[Takashi Toyoshima, 2013/06/20 06:59:53]

retry_count* used in function and variable names should be renamed to declare
it's for 5xx.
I introduced another retry, and need to avoid confusion.

[hajimehoshi, 2013/06/20 08:50:08]

OK, I'll do it later.

On 2013/06/20 06:59:53, Takashi Toyoshima (chromium) wrote:

+    return retry_count_;
+  }
+  void set_retry_count(int count) {
+    retry_count_ = count;
+  }
+
+  const std::string& extra_request_header() {
+    return extra_request_header_;
+  }
+  void set_extra_request_header(const std::string& header) {
+    extra_request_header_ = header;
+  }
+
   // Requests to |url|. |callback| will be invoked when the function returns
   // true, and the request is finished asynchronously.
   // Returns false if the previous request is not finished, or the request
@@ -55,6 +69,13 @@ class TranslateURLFetcher : public net::URLFetcherDelegate {
   // fetch operation is finished.
   Callback callback_;
 
+  // The count to retry sending the request when the server repllies 5xx
+  // error.
+  int retry_count_;
+
+  // An extra HTTP request header
+  std::string extra_request_header_;
+
   DISALLOW_COPY_AND_ASSIGN(TranslateURLFetcher);
 };