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

Unified Diff: chrome/browser/sync/engine/download_updates_command_unittest.cc

Issue 8189003: Send important client side event information to the server. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: committing. Created 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/engine/download_updates_command_unittest.cc
diff --git a/chrome/browser/sync/engine/download_updates_command_unittest.cc b/chrome/browser/sync/engine/download_updates_command_unittest.cc
index 1e2aa2f63ce2e267ed669192433523bbd3eb0e95..53d6bf72d39eacc7d95e3e3152c69c63740904c6 100644
--- a/chrome/browser/sync/engine/download_updates_command_unittest.cc
+++ b/chrome/browser/sync/engine/download_updates_command_unittest.cc
@@ -11,6 +11,7 @@
#include "chrome/browser/sync/test/engine/proto_extension_validator.h"
#include "chrome/browser/sync/test/engine/syncer_command_test.h"
+using ::testing::_;
namespace browser_sync {
using syncable::FIRST_REAL_MODEL_TYPE;
@@ -132,4 +133,15 @@ TEST_F(DownloadUpdatesCommandTest, ExecuteWithPayloads) {
command_.ExecuteImpl(session(source));
}
+TEST_F(DownloadUpdatesCommandTest, VerifyAppendDebugInfo) {
+ sync_pb::DebugInfo debug_info;
+ EXPECT_CALL(*(mock_debug_info_getter()), GetAndClearDebugInfo(_))
+ .Times(1);
+ command_.AppendClientDebugInfoIfNeeded(session(), &debug_info);
+
+ // Now try to add it once more and make sure |GetAndClearDebugInfo| is not
+ // called.
+ command_.AppendClientDebugInfoIfNeeded(session(), &debug_info);
+}
+
} // namespace browser_sync
« no previous file with comments | « chrome/browser/sync/engine/download_updates_command.cc ('k') | chrome/browser/sync/engine/sync_scheduler_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698