[Sync] Implement EstimateMemoryUsage for SharedModelTypeProcessor and ModelTypeWorker

components/sync/engine_impl/model_type_worker.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/model_type_worker.h"
 
 #include <stdint.h>
 
 #include <utility>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/format_macros.h"
 #include "base/guid.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/stringprintf.h"
+#include "base/trace_event/memory_usage_estimator.h"
 #include "components/sync/base/time.h"
 #include "components/sync/engine/model_type_processor.h"
 #include "components/sync/engine_impl/commit_contribution.h"
 #include "components/sync/engine_impl/non_blocking_type_commit_contribution.h"
 #include "components/sync/engine_impl/worker_entity_tracker.h"
+#include "components/sync/protocol/proto_memory_estimations.h"
 #include "components/sync/syncable/syncable_util.h"
 
 namespace syncer {
 
 ModelTypeWorker::ModelTypeWorker(
     ModelType type,
     const sync_pb::ModelTypeState& initial_state,
     bool trigger_initial_sync,
     std::unique_ptr<Cryptographer> cryptographer,
     NudgeHandler* nudge_handler,
@@ skipping 289 matching lines @@
 
 void ModelTypeWorker::AbortMigration() {
   DCHECK(!model_type_state_.initial_sync_done());
   model_type_state_ = sync_pb::ModelTypeState();
   entities_.clear();
   pending_updates_.clear();
   has_encrypted_updates_ = false;
   nudge_handler_->NudgeForInitialDownload(type_);
 }
 
+size_t ModelTypeWorker::EstimateMemoryUsage() const {
+  using base::trace_event::EstimateMemoryUsage;
+  size_t memory_usage = 0;

[Patrick Noland, 2017/03/29 19:42:54]

Should cryptographer be included, since the worker has a private copy?

[pavely, 2017/03/30 01:07:25]

I Included fields that can grow arbitrarily large with number of entities. I
could add cryptographer, it would show fixed cost per datatype and would pollute
cryptographer's class with memory estimation code.

+  memory_usage += EstimateMemoryUsage(model_type_state_);
+  memory_usage += EstimateMemoryUsage(entities_);
+  memory_usage += EstimateMemoryUsage(pending_updates_);
+  return memory_usage;
+}
+
 base::WeakPtr<ModelTypeWorker> ModelTypeWorker::AsWeakPtr() {
   return weak_ptr_factory_.GetWeakPtr();
 }
 
 bool ModelTypeWorker::IsTypeInitialized() const {
   return model_type_state_.initial_sync_done() &&
          !model_type_state_.progress_marker().token().empty();
 }
 
 bool ModelTypeWorker::CanCommitItems() const {
@@ skipping 134 matching lines @@
   return entity_ptr;
 }
 
 WorkerEntityTracker* ModelTypeWorker::GetOrCreateEntityTracker(
     const EntityData& data) {
   WorkerEntityTracker* entity = GetEntityTracker(data.client_tag_hash);
   return entity ? entity : CreateEntityTracker(data);
 }
 
 }  // namespace syncer