[ios] Do not use private web// API in ios_web_shell.

ios/web/web_state/web_state_impl.mm

 // 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.
 
 #include "ios/web/web_state/web_state_impl.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/strings/sys_string_conversions.h"
@@ skipping 15 matching lines @@
 #import "ios/web/web_state/ui/crw_web_controller.h"
 #import "ios/web/web_state/ui/crw_web_controller_container_view.h"
 #import "ios/web/web_state/ui/crw_wk_web_view_web_controller.h"
 #include "ios/web/web_state/web_state_facade_delegate.h"
 #import "ios/web/webui/web_ui_ios_controller_factory_registry.h"
 #import "ios/web/webui/web_ui_ios_impl.h"
 #include "net/http/http_response_headers.h"
 
 namespace web {
 
+/* static */
+std::unique_ptr<WebState> WebState::Create(const CreateParams& params) {
+  WebStateImpl* web_state(new WebStateImpl(params.browser_state));

[rohitrao (ping after 24h), 2016/04/21 00:17:18]

Is it worth wrapping this in a unique_ptr too?

[Eugene But (OOO till 7-30), 2016/04/21 00:48:16]

Done.

+
+  NSString* window_name = nil;
+  NSString* opener_id = nil;
+  BOOL opened_by_dom = NO;
+  int opener_navigation_index = 0;
+  web_state->GetNavigationManagerImpl().InitializeSession(
+      window_name, opener_id, opened_by_dom, opener_navigation_index);
+
+  return std::unique_ptr<WebState>(web_state);

[rohitrao (ping after 24h), 2016/04/21 00:17:18]

And then using web_state.release() here?

[Eugene But (OOO till 7-30), 2016/04/21 00:48:16]

Done.

+}
+
 WebStateImpl::WebStateImpl(BrowserState* browser_state)
     : delegate_(nullptr),
       is_loading_(false),
       is_being_destroyed_(false),
       facade_delegate_(nullptr),
       web_controller_(nil),
       navigation_manager_(this, browser_state),
       interstitial_(nullptr),
       weak_factory_(this) {
   GlobalWebStateEventTracker::GetInstance()->OnWebStateCreated(this);
@@ skipping 580 matching lines @@
     const LoadCommittedDetails& load_details) {
   FOR_EACH_OBSERVER(WebStateObserver, observers_,
                     NavigationItemCommitted(load_details));
 }
 
 WebState* WebStateImpl::GetWebState() {
   return this;
 }
 
 }  // namespace web