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

Unified Diff: components/sessions/core/serialized_navigation_entry.cc

Issue 2451583002: Add MDP for TabRestorer.
Patch Set: Created 4 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: components/sessions/core/serialized_navigation_entry.cc
diff --git a/components/sessions/core/serialized_navigation_entry.cc b/components/sessions/core/serialized_navigation_entry.cc
index 08685eb36021a03d8ca5a10c96d00e145d5c45fb..79c54cf56280be627ac2b2e5749cdfe61290f0ee 100644
--- a/components/sessions/core/serialized_navigation_entry.cc
+++ b/components/sessions/core/serialized_navigation_entry.cc
@@ -8,10 +8,18 @@
#include "base/pickle.h"
#include "base/strings/utf_string_conversions.h"
+#include "base/trace_event/estimate_memory_usage.h"
#include "components/sessions/core/serialized_navigation_driver.h"
#include "components/sync/base/time.h"
#include "components/sync/protocol/session_specifics.pb.h"
+// TODO(dskiba): move somemhere
+size_t EstimateMemoryUsage(const GURL& url) {
+ using base::trace_event::EstimateMemoryUsage;
+ return EstimateMemoryUsage(url.possibly_invalid_spec()) +
+ (url.inner_url() ? EstimateMemoryUsage(*url.inner_url()) : 0);
+}
+
namespace sessions {
// TODO(treib): Remove, not needed anymore. crbug.com/627747
@@ -500,4 +508,19 @@ sync_pb::TabNavigation SerializedNavigationEntry::ToSyncData() const {
return sync_data;
}
+size_t SerializedNavigationEntry::EstimateMemoryUsage() const {
+ using base::trace_event::EstimateMemoryUsage;
+ return
+ EstimateMemoryUsage(referrer_url_) +
+ EstimateMemoryUsage(virtual_url_) +
+ EstimateMemoryUsage(title_) +
+ EstimateMemoryUsage(encoded_page_state_) +
+ EstimateMemoryUsage(original_request_url_) +
+ EstimateMemoryUsage(search_terms_) +
+ EstimateMemoryUsage(favicon_url_) +
+ EstimateMemoryUsage(redirect_chain_) +
+ EstimateMemoryUsage(content_pack_categories_) +
+ EstimateMemoryUsage(extended_info_map_);
+}
+
} // namespace sessions
« no previous file with comments | « components/sessions/core/serialized_navigation_entry.h ('k') | components/sessions/core/tab_restore_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698