ResourceRequestAllowedNotifier didn't work as expected

chrome/browser/web_resource/resource_request_allowed_notifier.cc

 // Copyright 2013 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/web_resource/resource_request_allowed_notifier.h"
 
 #include "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
 
 ResourceRequestAllowedNotifier::ResourceRequestAllowedNotifier()
@@ skipping 27 matching lines @@
 
 bool ResourceRequestAllowedNotifier::ResourceRequestsAllowed() {
   if (CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kDisableBackgroundNetworking)) {
     return false;
   }
 
   // The observer requested permission. Return the current criteria state and
   // set a flag to remind this class to notify the observer once the criteria
   // is met.
-  observer_requested_permission_ = true;
-  return !waiting_for_user_to_accept_eula_ && !waiting_for_network_;
+  observer_requested_permission_ = waiting_for_user_to_accept_eula_ ||
+                                   waiting_for_network_;
+  return !observer_requested_permission_;
 }
 
 void ResourceRequestAllowedNotifier::SetWaitingForNetworkForTesting(
     bool waiting) {
   waiting_for_network_ = waiting;
 }
 
 void ResourceRequestAllowedNotifier::SetWaitingForEulaForTesting(
     bool waiting) {
   waiting_for_user_to_accept_eula_ = waiting;
 }
 
+void ResourceRequestAllowedNotifier::SetObserverRequestedForTesting(
+    bool requested) {
+  observer_requested_permission_ = requested;
+}
+
 void ResourceRequestAllowedNotifier::MaybeNotifyObserver() {
   // Need to ensure that all criteria are met before notifying observers.
   if (observer_requested_permission_ && ResourceRequestsAllowed()) {
     DVLOG(1) << "Notifying observer of state change.";
     observer_->OnResourceRequestsAllowed();
     // Reset this so the observer is not informed again unless they check
     // ResourceRequestsAllowed again.
     observer_requested_permission_ = false;
   }
 }
@@ skipping 17 matching lines @@
   // network to reconnect, and new network state is actually available. This
   // prevents the notifier from notifying the observer if the notifier was never
   // waiting on the network, or if the network changes from one online state
   // to another (for example, Wifi to 3G, or Wifi to Wifi, if the network were
   // flaky).
   if (waiting_for_network_ &&
       type != net::NetworkChangeNotifier::CONNECTION_NONE) {
     waiting_for_network_ = false;
     DVLOG(1) << "Network came back online.";
     MaybeNotifyObserver();
+  } else if (!waiting_for_network_ &&
+             type == net::NetworkChangeNotifier::CONNECTION_NONE) {
+    waiting_for_network_ = true;
+    DVLOG(1) << "Network went offline.";
   }
 }