Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(144)

Unified Diff: chrome/browser/translate/translate_script.h

Issue 17390018: Refactoring: Create TranslateScript from TranslateManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@refactoring-translate-url-fetcher
Patch Set: Removed the nits Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/translate/translate_script.h
diff --git a/chrome/browser/translate/translate_script.h b/chrome/browser/translate/translate_script.h
new file mode 100644
index 0000000000000000000000000000000000000000..08a15fc1f990f1d3cd2fd0630ead02b6f8feaa75
--- /dev/null
+++ b/chrome/browser/translate/translate_script.h
@@ -0,0 +1,66 @@
+// Copyright 2013 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_SCRIPT_H_
+#define CHROME_BROWSER_TRANSLATE_TRANSLATE_SCRIPT_H_
+
+#include <string>
+
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
+#include "base/time.h"
+
+class TranslateURLFetcher;
+
+class TranslateScript {
+ public:
+ typedef base::Callback<void(bool, const std::string&)> Callback;
+
+ TranslateScript();
+ virtual ~TranslateScript();
+
+ // Returns the feched the translate script.
+ const std::string& data() { return data_; }
+
+ // Used by unit-tests to override some defaults:
+ // Delay after which the translate script is fetched again from the
+ // translation server.
+ void set_expiration_delay(int delay_ms) {
+ expiration_delay_ = base::TimeDelta::FromMilliseconds(delay_ms);
+ }
+
+ // Clears the translate script, so it will be fetched next time we translate.
+ void Clear() { data_.clear(); }
+
+ // Fetches the JS translate script (the script that is injected in the page
+ // to translate it).
+ void Request(const Callback& callback);
+
+ // Returns true if this has a pending request.
+ bool HasPendingRequest() const { return fetcher_.get() != NULL; }
+
+ private:
+ // The callback when the script is fetched or a server error occured.
+ void OnScriptFetchComplete(int id, bool success, const std::string& data);
+
+ base::WeakPtrFactory<TranslateScript> weak_method_factory_;
+
+ // URL fetcher to fetch the translate script.
+ scoped_ptr<TranslateURLFetcher> fetcher_;
+
+ // The JS injected in the page to do the translation.
+ std::string data_;
+
+ // Delay after which the translate script is fetched again from the translate
+ // server.
+ base::TimeDelta expiration_delay_;
+
+ // The callback called when the server sends a response.
+ Callback callback_;
+
+ DISALLOW_COPY_AND_ASSIGN(TranslateScript);
+};
+
+#endif // CHROME_BROWSER_TRANSLATE_TRANSLATE_SCRIPT_H_
« no previous file with comments | « chrome/browser/translate/translate_manager_browsertest.cc ('k') | chrome/browser/translate/translate_script.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698