Chromium Code Reviews| Index: chrome/browser/android/contextualsearch/contextual_search_manager.h |
| diff --git a/chrome/browser/android/contextualsearch/contextual_search_manager.h b/chrome/browser/android/contextualsearch/contextual_search_manager.h |
| index 3df2d5122f3f65de8ba268d40ac3881816e60f1a..868aea7c591d1445d9c52254c93e725949a59f1b 100644 |
| --- a/chrome/browser/android/contextualsearch/contextual_search_manager.h |
| +++ b/chrome/browser/android/contextualsearch/contextual_search_manager.h |
| @@ -12,15 +12,17 @@ |
| #include "base/task/cancelable_task_tracker.h" |
| #include "chrome/browser/android/contextualsearch/contextual_search_context.h" |
| #include "chrome/browser/android/contextualsearch/contextual_search_delegate.h" |
| +#include "components/contextual_search/browser/contextual_search_js_api_handler.h" |
| // Manages the native extraction and request logic for Contextual Search, |
| // and interacts with the Java ContextualSearchManager for UX. |
| // Most of the work is done by the associated ContextualSearchDelegate. |
| -class ContextualSearchManager { |
| +class ContextualSearchManager |
| + : public contextual_search::ContextualSearchJsApiHandler { |
| public: |
| // Constructs a native manager associated with the Java manager. |
| ContextualSearchManager(JNIEnv* env, jobject obj); |
| - virtual ~ContextualSearchManager(); |
| + ~ContextualSearchManager() override; |
| // Called by the Java ContextualSearchManager when it is being destroyed. |
| void Destroy(JNIEnv* env, const base::android::JavaParamRef<jobject>& obj); |
| @@ -60,6 +62,14 @@ class ContextualSearchManager { |
| JNIEnv* env, |
| const base::android::JavaParamRef<jobject>& obj); |
| + // Enables the Contextual Search JS API for the given |ContentViewCore|. |
| + void AddViewForContextualSearchJsApi(JNIEnv* env, |
|
mdjones
2016/01/09 00:59:38
I would consider renaming to something like "enabl
Donn Denman
2016/01/11 20:31:34
Done.
|
| + jobject obj, |
| + jobject j_overlay_content_view_core); |
| + |
| + // ContextualSearchJsApiHandler overrides: |
| + void SetCaption(std::string caption, bool does_answer) override; |
| + |
| private: |
| void OnSearchTermResolutionResponse( |
| const ResolvedSearchTerm& resolved_search_term); |