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

Unified Diff: chrome/browser/chromeos/net/network_portal_detector_impl_browsertest.cc

Issue 1346843003: Refactor NetworkPortalDetector and NetworkPortalNotificationController. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: stub Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/net/network_portal_detector_impl_browsertest.cc
diff --git a/chrome/browser/chromeos/net/network_portal_detector_impl_browsertest.cc b/chrome/browser/chromeos/net/network_portal_detector_impl_browsertest.cc
index 0d395b55ed9913865b53e24c878da6cf1cebc544..7a45b5e650f7660e5b0d16643f7e614df14541af 100644
--- a/chrome/browser/chromeos/net/network_portal_detector_impl_browsertest.cc
+++ b/chrome/browser/chromeos/net/network_portal_detector_impl_browsertest.cc
@@ -128,7 +128,8 @@ class NetworkPortalDetectorImplBrowserTest
base::Bind(&ErrorCallbackFunction));
network_portal_detector_ = new NetworkPortalDetectorImpl(
- g_browser_process->system_request_context());
+ g_browser_process->system_request_context(),
+ true /* create_notification_controller */);
NetworkPortalDetector::InitializeForTesting(network_portal_detector_);
network_portal_detector_->Enable(false /* start_detection */);
set_detector(network_portal_detector_->captive_portal_detector_.get());
@@ -151,12 +152,12 @@ class NetworkPortalDetectorImplBrowserTest
void SetIgnoreNoNetworkForTesting() {
network_portal_detector_->notification_controller_
- .SetIgnoreNoNetworkForTesting();
+ ->SetIgnoreNoNetworkForTesting();
}
const NetworkPortalWebDialog* GetDialog() const {
return network_portal_detector_->notification_controller_
- .GetDialogForTesting();
+ ->GetDialogForTesting();
}
private:

Powered by Google App Engine
This is Rietveld 408576698