Reduce the memory usage of bookmarks storage

components/bookmarks/browser/bookmark_client.h

 // 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.
 
 #ifndef COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_CLIENT_H_
 #define COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_CLIENT_H_
 
 #include <map>
 #include <utility>
+#include <vector>
 
 #include "base/callback_forward.h"
 #include "base/task/cancelable_task_tracker.h"
 #include "components/bookmarks/browser/bookmark_storage.h"
 #include "components/favicon_base/favicon_callback.h"
 #include "components/favicon_base/favicon_types.h"
 #include "components/keyed_service/core/keyed_service.h"
 
 class GURL;
 
@@ skipping 39 matching lines @@
 
   // Returns true if the embedder supports typed count for URL.
   virtual bool SupportsTypedCountForUrls();
 
   // Retrieves the number of times each bookmark URL has been typed in
   // the Omnibox by the user. For each key (URL) in |url_typed_count_map|,
   // the corresponding value will be updated with the typed count of that URL.
   // |url_typed_count_map| must not be null.
   virtual void GetTypedCountForUrls(UrlTypedCountMap* url_typed_count_map);
 
+  // Metadata keys to be excluded from bookmark nodes in the storage.

[sky, 2017/05/26 03:07:38]

How about: Returns the set of keys that are not restored when reading bookmarks
back in.

[ssid, 2017/05/30 21:39:14]

Done.

+  virtual std::vector<std::string> ExecludedMetaKeys();

[sky, 2017/05/26 03:07:38]

GetExcludedMetaDataKeys().

[ssid, 2017/05/30 21:39:14]

Codec uses MetaInfo everywhere. Changed it to MetaInfo in all chnages in this
cl.

+
   // Returns whether the embedder wants permanent node |node|
   // to always be visible or to only show them when not empty.
   virtual bool IsPermanentNodeVisible(const BookmarkPermanentNode* node) = 0;
 
   // Wrapper around RecordAction defined in base/metrics/user_metrics.h
   // that ensure that the action is posted from the correct thread.
   virtual void RecordAction(const base::UserMetricsAction& action) = 0;
 
   // Returns a task that will be used to load any additional root nodes. This
   // task will be invoked in the Profile's IO task runner.
   virtual LoadExtraCallback GetLoadExtraNodesCallback() = 0;
 
   // Returns true if the |permanent_node| can have its title updated.
   virtual bool CanSetPermanentNodeTitle(const BookmarkNode* permanent_node) = 0;
 
   // Returns true if |node| should sync.
   virtual bool CanSyncNode(const BookmarkNode* node) = 0;
 
   // Returns true if this node can be edited by the user.
   // TODO(joaodasilva): the model should check this more aggressively, and
   // should give the client a means to temporarily disable those checks.
   // http://crbug.com/49598
   virtual bool CanBeEditedByUser(const BookmarkNode* node) = 0;
 };
 
 }  // namespace bookmarks
 
 #endif  // COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_CLIENT_H_