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

Side by Side Diff: chrome/browser/sync/engine/syncer_proto_util.cc

Issue 9271061: Cleanup: Remove static storage for variables in an unnamed namespace. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Style change. Created 8 years, 11 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
« no previous file with comments | « base/tracked_objects.cc ('k') | chrome/browser/sync/syncable/syncable.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "chrome/browser/sync/engine/syncer_proto_util.h" 5 #include "chrome/browser/sync/engine/syncer_proto_util.h"
6 6
7 #include "base/format_macros.h" 7 #include "base/format_macros.h"
8 #include "base/stringprintf.h" 8 #include "base/stringprintf.h"
9 #include "chrome/browser/sync/engine/net/server_connection_manager.h" 9 #include "chrome/browser/sync/engine/net/server_connection_manager.h"
10 #include "chrome/browser/sync/engine/syncer.h" 10 #include "chrome/browser/sync/engine/syncer.h"
(...skipping 21 matching lines...) Expand all
32 using syncable::MTIME; 32 using syncable::MTIME;
33 using syncable::PARENT_ID; 33 using syncable::PARENT_ID;
34 using syncable::ScopedDirLookup; 34 using syncable::ScopedDirLookup;
35 35
36 namespace browser_sync { 36 namespace browser_sync {
37 using sessions::SyncSession; 37 using sessions::SyncSession;
38 38
39 namespace { 39 namespace {
40 40
41 // Time to backoff syncing after receiving a throttled response. 41 // Time to backoff syncing after receiving a throttled response.
42 static const int kSyncDelayAfterThrottled = 2 * 60 * 60; // 2 hours 42 const int kSyncDelayAfterThrottled = 2 * 60 * 60; // 2 hours
43
43 void LogResponseProfilingData(const ClientToServerResponse& response) { 44 void LogResponseProfilingData(const ClientToServerResponse& response) {
44 if (response.has_profiling_data()) { 45 if (response.has_profiling_data()) {
45 stringstream response_trace; 46 stringstream response_trace;
46 response_trace << "Server response trace:"; 47 response_trace << "Server response trace:";
47 48
48 if (response.profiling_data().has_user_lookup_time()) { 49 if (response.profiling_data().has_user_lookup_time()) {
49 response_trace << " user lookup: " 50 response_trace << " user lookup: "
50 << response.profiling_data().user_lookup_time() << "ms"; 51 << response.profiling_data().user_lookup_time() << "ms";
51 } 52 }
52 53
(...skipping 479 matching lines...) Expand 10 before | Expand all | Expand 10 after
532 std::string SyncerProtoUtil::ClientToServerResponseDebugString( 533 std::string SyncerProtoUtil::ClientToServerResponseDebugString(
533 const sync_pb::ClientToServerResponse& response) { 534 const sync_pb::ClientToServerResponse& response) {
534 // Add more handlers as needed. 535 // Add more handlers as needed.
535 std::string output; 536 std::string output;
536 if (response.has_get_updates()) 537 if (response.has_get_updates())
537 output.append(GetUpdatesResponseString(response.get_updates())); 538 output.append(GetUpdatesResponseString(response.get_updates()));
538 return output; 539 return output;
539 } 540 }
540 541
541 } // namespace browser_sync 542 } // namespace browser_sync
OLDNEW
« no previous file with comments | « base/tracked_objects.cc ('k') | chrome/browser/sync/syncable/syncable.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698