[cros] Captive portal dialog fixes.

chrome/browser/chromeos/login/captive_portal_view.cc

Index: chrome/browser/chromeos/login/captive_portal_view.cc
diff --git a/chrome/browser/chromeos/login/captive_portal_view.cc b/chrome/browser/chromeos/login/captive_portal_view.cc
index 8eebf16c3045723375613d2a57f330b4b4ea3299..b1700e427ddf1e400f956892cd0bb37de3214051 100644
--- a/chrome/browser/chromeos/login/captive_portal_view.cc
+++ b/chrome/browser/chromeos/login/captive_portal_view.cc
@@ -290,8 +290,11 @@ void CaptivePortalView::LoadingStateChanged(WebContents* source) {
   bool is_loading = source->IsLoading();
   UpdateReload(is_loading, false);
   command_updater_->UpdateCommandEnabled(IDC_STOP, is_loading);
-  if (!is_loading && !redirected_)
-    proxy_->OnOriginalURLLoaded();
+  // TODO(nkostylev): Fix case of no connectivity, check HTTP code returned.
+  // Disable this heuristic as it has false positives.
+  // Relying on just flimflam portal check to close dialog is fine.
+  /*if (!is_loading && !redirected_)

[Dmitry Polukhin, 2012/03/27 13:40:41]

Please use C++ comments.

[Nikita (slow), 2012/03/27 14:01:36]

Done.

+    proxy_->OnOriginalURLLoaded();*/
 }
 
 TabContentsWrapper* CaptivePortalView::GetTabContentsWrapper() const {