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

Side by Side Diff: chrome/browser/android/contextualsearch/contextual_search_manager.cc

Issue 1205033005: Adds selection expansion support for Contextual Search. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed java tests Created 5 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/android/contextualsearch/contextual_search_manager.h" 5 #include "chrome/browser/android/contextualsearch/contextual_search_manager.h"
6 6
7 #include <set> 7 #include <set>
8 8
9 #include "base/android/jni_string.h" 9 #include "base/android/jni_string.h"
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 base_content_view_core, 102 base_content_view_core,
103 may_send_base_page_url); 103 may_send_base_page_url);
104 } 104 }
105 105
106 void ContextualSearchManager::OnSearchTermResolutionResponse( 106 void ContextualSearchManager::OnSearchTermResolutionResponse(
107 bool is_invalid, 107 bool is_invalid,
108 int response_code, 108 int response_code,
109 const std::string& search_term, 109 const std::string& search_term,
110 const std::string& display_text, 110 const std::string& display_text,
111 const std::string& alternate_term, 111 const std::string& alternate_term,
112 bool prevent_preload) { 112 bool prevent_preload,
113 int selection_start_adjust,
114 int selection_end_adjust) {
113 // Notify the Java UX of the result. 115 // Notify the Java UX of the result.
114 JNIEnv* env = base::android::AttachCurrentThread(); 116 JNIEnv* env = base::android::AttachCurrentThread();
115 base::android::ScopedJavaLocalRef<jstring> j_search_term = 117 base::android::ScopedJavaLocalRef<jstring> j_search_term =
116 base::android::ConvertUTF8ToJavaString(env, search_term.c_str()); 118 base::android::ConvertUTF8ToJavaString(env, search_term.c_str());
117 base::android::ScopedJavaLocalRef<jstring> j_display_text = 119 base::android::ScopedJavaLocalRef<jstring> j_display_text =
118 base::android::ConvertUTF8ToJavaString(env, display_text.c_str()); 120 base::android::ConvertUTF8ToJavaString(env, display_text.c_str());
119 base::android::ScopedJavaLocalRef<jstring> j_alternate_term = 121 base::android::ScopedJavaLocalRef<jstring> j_alternate_term =
120 base::android::ConvertUTF8ToJavaString(env, alternate_term.c_str()); 122 base::android::ConvertUTF8ToJavaString(env, alternate_term.c_str());
121 Java_ContextualSearchManager_onSearchTermResolutionResponse( 123 Java_ContextualSearchManager_onSearchTermResolutionResponse(
122 env, 124 env, java_manager_.obj(), is_invalid, response_code, j_search_term.obj(),
123 java_manager_.obj(), 125 j_display_text.obj(), j_alternate_term.obj(), prevent_preload,
124 is_invalid, 126 selection_start_adjust, selection_end_adjust);
125 response_code,
126 j_search_term.obj(),
127 j_display_text.obj(),
128 j_alternate_term.obj(),
129 prevent_preload);
130 } 127 }
131 128
132 void ContextualSearchManager::OnSurroundingTextAvailable( 129 void ContextualSearchManager::OnSurroundingTextAvailable(
133 const std::string& before_text, 130 const std::string& before_text,
134 const std::string& after_text) { 131 const std::string& after_text) {
135 JNIEnv* env = base::android::AttachCurrentThread(); 132 JNIEnv* env = base::android::AttachCurrentThread();
136 base::android::ScopedJavaLocalRef<jstring> j_before_text = 133 base::android::ScopedJavaLocalRef<jstring> j_before_text =
137 base::android::ConvertUTF8ToJavaString(env, before_text.c_str()); 134 base::android::ConvertUTF8ToJavaString(env, before_text.c_str());
138 base::android::ScopedJavaLocalRef<jstring> j_after_text = 135 base::android::ScopedJavaLocalRef<jstring> j_after_text =
139 base::android::ConvertUTF8ToJavaString(env, after_text.c_str()); 136 base::android::ConvertUTF8ToJavaString(env, after_text.c_str());
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 } 254 }
258 255
259 bool RegisterContextualSearchManager(JNIEnv* env) { 256 bool RegisterContextualSearchManager(JNIEnv* env) {
260 return RegisterNativesImpl(env); 257 return RegisterNativesImpl(env);
261 } 258 }
262 259
263 jlong Init(JNIEnv* env, jobject obj) { 260 jlong Init(JNIEnv* env, jobject obj) {
264 ContextualSearchManager* manager = new ContextualSearchManager(env, obj); 261 ContextualSearchManager* manager = new ContextualSearchManager(env, obj);
265 return reinterpret_cast<intptr_t>(manager); 262 return reinterpret_cast<intptr_t>(manager);
266 } 263 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698