Remove windowName from CRWSessionController.

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.
 
 #import "ios/web/web_state/web_state_impl.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/logging.h"
@@ skipping 29 matching lines @@
 #include "net/http/http_response_headers.h"
 #include "services/service_manager/public/cpp/interface_registry.h"
 
 namespace web {
 
 /* static */
 std::unique_ptr<WebState> WebState::Create(const CreateParams& params) {
   std::unique_ptr<WebStateImpl> web_state(
       new WebStateImpl(params.browser_state));
 
-  // Initialized the new session.
-  NSString* window_name = nil;
-  BOOL opened_by_dom = NO;
-  web_state->GetNavigationManagerImpl().InitializeSession(window_name,
-                                                          opened_by_dom);
+  // Initialize the new session.
+  web_state->GetNavigationManagerImpl().InitializeSession(NO);

[Eugene But (OOO till 7-30), 2017/02/28 16:04:09]

Could you please either leave opened_by_dom locakl variable or add /*
opened_by_dom */ inlined comment for NO.

[sdefresne, 2017/02/28 17:41:07]

Done.

 
   // This std::move is required to compile with the version of clang shipping
   // with Xcode 8.0+. Evalute whether the issue is fixed once a new version of
   // Xcode is released.
   return std::move(web_state);
 }
 
 /* static */
 std::unique_ptr<WebState> WebState::Create(const CreateParams& params,
                                            CRWSessionStorage* session_storage) {
@@ skipping 664 matching lines @@
     const LoadCommittedDetails& load_details) {
   for (auto& observer : observers_)
     observer.NavigationItemCommitted(load_details);
 }
 
 WebState* WebStateImpl::GetWebState() {
   return this;
 }
 
 }  // namespace web