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 |