Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(401)

Side by Side Diff: chrome/browser/ui/login/login_prompt.cc

Issue 16290004: Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/login/login_prompt.h" 5 #include "chrome/browser/ui/login/login_prompt.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 auth_info_(auth_info), 80 auth_info_(auth_info),
81 request_(request), 81 request_(request),
82 http_network_session_( 82 http_network_session_(
83 request_->context()->http_transaction_factory()->GetSession()), 83 request_->context()->http_transaction_factory()->GetSession()),
84 password_manager_(NULL), 84 password_manager_(NULL),
85 login_model_(NULL) { 85 login_model_(NULL) {
86 // This constructor is called on the I/O thread, so we cannot load the nib 86 // This constructor is called on the I/O thread, so we cannot load the nib
87 // here. BuildViewForPasswordManager() will be invoked on the UI thread 87 // here. BuildViewForPasswordManager() will be invoked on the UI thread
88 // later, so wait with loading the nib until then. 88 // later, so wait with loading the nib until then.
89 DCHECK(request_) << "LoginHandler constructed with NULL request"; 89 DCHECK(request_) << "LoginHandler constructed with NULL request";
90 DCHECK(auth_info_) << "LoginHandler constructed with NULL auth info"; 90 DCHECK(auth_info_.get()) << "LoginHandler constructed with NULL auth info";
91 91
92 AddRef(); // matched by LoginHandler::ReleaseSoon(). 92 AddRef(); // matched by LoginHandler::ReleaseSoon().
93 93
94 BrowserThread::PostTask( 94 BrowserThread::PostTask(
95 BrowserThread::UI, FROM_HERE, 95 BrowserThread::UI, FROM_HERE,
96 base::Bind(&LoginHandler::AddObservers, this)); 96 base::Bind(&LoginHandler::AddObservers, this));
97 97
98 if (!ResourceRequestInfo::ForRequest(request_)->GetAssociatedRenderView( 98 if (!ResourceRequestInfo::ForRequest(request_)->GetAssociatedRenderView(
99 &render_process_host_id_, &tab_contents_id_)) { 99 &render_process_host_id_, &tab_contents_id_)) {
100 NOTREACHED(); 100 NOTREACHED();
(...skipping 358 matching lines...) Expand 10 before | Expand all | Expand 10 after
459 459
460 LoginHandler* CreateLoginPrompt(net::AuthChallengeInfo* auth_info, 460 LoginHandler* CreateLoginPrompt(net::AuthChallengeInfo* auth_info,
461 net::URLRequest* request) { 461 net::URLRequest* request) {
462 LoginHandler* handler = LoginHandler::Create(auth_info, request); 462 LoginHandler* handler = LoginHandler::Create(auth_info, request);
463 BrowserThread::PostTask( 463 BrowserThread::PostTask(
464 BrowserThread::UI, FROM_HERE, 464 BrowserThread::UI, FROM_HERE,
465 base::Bind(&LoginDialogCallback, request->url(), 465 base::Bind(&LoginDialogCallback, request->url(),
466 make_scoped_refptr(auth_info), make_scoped_refptr(handler))); 466 make_scoped_refptr(auth_info), make_scoped_refptr(handler)));
467 return handler; 467 return handler;
468 } 468 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/gtk/ssl_client_certificate_selector.cc ('k') | chrome/browser/ui/login/login_prompt_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698