[USS] Show USS counters in Types tab

components/sync/engine_impl/worker_entity_tracker.cc

 // Copyright 2014 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.
 
 #include "components/sync/engine_impl/worker_entity_tracker.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "components/sync/base/model_type.h"
@@ skipping 109 matching lines @@
   ack->specifics_hash = pending_commit_specifics_hash_;
 
   // Because an in-progress commit blocks the sync thread, we can assume that
   // the item we just committed successfully is exactly the one we have now.
   // Nothing changed it while the commit was happening. Since we're now in
   // sync with the server, we can clear the pending commit.
   ClearPendingCommit();
 }
 
 void WorkerEntityTracker::ReceiveUpdate(const UpdateResponseData& update) {
-  if (update.response_version <= highest_gu_response_version_)
+  if (!UpdateContainsNewVersion(update))
     return;
 
   highest_gu_response_version_ = update.response_version;
   id_ = update.entity->id;
 
   // Got an applicable update newer than any pending updates. It must be safe
   // to discard the old encrypted update, if there was one.
   ClearEncryptedUpdate();
 
   if (IsInConflict()) {
     // Incoming update clobbers the pending commit on the sync thread.
     // The model thread can re-request this commit later if it wants to.
     ClearPendingCommit();
   }
 }
 
+bool WorkerEntityTracker::UpdateContainsNewVersion(
+    const UpdateResponseData& update) {
+  if (update.response_version <= highest_gu_response_version_)

[pavely, 2016/10/19 18:18:59]

It is easier to read if condition follows name/semantic of function:
if (response_version > hughest_gu_response_version)
  return true;

Or even:
return (response_version > hughest_gu_response_version);

[Gang Wu, 2016/10/20 21:35:36]

Done.

+    return false;
+  return true;
+}
+
 bool WorkerEntityTracker::ReceiveEncryptedUpdate(
     const UpdateResponseData& data) {
   if (data.response_version < highest_gu_response_version_)
     return false;
 
   highest_gu_response_version_ = data.response_version;
   encrypted_update_.reset(new UpdateResponseData(data));
   ClearPendingCommit();
   return true;
 }
@@ skipping 36 matching lines @@
 bool WorkerEntityTracker::IsServerKnown() const {
   return base_version_ != kUncommittedVersion;
 }
 
 void WorkerEntityTracker::ClearPendingCommit() {
   pending_commit_.reset();
   pending_commit_specifics_hash_.clear();
 }
 
 }  // namespace syncer