ResourceRequestAllowedNotifier didn't work as expected

chrome/browser/web_resource/resource_request_allowed_notifier_test_util.cc

Index: chrome/browser/web_resource/resource_request_allowed_notifier_test_util.cc
diff --git a/chrome/browser/web_resource/resource_request_allowed_notifier_test_util.cc b/chrome/browser/web_resource/resource_request_allowed_notifier_test_util.cc
index 073c4bcffa38c913db5ce2dcddce22a0a2e2cb66..321023fbf5da174e4060f7ba38bbb411f68a11e7 100644
--- a/chrome/browser/web_resource/resource_request_allowed_notifier_test_util.cc
+++ b/chrome/browser/web_resource/resource_request_allowed_notifier_test_util.cc
@@ -25,22 +25,22 @@ void TestRequestAllowedNotifier::SetRequestsAllowedOverride(bool allowed) {
 }
 
 void TestRequestAllowedNotifier::NotifyObserver() {
-  // Force the allowed state to true. This forces MaybeNotifyObserver to always
-  // notify observers, as MaybeNotifyObserver checks ResourceRequestsAllowed.
+  // Force the allowed state and requested state to true. This forces
+  // MaybeNotifyObserver to always notify observers, as MaybeNotifyObserver
+  // checks ResourceRequestsAllowed and requested state.
   override_requests_allowed_ = true;
   requests_allowed_ = true;
+  SetObserverRequestedForTesting(true);
   MaybeNotifyObserver();
 }
 
 bool TestRequestAllowedNotifier::ResourceRequestsAllowed() {
-  // Call ResourceRequestAllowedNotifier::ResourceRequestsAllowed once to
-  // simulate that the user requested permission. Only return that result if
-  // the override flag was set.
-  bool requests_allowed =
-      ResourceRequestAllowedNotifier::ResourceRequestsAllowed();
+  // Return the override result if the override flag was set. Otherwise, call
+  // ResourceRequestAllowedNotifier::ResourceRequestsAllowed and return the
+  // result.

[Alexei Svitkine (slow), 2013/06/18 15:35:04]

Nit: This comment no longer describes something tricky, just the code below it.
That's redundant, just remove it then.

[Takashi Toyoshima, 2013/06/19 04:35:40]

Done.

   if (override_requests_allowed_)
     return requests_allowed_;
-  return requests_allowed;
+  return ResourceRequestAllowedNotifier::ResourceRequestsAllowed();
 }
 
 EulaAcceptedNotifier* TestRequestAllowedNotifier::CreateEulaNotifier() {