Chromium Code Reviews| Index: chrome/browser/android/net/external_estimate_provider_android.cc |
| diff --git a/chrome/browser/android/net/external_estimate_provider_android.cc b/chrome/browser/android/net/external_estimate_provider_android.cc |
| index 2d8c2dbe80ba822a734d8a79d0fbb80eabef2a0d..b650704b6d106e92a505578611831caf2dbf6ec4 100644 |
| --- a/chrome/browser/android/net/external_estimate_provider_android.cc |
| +++ b/chrome/browser/android/net/external_estimate_provider_android.cc |
| @@ -33,13 +33,6 @@ ExternalEstimateProviderAndroid::~ExternalEstimateProviderAndroid() { |
| net::NetworkChangeNotifier::RemoveConnectionTypeObserver(this); |
| } |
| -void ExternalEstimateProviderAndroid::RequestUpdate() const { |
| - DCHECK(thread_checker_.CalledOnValidThread()); |
| - JNIEnv* env = base::android::AttachCurrentThread(); |
| - Java_ExternalEstimateProviderAndroid_requestUpdate( |
| - env, j_external_estimate_provider_.obj()); |
| -} |
| - |
| bool ExternalEstimateProviderAndroid::GetRTT(base::TimeDelta* rtt) const { |
| DCHECK(thread_checker_.CalledOnValidThread()); |
| JNIEnv* env = base::android::AttachCurrentThread(); |
| @@ -96,6 +89,23 @@ bool ExternalEstimateProviderAndroid::GetTimeSinceLastUpdate( |
| return true; |
| } |
| +void ExternalEstimateProviderAndroid::SetUpdatedEstimateDelegate( |
| + net::ExternalEstimateProvider::UpdatedEstimateDelegate* delegate) { |
| + delegate_ = delegate; |
| +} |
| + |
| +void ExternalEstimateProviderAndroid::RequestUpdate() const { |
|
mmenke
2015/09/15 20:09:09
Update()
tbansal1
2015/09/15 22:33:07
Done.
|
| + DCHECK(thread_checker_.CalledOnValidThread()); |
| + JNIEnv* env = base::android::AttachCurrentThread(); |
| + Java_ExternalEstimateProviderAndroid_requestUpdate( |
| + env, j_external_estimate_provider_.obj()); |
| +} |
| + |
| +void ExternalEstimateProviderAndroid::OnConnectionTypeChanged( |
| + net::NetworkChangeNotifier::ConnectionType type) { |
| + RequestUpdate(); |
| +} |
| + |
| void ExternalEstimateProviderAndroid:: |
| NotifyExternalEstimateProviderAndroidUpdate(JNIEnv* env, jobject obj) { |
| if (!task_runner_) |
| @@ -113,19 +123,9 @@ void ExternalEstimateProviderAndroid::NotifyUpdatedEstimateAvailable() const { |
| delegate_->OnUpdatedEstimateAvailable(); |
| } |
| -void ExternalEstimateProviderAndroid::OnConnectionTypeChanged( |
| - net::NetworkChangeNotifier::ConnectionType type) { |
| - RequestUpdate(); |
| -} |
| - |
| bool RegisterExternalEstimateProviderAndroid(JNIEnv* env) { |
| return RegisterNativesImpl(env); |
| } |
| -void ExternalEstimateProviderAndroid::SetUpdatedEstimateDelegate( |
| - net::ExternalEstimateProvider::UpdatedEstimateDelegate* delegate) { |
| - delegate_ = delegate; |
| -} |
| - |
| } // namespace android |
| } // namespace chrome |