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

Side by Side Diff: content/shell/browser/shell_login_dialog.cc

Issue 1010963002: favor DCHECK_CURRENTLY_ON for better logs in content/{public,shell,test}/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase on master Created 5 years, 9 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "content/shell/browser/shell_login_dialog.h" 5 #include "content/shell/browser/shell_login_dialog.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "content/public/browser/browser_thread.h" 10 #include "content/public/browser/browser_thread.h"
11 #include "content/public/browser/resource_dispatcher_host.h" 11 #include "content/public/browser/resource_dispatcher_host.h"
12 #include "net/base/auth.h" 12 #include "net/base/auth.h"
13 #include "net/url_request/url_request.h" 13 #include "net/url_request/url_request.h"
14 #include "ui/gfx/text_elider.h" 14 #include "ui/gfx/text_elider.h"
15 15
16 namespace content { 16 namespace content {
17 17
18 ShellLoginDialog::ShellLoginDialog( 18 ShellLoginDialog::ShellLoginDialog(
19 net::AuthChallengeInfo* auth_info, 19 net::AuthChallengeInfo* auth_info,
20 net::URLRequest* request) : auth_info_(auth_info), 20 net::URLRequest* request) : auth_info_(auth_info),
21 request_(request) { 21 request_(request) {
22 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 22 DCHECK_CURRENTLY_ON(BrowserThread::IO);
23 BrowserThread::PostTask( 23 BrowserThread::PostTask(
24 BrowserThread::UI, FROM_HERE, 24 BrowserThread::UI, FROM_HERE,
25 base::Bind(&ShellLoginDialog::PrepDialog, this, 25 base::Bind(&ShellLoginDialog::PrepDialog, this,
26 base::ASCIIToUTF16(auth_info->challenger.ToString()), 26 base::ASCIIToUTF16(auth_info->challenger.ToString()),
27 base::UTF8ToUTF16(auth_info->realm))); 27 base::UTF8ToUTF16(auth_info->realm)));
28 } 28 }
29 29
30 void ShellLoginDialog::OnRequestCancelled() { 30 void ShellLoginDialog::OnRequestCancelled() {
31 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 31 DCHECK_CURRENTLY_ON(BrowserThread::IO);
32 BrowserThread::PostTask( 32 BrowserThread::PostTask(
33 BrowserThread::UI, FROM_HERE, 33 BrowserThread::UI, FROM_HERE,
34 base::Bind(&ShellLoginDialog::PlatformRequestCancelled, this)); 34 base::Bind(&ShellLoginDialog::PlatformRequestCancelled, this));
35 } 35 }
36 36
37 void ShellLoginDialog::UserAcceptedAuth(const base::string16& username, 37 void ShellLoginDialog::UserAcceptedAuth(const base::string16& username,
38 const base::string16& password) { 38 const base::string16& password) {
39 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 39 DCHECK_CURRENTLY_ON(BrowserThread::UI);
40 BrowserThread::PostTask( 40 BrowserThread::PostTask(
41 BrowserThread::IO, FROM_HERE, 41 BrowserThread::IO, FROM_HERE,
42 base::Bind(&ShellLoginDialog::SendAuthToRequester, this, 42 base::Bind(&ShellLoginDialog::SendAuthToRequester, this,
43 true, username, password)); 43 true, username, password));
44 } 44 }
45 45
46 void ShellLoginDialog::UserCancelledAuth() { 46 void ShellLoginDialog::UserCancelledAuth() {
47 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 47 DCHECK_CURRENTLY_ON(BrowserThread::UI);
48 BrowserThread::PostTask( 48 BrowserThread::PostTask(
49 BrowserThread::IO, FROM_HERE, 49 BrowserThread::IO, FROM_HERE,
50 base::Bind(&ShellLoginDialog::SendAuthToRequester, this, 50 base::Bind(&ShellLoginDialog::SendAuthToRequester, this,
51 false, base::string16(), base::string16())); 51 false, base::string16(), base::string16()));
52 BrowserThread::PostTask( 52 BrowserThread::PostTask(
53 BrowserThread::UI, FROM_HERE, 53 BrowserThread::UI, FROM_HERE,
54 base::Bind(&ShellLoginDialog::PlatformCleanUp, this)); 54 base::Bind(&ShellLoginDialog::PlatformCleanUp, this));
55 } 55 }
56 56
57 ShellLoginDialog::~ShellLoginDialog() { 57 ShellLoginDialog::~ShellLoginDialog() {
58 // Cannot post any tasks here; this object is going away and cannot be 58 // Cannot post any tasks here; this object is going away and cannot be
59 // referenced/dereferenced. 59 // referenced/dereferenced.
60 } 60 }
61 61
62 #if !defined(OS_MACOSX) 62 #if !defined(OS_MACOSX)
63 // Bogus implementations for linking. They are never called because 63 // Bogus implementations for linking. They are never called because
64 // ResourceDispatcherHostDelegate::CreateLoginDelegate returns NULL. 64 // ResourceDispatcherHostDelegate::CreateLoginDelegate returns NULL.
65 // TODO: implement ShellLoginDialog for other platforms, drop this #if 65 // TODO: implement ShellLoginDialog for other platforms, drop this #if
66 void ShellLoginDialog::PlatformCreateDialog(const base::string16& message) {} 66 void ShellLoginDialog::PlatformCreateDialog(const base::string16& message) {}
67 void ShellLoginDialog::PlatformCleanUp() {} 67 void ShellLoginDialog::PlatformCleanUp() {}
68 void ShellLoginDialog::PlatformRequestCancelled() {} 68 void ShellLoginDialog::PlatformRequestCancelled() {}
69 #endif 69 #endif
70 70
71 void ShellLoginDialog::PrepDialog(const base::string16& host, 71 void ShellLoginDialog::PrepDialog(const base::string16& host,
72 const base::string16& realm) { 72 const base::string16& realm) {
73 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 73 DCHECK_CURRENTLY_ON(BrowserThread::UI);
74 // The realm is controlled by the remote server, so there is no reason to 74 // The realm is controlled by the remote server, so there is no reason to
75 // believe it is of a reasonable length. 75 // believe it is of a reasonable length.
76 base::string16 elided_realm; 76 base::string16 elided_realm;
77 gfx::ElideString(realm, 120, &elided_realm); 77 gfx::ElideString(realm, 120, &elided_realm);
78 78
79 base::string16 explanation = 79 base::string16 explanation =
80 base::ASCIIToUTF16("The server ") + host + 80 base::ASCIIToUTF16("The server ") + host +
81 base::ASCIIToUTF16(" requires a username and password."); 81 base::ASCIIToUTF16(" requires a username and password.");
82 82
83 if (!elided_realm.empty()) { 83 if (!elided_realm.empty()) {
84 explanation += base::ASCIIToUTF16(" The server says: "); 84 explanation += base::ASCIIToUTF16(" The server says: ");
85 explanation += elided_realm; 85 explanation += elided_realm;
86 explanation += base::ASCIIToUTF16("."); 86 explanation += base::ASCIIToUTF16(".");
87 } 87 }
88 88
89 PlatformCreateDialog(explanation); 89 PlatformCreateDialog(explanation);
90 } 90 }
91 91
92 void ShellLoginDialog::SendAuthToRequester(bool success, 92 void ShellLoginDialog::SendAuthToRequester(bool success,
93 const base::string16& username, 93 const base::string16& username,
94 const base::string16& password) { 94 const base::string16& password) {
95 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 95 DCHECK_CURRENTLY_ON(BrowserThread::IO);
96 if (success) 96 if (success)
97 request_->SetAuth(net::AuthCredentials(username, password)); 97 request_->SetAuth(net::AuthCredentials(username, password));
98 else 98 else
99 request_->CancelAuth(); 99 request_->CancelAuth();
100 ResourceDispatcherHost::Get()->ClearLoginDelegateForRequest(request_); 100 ResourceDispatcherHost::Get()->ClearLoginDelegateForRequest(request_);
101 101
102 BrowserThread::PostTask( 102 BrowserThread::PostTask(
103 BrowserThread::UI, FROM_HERE, 103 BrowserThread::UI, FROM_HERE,
104 base::Bind(&ShellLoginDialog::PlatformCleanUp, this)); 104 base::Bind(&ShellLoginDialog::PlatformCleanUp, this));
105 } 105 }
106 106
107 } // namespace content 107 } // namespace content
OLDNEW
« no previous file with comments | « content/shell/browser/shell_download_manager_delegate.cc ('k') | content/shell/browser/shell_login_dialog_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698