Making AutoLoginPrompter::ParseAutoLoginHeader public.

Making AutoLoginPrompter::ParseAutoLoginHeader public.

 iOS needs to parse the AutoLogin header but doesn't follow the same
path than other platform to obtain it. Making this method public ensures
that implementation is shared between platforms.

R=pkasting@chromium.org
TEST=None
BUG=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148128

[qsr, 2012-07-23 15:43:35]

With the right pkasting.

[Peter Kasting, 2012-07-23 18:09:31]

OWNERS LGTM, but you should ensure that whatever you're ultimately doing is OK
with the people who actually wrote and maintain this code.

https://chromiumcodereview.appspot.com/10800074/diff/1/chrome/browser/ui/auto...
File chrome/browser/ui/auto_login_prompter.h (right):

https://chromiumcodereview.appspot.com/10800074/diff/1/chrome/browser/ui/auto...
chrome/browser/ui/auto_login_prompter.h:41: static bool
ParseAutoLoginHeader(const std::string& input, Params* output);
Nit: If you change the declaration order, you need to change the definition
order to match.

[qsr, 2012-07-24 07:01:43]

> OWNERS LGTM, but you should ensure that whatever you're ultimately doing is OK
> with the people who actually wrote and maintain this code.

 Sorry, just looked at the blame command and not the log, and you seemed to be
the right one. Will ask roger to take a look.

[qsr, 2012-07-24 07:02:19]

+Roger for review.

http://codereview.chromium.org/10800074/diff/1/chrome/browser/ui/auto_login_p...
File chrome/browser/ui/auto_login_prompter.h (right):

http://codereview.chromium.org/10800074/diff/1/chrome/browser/ui/auto_login_p...
chrome/browser/ui/auto_login_prompter.h:41: static bool
ParseAutoLoginHeader(const std::string& input, Params* output);
On 2012/07/23 18:09:31, Peter Kasting wrote:
> Nit: If you change the declaration order, you need to change the definition
> order to match.

Done.

[Roger Tawa OOO till Jul 10th, 2012-07-24 14:51:25]

lgtm

[commit-bot: I haz the power, 2012-07-24 15:31:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qsr@chromium.org/10800074/1002

[commit-bot: I haz the power, 2012-07-24 17:00:07]

Change committed as 148128