Don't drop debug_info when posting to sync server fails

sync/test/sessions/mock_debug_info_getter.h

Index: sync/test/sessions/mock_debug_info_getter.h
diff --git a/sync/test/sessions/mock_debug_info_getter.h b/sync/test/sessions/mock_debug_info_getter.h
new file mode 100644
index 0000000000000000000000000000000000000000..39456776d0c79efa9c963fef345fd934c05fd1a2
--- /dev/null
+++ b/sync/test/sessions/mock_debug_info_getter.h
@@ -0,0 +1,33 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef SYNC_SESSIONS_MOCK_DEBUG_INFO_GETTER_H_
+#define SYNC_SESSIONS_MOCK_DEBUG_INFO_GETTER_H_
+
+#include "base/compiler_specific.h"
+#include "sync/base/sync_export.h"
+#include "sync/protocol/sync.pb.h"
+#include "sync/sessions/debug_info_getter.h"
+
+namespace syncer {
+namespace sessions {
+
+class SYNC_EXPORT_PRIVATE MockDebugInfoGetter

[Nicolas Zea, 2013/11/19 23:43:04]

nit: Brief comment about what the class is.

[maniscalco, 2013/11/20 01:03:32]

Done.

+    : public sessions::DebugInfoGetter {
+ public:
+  MockDebugInfoGetter();
+  virtual ~MockDebugInfoGetter();
+  virtual void GetDebugInfo(sync_pb::DebugInfo* debug_info) OVERRIDE;

[Nicolas Zea, 2013/11/19 23:43:04]

nit: typically we'll comment "DebugInfoGetter implementation" when implementing
an interface.

[maniscalco, 2013/11/20 01:03:32]

Done.

+  virtual void ClearDebugInfo() OVERRIDE;
+
+  void AddDebugEvent();
+
+ private:
+  sync_pb::DebugInfo debug_info_;

[Nicolas Zea, 2013/11/19 23:43:04]

nit: DISALLOW_COPY_AND_ASSIGN(MockDebugInfoGetter);

[maniscalco, 2013/11/20 01:03:32]

Done.

+};
+
+}  // namespace sessions
+}  // namespace syncer
+
+#endif  // SYNC_SESSIONS_MOCK_DEBUG_INFO_GETTER_H_