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

Side by Side Diff: chrome/browser/signin/chrome_signin_client.cc

Issue 348313003: Create withheld permissions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/signin/chrome_signin_client.h" 5 #include "chrome/browser/signin/chrome_signin_client.h"
6 6
7 #include "chrome/browser/chrome_notification_types.h" 7 #include "chrome/browser/chrome_notification_types.h"
8 #include "chrome/browser/content_settings/cookie_settings.h" 8 #include "chrome/browser/content_settings/cookie_settings.h"
9 #include "chrome/browser/net/chrome_cookie_notification_details.h" 9 #include "chrome/browser/net/chrome_cookie_notification_details.h"
10 #include "chrome/browser/signin/local_auth.h" 10 #include "chrome/browser/signin/local_auth.h"
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 DCHECK(host); 72 DCHECK(host);
73 host->AddObserver(this); 73 host->AddObserver(this);
74 signin_hosts_observed_.insert(host); 74 signin_hosts_observed_.insert(host);
75 } 75 }
76 76
77 void ChromeSigninClient::ClearSigninProcess() { 77 void ChromeSigninClient::ClearSigninProcess() {
78 signin_host_id_ = ChildProcessHost::kInvalidUniqueID; 78 signin_host_id_ = ChildProcessHost::kInvalidUniqueID;
79 } 79 }
80 80
81 bool ChromeSigninClient::IsSigninProcess(int process_id) const { 81 bool ChromeSigninClient::IsSigninProcess(int process_id) const {
82 return process_id == signin_host_id_; 82 return process_id != ChildProcessHost::kInvalidUniqueID &&
83 process_id == signin_host_id_;
83 } 84 }
84 85
85 bool ChromeSigninClient::HasSigninProcess() const { 86 bool ChromeSigninClient::HasSigninProcess() const {
86 return signin_host_id_ != ChildProcessHost::kInvalidUniqueID; 87 return signin_host_id_ != ChildProcessHost::kInvalidUniqueID;
87 } 88 }
88 89
89 void ChromeSigninClient::RenderProcessHostDestroyed(RenderProcessHost* host) { 90 void ChromeSigninClient::RenderProcessHostDestroyed(RenderProcessHost* host) {
90 // It's possible we're listening to a "stale" renderer because it was replaced 91 // It's possible we're listening to a "stale" renderer because it was replaced
91 // with a new process by process-per-site. In either case, stop observing it, 92 // with a new process by process-per-site. In either case, stop observing it,
92 // but only reset signin_host_id_ tracking if this was from the current signin 93 // but only reset signin_host_id_ tracking if this was from the current signin
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
200 201
201 void ChromeSigninClient::UnregisterForCookieChangedNotification() { 202 void ChromeSigninClient::UnregisterForCookieChangedNotification() {
202 // Note that it's allowed to call this method multiple times without an 203 // Note that it's allowed to call this method multiple times without an
203 // intervening call to |RegisterForCookieChangedNotification()|. 204 // intervening call to |RegisterForCookieChangedNotification()|.
204 content::Source<Profile> source(profile_); 205 content::Source<Profile> source(profile_);
205 if (!registrar_.IsRegistered( 206 if (!registrar_.IsRegistered(
206 this, chrome::NOTIFICATION_COOKIE_CHANGED, source)) 207 this, chrome::NOTIFICATION_COOKIE_CHANGED, source))
207 return; 208 return;
208 registrar_.Remove(this, chrome::NOTIFICATION_COOKIE_CHANGED, source); 209 registrar_.Remove(this, chrome::NOTIFICATION_COOKIE_CHANGED, source);
209 } 210 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698