Add ability to exclude page state in ContentSerializedNavigationBuilder

Desktop / Chromium - Nicolas Zea [chromium.org] - 12 July 2017 18:06 EDT

Serialization of page state can be a cpu intensive process as the serializer must recursively traverse a tree of iframes. One of the common cases for serializing navigation entries is Sync, which does not actually use the encoded page state information.

This CL adds a new SerializationOptions param and support for using it to exclude the page state when serializing a NavigationEntry, allowing Sync's tab delegate to bypass encoding the page state.

Bug: 738187 Change-Id: Ib758db83e3f688c7016044e16a9f2818062e1da0 Reviewed-on: https://chromium-review.googlesource.com/560696 Commit-Queue: Nicolas Zea

7192987 Add ability to exclude page state in ContentSerializedNavigationBuilder
.../ui/sync/tab_contents_synced_tab_delegate.cc | 6 +++++-
.../content/content_serialized_navigation_builder.cc | 6 ++++--
.../content/content_serialized_navigation_builder.h | 16 +++++++++++++++-
...content_serialized_navigation_builder_unittest.cc | 20 ++++++++++++++++++++
4 files changed, 44 insertions(+), 4 deletions(-)

Upstream: git.chromium.org


  • Share