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

Issue 6541088: When enabling password sync manually, use the passphrase migration flow to cr... (Closed)

Created:
9 years, 10 months ago by John Gregg
Modified:
9 years ago
CC:
chromium-reviews, Raghu Simha, ncarter (slow), tim (not reviewing), idana
Visibility:
Public.

Description

When enabling password sync manually, use the passphrase migration flow to create a passphrase, rather than the confusing "enter your existing passphrase" dialog. BUG=73791 TEST=see bug Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=75769

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -2 lines) Patch
M chrome/browser/sync/profile_sync_service.cc View 2 chunks +4 lines, -1 line 2 comments Download
M chrome/browser/sync/sync_setup_flow.cc View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
John Gregg
9 years, 10 months ago (2011-02-23 00:04:40 UTC) #1
tim (not reviewing)
LGTM w nits http://codereview.chromium.org/6541088/diff/1/chrome/browser/sync/profile_sync_service.cc File chrome/browser/sync/profile_sync_service.cc (right): http://codereview.chromium.org/6541088/diff/1/chrome/browser/sync/profile_sync_service.cc#newcode735 chrome/browser/sync/profile_sync_service.cc:735: SigninForPassphraseMigration(NULL); comment explaining why we do ...
9 years, 10 months ago (2011-02-23 18:10:44 UTC) #2
John Gregg
9 years, 10 months ago (2011-02-23 18:56:11 UTC) #3
On 2011/02/23 18:10:44, timsteele wrote:
> LGTM w nits
> 
>
http://codereview.chromium.org/6541088/diff/1/chrome/browser/sync/profile_syn...
> File chrome/browser/sync/profile_sync_service.cc (right):
> 
>
http://codereview.chromium.org/6541088/diff/1/chrome/browser/sync/profile_syn...
> chrome/browser/sync/profile_sync_service.cc:735:
> SigninForPassphraseMigration(NULL);
> comment explaining why we do this would be nice
> 
>
http://codereview.chromium.org/6541088/diff/1/chrome/browser/sync/profile_syn...
> chrome/browser/sync/profile_sync_service.cc:767:
> wizard_.Step(SyncSetupWizard::GAIA_LOGIN);
> Can you add a comment here why the step is necessary

Done.

Powered by Google App Engine
This is Rietveld 408576698