Use "chrome://settings/search#Users" instead of "chrome://settings"

Use "chrome://settings/search#Users" instead of "chrome://settings"
when the user choose to create a new user from the sign in confirmation
dialog.

BUG=175541
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182510

[Roger Tawa OOO till Jul 10th, 2013-02-13 16:11:13]

Hi Drew,

Can you please take a look?  Thanks.

[Andrew T Wilson (Slow), 2013-02-13 20:43:10]

LGTM with one change.

https://codereview.chromium.org/12264002/diff/1/chrome/browser/ui/sync/one_cl...
File chrome/browser/ui/sync/one_click_signin_helper.cc (right):

https://codereview.chromium.org/12264002/diff/1/chrome/browser/ui/sync/one_cl...
chrome/browser/ui/sync/one_click_signin_helper.cc:174:
chrome::ShowSettingsSubPage(args.browser, std::string("search#Users"));
Instead of hard-coding a string, can you add a constant string to
common/url_constants.h, just like kSyncSetupSubPage?

[Roger Tawa OOO till Jul 10th, 2013-02-14 15:39:27]

Thanks Drew.  Changes uploaded.

https://codereview.chromium.org/12264002/diff/1/chrome/browser/ui/sync/one_cl...
File chrome/browser/ui/sync/one_click_signin_helper.cc (right):

https://codereview.chromium.org/12264002/diff/1/chrome/browser/ui/sync/one_cl...
chrome/browser/ui/sync/one_click_signin_helper.cc:174:
chrome::ShowSettingsSubPage(args.browser, std::string("search#Users"));
On 2013/02/13 20:43:10, Andrew T Wilson wrote:
> Instead of hard-coding a string, can you add a constant string to
> common/url_constants.h, just like kSyncSetupSubPage?

Done.

[commit-bot: I haz the power, 2013-02-14 15:39:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rogerta@chromium.org/12264002/8001

[commit-bot: I haz the power, 2013-02-14 19:26:26]

Change committed as 182510