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

Side by Side Diff: sync/internal_api/js_sync_manager_observer_unittest.cc

Issue 10917234: sync: make scheduling logic and job ownership more obvious. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: test + comment + rebase Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "sync/internal_api/js_sync_manager_observer.h" 5 #include "sync/internal_api/js_sync_manager_observer.h"
6 6
7 #include "base/basictypes.h" 7 #include "base/basictypes.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "base/message_loop.h" 9 #include "base/message_loop.h"
10 #include "base/values.h" 10 #include "base/values.h"
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 download_progress_markers, 79 download_progress_markers,
80 false, 80 false,
81 true, 81 true,
82 8, 82 8,
83 5, 83 5,
84 2, 84 2,
85 7, 85 7,
86 sessions::SyncSourceInfo(), 86 sessions::SyncSourceInfo(),
87 false, 87 false,
88 0, 88 0,
89 base::Time::Now(), 89 base::Time::Now());
90 false);
91 DictionaryValue expected_details; 90 DictionaryValue expected_details;
92 expected_details.Set("snapshot", snapshot.ToValue()); 91 expected_details.Set("snapshot", snapshot.ToValue());
93 92
94 EXPECT_CALL(mock_js_event_handler_, 93 EXPECT_CALL(mock_js_event_handler_,
95 HandleJsEvent("onSyncCycleCompleted", 94 HandleJsEvent("onSyncCycleCompleted",
96 HasDetailsAsDictionary(expected_details))); 95 HasDetailsAsDictionary(expected_details)));
97 96
98 js_sync_manager_observer_.OnSyncCycleCompleted(snapshot); 97 js_sync_manager_observer_.OnSyncCycleCompleted(snapshot);
99 PumpLoop(); 98 PumpLoop();
100 } 99 }
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 EXPECT_CALL(mock_js_event_handler_, 137 EXPECT_CALL(mock_js_event_handler_,
139 HandleJsEvent("onUpdatedToken", 138 HandleJsEvent("onUpdatedToken",
140 HasDetailsAsDictionary(redacted_token_details))); 139 HasDetailsAsDictionary(redacted_token_details)));
141 140
142 js_sync_manager_observer_.OnUpdatedToken("sensitive_token"); 141 js_sync_manager_observer_.OnUpdatedToken("sensitive_token");
143 PumpLoop(); 142 PumpLoop();
144 } 143 }
145 144
146 } // namespace 145 } // namespace
147 } // namespace syncer 146 } // namespace syncer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698