[OfflinePages, NetworkQualityEstimator] Use NetworkQualityEstimator to decide on triggering Backgro…

chrome/browser/net/nqe/ui_network_quality_estimator_service.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/net/nqe/ui_network_quality_estimator_service.h"
 
 #include "base/bind.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/io_thread.h"
 #include "content/public/browser/browser_thread.h"
 #include "net/nqe/network_quality_estimator.h"

[tbansal1, 2016/09/16 22:05:55]

nit: Remove this include since it is already in the .h file.

[dougarnett, 2016/09/19 19:34:32]

Done.

 
 // A class that sets itself as an observer of the EffectiveconnectionType for
 // the browser IO thread. It reports any change in EffectiveConnectionType back
 // to the UI service.
 // It is created on the UI thread, but used and deleted on the IO thread.
 class UINetworkQualityEstimatorService::IONetworkQualityObserver
     : public net::NetworkQualityEstimator::EffectiveConnectionTypeObserver {
  public:
   IONetworkQualityObserver(
       base::WeakPtr<UINetworkQualityEstimatorService> service)
@@ skipping 75 matching lines @@
   type_ = type;
 }
 
 void UINetworkQualityEstimatorService::SetEffectiveConnectionTypeForTesting(
     net::EffectiveConnectionType type) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   type_ = type;
 }
 
 net::EffectiveConnectionType
-UINetworkQualityEstimatorService::GetEffectiveConnectionType() const {
+UINetworkQualityEstimatorService::GetEffectiveConnectionType() {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   return type_;
 }