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

Side by Side Diff: chrome/browser/android/physical_web/physical_web_data_source_android.cc

Issue 2565023002: Use GURLs in Physical Web data source (Closed)
Patch Set: Register missing dependencies Created 4 years 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/physical_web/physical_web_data_source_android.h " 5 #include "chrome/browser/android/physical_web/physical_web_data_source_android.h "
6 6
7 #include <jni.h> 7 #include <jni.h>
8 8
9 #include "base/android/jni_android.h" 9 #include "base/android/jni_android.h"
10 #include "base/android/jni_string.h" 10 #include "base/android/jni_string.h"
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 97
98 bool PhysicalWebDataSourceAndroid::HasUnresolvedDiscoveries() { 98 bool PhysicalWebDataSourceAndroid::HasUnresolvedDiscoveries() {
99 NOTIMPLEMENTED(); 99 NOTIMPLEMENTED();
100 return false; 100 return false;
101 } 101 }
102 102
103 void PhysicalWebDataSourceAndroid::OnFound( 103 void PhysicalWebDataSourceAndroid::OnFound(
104 JNIEnv* env, 104 JNIEnv* env,
105 const base::android::JavaParamRef<jobject>& obj, 105 const base::android::JavaParamRef<jobject>& obj,
106 const base::android::JavaParamRef<jstring>& j_url) { 106 const base::android::JavaParamRef<jstring>& j_url) {
107 std::string url = ConvertJavaStringToUTF8(env, j_url); 107 NotifyOnFound(GURL(ConvertJavaStringToUTF8(env, j_url)));
108 NotifyOnFound(url);
109 } 108 }
110 109
111 void PhysicalWebDataSourceAndroid::OnLost( 110 void PhysicalWebDataSourceAndroid::OnLost(
112 JNIEnv* env, 111 JNIEnv* env,
113 const base::android::JavaParamRef<jobject>& obj, 112 const base::android::JavaParamRef<jobject>& obj,
114 const base::android::JavaParamRef<jstring>& j_url) { 113 const base::android::JavaParamRef<jstring>& j_url) {
115 std::string url = ConvertJavaStringToUTF8(env, j_url); 114 NotifyOnLost(GURL(ConvertJavaStringToUTF8(env, j_url)));
116 NotifyOnLost(url);
117 } 115 }
118 116
119 void PhysicalWebDataSourceAndroid::OnDistanceChanged( 117 void PhysicalWebDataSourceAndroid::OnDistanceChanged(
120 JNIEnv* env, 118 JNIEnv* env,
121 const base::android::JavaParamRef<jobject>& obj, 119 const base::android::JavaParamRef<jobject>& obj,
122 const base::android::JavaParamRef<jstring>& j_url, 120 const base::android::JavaParamRef<jstring>& j_url,
123 jdouble distance_estimate) { 121 jdouble distance_estimate) {
124 std::string url = ConvertJavaStringToUTF8(env, j_url); 122 NotifyOnDistanceChanged(GURL(ConvertJavaStringToUTF8(env, j_url)),
125 NotifyOnDistanceChanged(url, distance_estimate); 123 distance_estimate);
126 } 124 }
127 125
128 // static 126 // static
129 bool PhysicalWebDataSourceAndroid::RegisterPhysicalWebDataSource(JNIEnv* env) { 127 bool PhysicalWebDataSourceAndroid::RegisterPhysicalWebDataSource(JNIEnv* env) {
130 return RegisterNativesImpl(env); 128 return RegisterNativesImpl(env);
131 } 129 }
132 130
133 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) { 131 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) {
134 physical_web::PhysicalWebDataSource* data_source = 132 physical_web::PhysicalWebDataSource* data_source =
135 g_browser_process->GetPhysicalWebDataSource(); 133 g_browser_process->GetPhysicalWebDataSource();
136 return reinterpret_cast<intptr_t>(data_source); 134 return reinterpret_cast<intptr_t>(data_source);
137 } 135 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698