Change HttpAuthHandlerMock to accept and return more than one

net/http/http_auth_handler_mock.cc

 // Copyright (c) 2010 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 "net/http/http_auth_handler_mock.h"
 
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "net/base/net_errors.h"
 #include "net/http/http_request_info.h"
@@ skipping 124 matching lines @@
 }
 
 HttpAuthHandlerMock::Factory::Factory()
     : do_init_from_challenge_(false) {
   // TODO(cbentzel): Default do_init_from_challenge_ to true.
 }
 
 HttpAuthHandlerMock::Factory::~Factory() {
 }
 
-void HttpAuthHandlerMock::Factory::set_mock_handler(
+void HttpAuthHandlerMock::Factory::AddMockHandler(
     HttpAuthHandler* handler, HttpAuth::Target target) {
-  EXPECT_TRUE(handlers_[target].get() == NULL);
-  handlers_[target].reset(handler);
+  handlers_[target].push_back(handler);
 }
 
 int HttpAuthHandlerMock::Factory::CreateAuthHandler(
     HttpAuth::ChallengeTokenizer* challenge,
     HttpAuth::Target target,
     const GURL& origin,
     CreateReason reason,
     int nonce_count,
     const BoundNetLog& net_log,
     scoped_ptr<HttpAuthHandler>* handler) {
-  if (!handlers_[target].get())
+  if (handlers_[target].empty())
     return ERR_UNEXPECTED;
-  scoped_ptr<HttpAuthHandler> tmp_handler(handlers_[target].release());
+  scoped_ptr<HttpAuthHandler> tmp_handler(handlers_[target][0]);

[cbentzel, 2011/08/03 21:22:10]

I'd add an alias to handlers_[target].get() here to make this less confusing. In
all cases we can go through the un-scoped vector. 

[Ryan Hamilton, 2011/08/03 22:03:37]

Done.

+  handlers_[target].get().erase(handlers_[target].get().begin());
   if (do_init_from_challenge_ &&
       !tmp_handler->InitFromChallenge(challenge, target, origin, net_log))
     return ERR_INVALID_RESPONSE;
   handler->swap(tmp_handler);
   return OK;
 }
 
 }  // namespace net