Added Setters to NavigationContextImpl.

ios/web/web_state/navigation_context_impl.h

Index: ios/web/web_state/navigation_context_impl.h
diff --git a/ios/web/web_state/navigation_context_impl.h b/ios/web/web_state/navigation_context_impl.h
index 5756cedf4a711abf6da402cadb8935a7261eae6d..de9a9a763df16c5506a22fa0269e73f40cc1d993 100644
--- a/ios/web/web_state/navigation_context_impl.h
+++ b/ios/web/web_state/navigation_context_impl.h
@@ -18,6 +18,12 @@ namespace web {
 class NavigationContextImpl : public NavigationContext {
  public:
   // Creates navigation context for sucessful navigation to a different page.
+  // Response headers will ne null.
+  static std::unique_ptr<NavigationContextImpl> CreateNavigationContext(
+      WebState* web_state,
+      const GURL& url);
+
+  // Creates navigation context for sucessful navigation to a different page.

[kkhorimoto, 2017/04/25 13:53:26]

Do you want to mention that this factory method is deprecated now?  Your test
rename seems to indicate that's the case.

[Eugene But (OOO till 7-30), 2017/04/27 12:18:59]

I'll just remove them in this CL:
https://codereview.chromium.org/2845913003/

   static std::unique_ptr<NavigationContextImpl> CreateNavigationContext(
       WebState* web_state,
       const GURL& url,
@@ -47,6 +53,12 @@ class NavigationContextImpl : public NavigationContext {
   net::HttpResponseHeaders* GetResponseHeaders() const override;
   ~NavigationContextImpl() override;
 
+  // Setters for navigation context data members.
+  void SetIsSameDocument(bool is_same_document);

[kkhorimoto, 2017/04/25 13:53:25]

Do we need to expose a setter for IsSameDocument?  I can see why a normal
NavigationContext might become an error page after some kind of navigation
error, but it doesn't seem possible that a same document navigation becomes a
different-document navigation.

[Eugene But (OOO till 7-30), 2017/04/27 12:18:59]

This is needed for user-initiated fragment change navigations (when user types
in the the url which is the same, but has a different fragment). Also having
setter is generally consistent with the rest of the code.

+  void SetIsErrorPage(bool is_error_page);
+  void SetResponseHeaders(
+      const scoped_refptr<net::HttpResponseHeaders>& response_headers);
+
  private:
   NavigationContextImpl(
       WebState* web_state,