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

Issue 7492044: During SetupSync, check for missing passphrase before waiting for sync (Closed)

Created:
9 years, 5 months ago by Raghu Simha
Modified:
9 years, 5 months ago
Reviewers:
Nicolas Zea
CC:
chromium-reviews, ncarter (slow), tim (not reviewing), idana
Visibility:
Public.

Description

During SetupSync, check for missing passphrase before waiting for sync In SetupSync used by the sync integration tests, clients used to check for a missing passphrase if initial sync failed, and return false if one was required (indicating the fact that forward progress is impossible). Due to a change in the sync startup flow, the check for a required passphrase must now be done prior to waiting for initial sync. This patch makes the above change, and thereby fixes the test case SetPassphraseAndThenSetupSync that currently fails on the sync trybots. TBR=zea@chromium.org BUG=90371 TEST=sync_integration_tests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=93873

Patch Set 1 #

Patch Set 2 : Fix indent #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -6 lines) Patch
M chrome/browser/sync/profile_sync_service_harness.cc View 1 2 chunks +7 lines, -6 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Raghu Simha
Nicolas, please review when you get a chance -- I'm TBRing you on this patch. ...
9 years, 5 months ago (2011-07-25 10:43:58 UTC) #1
Raghu Simha
9 years, 5 months ago (2011-07-25 10:45:46 UTC) #2
Also, note that there are other failing tests on mac_sync that have to do with
mock keychain -- that issue is being fixed in another patch and has nothing to
do with this one.

Powered by Google App Engine
This is Rietveld 408576698