ResourceRequestAllowedNotifier didn't work as expected

chrome/browser/web_resource/resource_request_allowed_notifier_test_util.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_test_util.h"
 
 TestRequestAllowedNotifier::TestRequestAllowedNotifier()
     : override_requests_allowed_(false),
       requests_allowed_(true) {
 }
 
 TestRequestAllowedNotifier::~TestRequestAllowedNotifier() {
 }
 
 void TestRequestAllowedNotifier::InitWithEulaAcceptNotifier(
     Observer* observer,
     scoped_ptr<EulaAcceptedNotifier> eula_notifier) {
   test_eula_notifier_.swap(eula_notifier);
   Init(observer);
 }
 
 void TestRequestAllowedNotifier::SetRequestsAllowedOverride(bool allowed) {
   override_requests_allowed_ = true;
   requests_allowed_ = 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() {
   return test_eula_notifier_.release();
 }