Ensure History > Recent Tabs restore preserves window disposition.

ios/chrome/browser/sessions/ios_chrome_tab_restore_service_client.mm

Index: ios/chrome/browser/sessions/ios_chrome_tab_restore_service_client.mm
diff --git a/ios/chrome/browser/sessions/ios_chrome_tab_restore_service_client.mm b/ios/chrome/browser/sessions/ios_chrome_tab_restore_service_client.mm
index f8f47ec7b33e42712ffd8008487b9d917478c875..9ab7b92eb181453e256b9d9cb9270c2c6f00b88c 100644
--- a/ios/chrome/browser/sessions/ios_chrome_tab_restore_service_client.mm
+++ b/ios/chrome/browser/sessions/ios_chrome_tab_restore_service_client.mm
@@ -71,7 +71,10 @@ IOSChromeTabRestoreServiceClient::~IOSChromeTabRestoreServiceClient() {}
 
 sessions::LiveTabContext*
 IOSChromeTabRestoreServiceClient::CreateLiveTabContext(
-    const std::string& app_name) {
+    const std::string& unused_app_name,

[sky, 2017/07/11 20:28:09]

Parameters should match that of the header.

[chrisha, 2017/08/14 14:43:21]

I'm generally in favor of that, but I suppose I see it as clear documentation
that the indicated parameters are being ignored. From the style guide they say
to use the same parameter names, but to comment them out:

https://google.github.io/styleguide/cppguide.html#Function_Declarations_and_D...

+    const gfx::Rect& unused_bounds,
+    ui::WindowShowState unused_show_state,
+    const std::string& unused_workspace) {
   return TabRestoreServiceDelegateImplIOSFactory::GetForBrowserState(
       browser_state_);
 }