Forced cookie removal before sign-in page is shown.

Forced cookie removal before sign-in page is shown.

TEST=none
BUG=chromium-os:18248
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96903

[xiyuan, 2011-08-13 15:43:21]

http://codereview.chromium.org/7629016/diff/1/chrome/browser/ui/webui/chromeo...
File chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc (right):

http://codereview.chromium.org/7629016/diff/1/chrome/browser/ui/webui/chromeo...
chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc:261:
StartClearingCookies();
Think we want to clear cookie regardless of whether we have an email to use or
not, e.g. http://crosbug.com/18925 where we want to show add user UI but reauth
UI might show up because of previous success sign-in.

http://codereview.chromium.org/7629016/diff/1/chrome/browser/ui/webui/chromeo...
chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc:369: 
nit: nuke this empty line

[zel, 2011-08-15 06:24:34]

http://codereview.chromium.org/7629016/diff/1/chrome/browser/ui/webui/chromeo...
File chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc (right):

http://codereview.chromium.org/7629016/diff/1/chrome/browser/ui/webui/chromeo...
chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc:261:
StartClearingCookies();
On 2011/08/13 15:43:21, xiyuan wrote:
> Think we want to clear cookie regardless of whether we have an email to use or
> not, e.g. http://crosbug.com/18925 where we want to show add user UI but
reauth
> UI might show up because of previous success sign-in.


Good catch. Done.

http://codereview.chromium.org/7629016/diff/1/chrome/browser/ui/webui/chromeo...
chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc:369: 
On 2011/08/13 15:43:21, xiyuan wrote:
> nit: nuke this empty line

Done.

[xiyuan, 2011-08-15 16:30:28]

LGTM