Moved NavigationManagerImpl serialization out of CRWSessionController.

ios/web/public/web_state/web_state.h

 // Copyright 2013 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 IOS_WEB_PUBLIC_WEB_STATE_WEB_STATE_H_
 #define IOS_WEB_PUBLIC_WEB_STATE_WEB_STATE_H_
 
 #include <stdint.h>
 
 #include <string>
 #include <vector>
 
 #include "base/callback_forward.h"
+#import "base/mac/scoped_nsobject.h"
 #include "base/memory/weak_ptr.h"
 #include "base/supports_user_data.h"
 #include "ios/web/public/referrer.h"
 #include "ios/web/public/web_state/url_verification_constants.h"
 #include "ui/base/page_transition_types.h"
 #include "ui/base/window_open_disposition.h"
 #include "ui/gfx/geometry/size.h"
 #include "url/gurl.h"
 
 class GURL;
 class SkBitmap;
 
 @class CRWJSInjectionReceiver;
+@class CRWNavigationManagerSerialization;
 @protocol CRWScrollableContent;
 @protocol CRWWebViewProxy;
 typedef id<CRWWebViewProxy> CRWWebViewProxyType;
 @class UIView;
 typedef UIView<CRWScrollableContent> CRWContentView;
 
 namespace base {
 class DictionaryValue;
 class Value;
 }
@@ skipping 10 matching lines @@
 class WebStateDelegate;
 class WebStateObserver;
 class WebStatePolicyDecider;
 class WebStateWeakPtrFactory;
 
 // Core interface for interaction with the web.
 class WebState : public base::SupportsUserData {
  public:
   // Parameters for the Create() method.
   struct CreateParams {
+    // Constructor used to create new WebStates.
     explicit CreateParams(web::BrowserState* browser_state);
+    // Constructor used to create a WebState with restores session history.
+    explicit CreateParams(web::BrowserState* browser_state,
+                          CRWNavigationManagerSerialization* serialization);

[Eugene But (OOO till 7-30), 2017/01/25 16:03:38]

Content does not have serialized state in CreateParams. Do we have a good reason
to diverge from content? I noticed that they have
WebContents::CreateWithSessionStorage. Do you think CreateWithSessionStorage is
something that we should use on iOS?

Serialization seems like an area where diverging from content is unnecessary.

[kkhorimoto, 2017/01/26 07:19:10]

Done.

     ~CreateParams();
 
     web::BrowserState* browser_state;
+    base::scoped_nsobject<CRWNavigationManagerSerialization> serialization;
   };
 
   // Parameters for the OpenURL() method.
   struct OpenURLParams {
     OpenURLParams(const GURL& url,
                   const Referrer& referrer,
                   WindowOpenDisposition disposition,
                   ui::PageTransition transition,
                   bool is_renderer_initiated);
     ~OpenURLParams();
@@ skipping 57 matching lines @@
   virtual void OpenURL(const OpenURLParams& params) = 0;
 
   // Stops any pending navigation.
   virtual void Stop() = 0;
 
   // Gets the NavigationManager associated with this WebState. Can never return
   // null.
   virtual const NavigationManager* GetNavigationManager() const = 0;
   virtual NavigationManager* GetNavigationManager() = 0;
 
+  // Creates a serialized version of the NavigationManager. The returned value
+  // is autoreleased.
+  virtual CRWNavigationManagerSerialization*
+  BuildSerializedNavigationManager() = 0;
+
   // Gets the CRWJSInjectionReceiver associated with this WebState.
   virtual CRWJSInjectionReceiver* GetJSInjectionReceiver() const = 0;
 
   // Runs JavaScript in the main frame's context. If a callback is provided, it
   // will be used to return the result, when the result is available or script
   // execution has failed due to an error.
   // NOTE: Integer values will be returned as Type::DOUBLE because of underlying
   // library limitation.
   typedef base::Callback<void(const base::Value*)> JavaScriptResultCallback;
   virtual void ExecuteJavaScript(const base::string16& javascript) = 0;
@@ skipping 124 matching lines @@
 
   // Returns a WeakPtr<WebState> to the current WebState. Must remain private
   // and only call must be in WebStateWeakPtrFactory. Please consult that class
   // for more details. Remove as part of http://crbug.com/556736.
   virtual base::WeakPtr<WebState> AsWeakPtr() = 0;
 };
 
 }  // namespace web
 
 #endif  // IOS_WEB_PUBLIC_WEB_STATE_WEB_STATE_H_