cros: Fix restoring session windows after crash inconsistent minimized window counts

chrome/browser/sessions/session_restore.cc

Index: chrome/browser/sessions/session_restore.cc
diff --git a/chrome/browser/sessions/session_restore.cc b/chrome/browser/sessions/session_restore.cc
index eeafc2d4fed5bd8ac73abeea509a04cf4f662b5f..b329a94500549d93526455c7b34e76acb5029c6d 100644
--- a/chrome/browser/sessions/session_restore.cc
+++ b/chrome/browser/sessions/session_restore.cc
@@ -410,6 +410,10 @@ class SessionRestoreImpl : public content::NotificationObserver {
           !browser_->profile()->IsOffTheRecord()) {
         // The first set of tabs is added to the existing browser.
         browser = browser_;
+        // If the first set of tabs belongs to minimized browser, the existing
+        // browser should be minimized to keep consistent show state.
+        if ((*i)->show_state == ui::SHOW_STATE_MINIMIZED)

[sky, 2017/02/23 21:41:06]

If there is only one window, I think we should not minimize it. Otherwise it
looks like nothing opened. Also, I think this should set the initial show state
(assuming browser isn't visible already).

[Qiang(Joe) Xu, 2017/03/06 22:36:03]

Yes, I agree. We shall not minimize here. 
Another way is, for restoring pages to the existing browser, we already have the
activated browser (the existing browser), but the first set of tabs may not have
|active_window_id|, which leads to another |browser_to_activate| below.

After this change, it works well on emulator tests. But it may occasionally
break on device, and I find it is because
https://cs.chromium.org/chromium/src/chrome/browser/sessions/session_restore....
function call will change browser from minimized to unminimized. It doesn't
always change. I believe this is a separate issue.

+          browser->window()->Minimize();
       } else {
 #if defined(OS_CHROMEOS)
         chromeos::BootTimesRecorder::Get()->AddLoginTimeMarker(