Moved serialization out of CRWSessionEntry.

ios/web/navigation/navigation_item_impl.h

Index: ios/web/navigation/navigation_item_impl.h
diff --git a/ios/web/navigation/navigation_item_impl.h b/ios/web/navigation/navigation_item_impl.h
index aafc060bc49c4ea5e8e3428d50e7a4dd87c48736..fd6fdda415f4968dd4ec74ce637687ce18d6b66b 100644
--- a/ios/web/navigation/navigation_item_impl.h
+++ b/ios/web/navigation/navigation_item_impl.h
@@ -21,6 +21,7 @@
 namespace web {
 
 class NavigationItemFacadeDelegate;
+class NavigationItemStorageBuilder;
 
 // Implementation of NavigationItem.
 class NavigationItemImpl : public web::NavigationItem {
@@ -141,6 +142,10 @@ class NavigationItemImpl : public web::NavigationItem {
   bool should_skip_repost_form_confirmation_;
   base::scoped_nsobject<NSData> post_data_;
 
+  // The NavigationManItemStorageBuilder functions require access to
+  // private variables of NavigationItemImpl.
+  friend NavigationItemStorageBuilder;

[Eugene But (OOO till 7-30), 2017/01/31 22:12:26]

nit: Do you want to move this to the top (before all ivars)?

[kkhorimoto, 2017/02/03 00:36:09]

Done.

+
   // Whether the item, while loading, was created for a renderer-initiated
   // navigation.  This dictates whether the URL should be displayed before the
   // navigation commits.  It is cleared in |ResetForCommit| and not persisted.