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

Unified Diff: ios/web_view/internal/translate/web_view_translate_service.h

Issue 2894483003: Initialize ios/web_view translate with a system-wide URLRequestContext. (Closed)
Patch Set: Respond to comments. Created 3 years, 7 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: ios/web_view/internal/translate/web_view_translate_service.h
diff --git a/ios/web_view/internal/translate/web_view_translate_service.h b/ios/web_view/internal/translate/web_view_translate_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..14d8f866e1e32bf01abd65c1912696e343348c44
--- /dev/null
+++ b/ios/web_view/internal/translate/web_view_translate_service.h
@@ -0,0 +1,67 @@
+// Copyright 2017 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 IOS_WEB_VIEW_INTERNAL_TRANSLATE_WEB_VIEW_TRANSLATE_SERVICE_H_
+#define IOS_WEB_VIEW_INTERNAL_TRANSLATE_WEB_VIEW_TRANSLATE_SERVICE_H_
+
+#include "base/macros.h"
+#include "base/memory/ptr_util.h"
+#include "components/web_resource/resource_request_allowed_notifier.h"
+
+namespace base {
+template <typename T>
+struct DefaultSingletonTraits;
+}
+
+namespace ios_web_view {
+
+// Singleton managing the resources required for Translate.
+class WebViewTranslateService {
+ public:
+ static WebViewTranslateService* GetInstance();
+
+ // Must be called before the Translate feature can be used.
+ void Initialize();
+
+ // Must be called to shut down the Translate feature.
+ void Shutdown();
+
+ private:
+ // Manages enabling translate requests only when resource requests are
+ // allowed.
+ // TODO(crbug.com/728776): Merge TranslateRequestsAllowedListener and
+ // WebViewTranslateService. They currently must be separate classes because
+ // the destructor of web_resource::ResourceRequestAllowedNotifier::Observer is
+ // not virtual.
+ class TranslateRequestsAllowedListener
+ : public web_resource::ResourceRequestAllowedNotifier::Observer {
+ public:
+ TranslateRequestsAllowedListener();
+ ~TranslateRequestsAllowedListener();
+
+ // ResourceRequestAllowedNotifier::Observer methods.
+ void OnResourceRequestsAllowed() override;
+
+ private:
+ // Notifier class to know if it's allowed to make network resource requests.
+ web_resource::ResourceRequestAllowedNotifier
+ resource_request_allowed_notifier_;
+
+ DISALLOW_COPY_AND_ASSIGN(TranslateRequestsAllowedListener);
+ };
+
+ WebViewTranslateService();
+ ~WebViewTranslateService();
+
+ friend struct base::DefaultSingletonTraits<WebViewTranslateService>;
+
+ // Listener which manages when translate requests can occur.
+ TranslateRequestsAllowedListener translate_requests_allowed_listener_;
+
+ DISALLOW_COPY_AND_ASSIGN(WebViewTranslateService);
+};
+
+} // namespace ios_web_view
+
+#endif // IOS_WEB_VIEW_INTERNAL_TRANSLATE_WEB_VIEW_TRANSLATE_SERVICE_H_
« no previous file with comments | « ios/web_view/internal/cwv_web_view_configuration.mm ('k') | ios/web_view/internal/translate/web_view_translate_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698