Do not select a Tab while closing all tabs or undoing

chrome/android/java/src/org/chromium/chrome/browser/snackbar/undo/UndoBarPopupController.java

Index: chrome/android/java/src/org/chromium/chrome/browser/snackbar/undo/UndoBarPopupController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/snackbar/undo/UndoBarPopupController.java b/chrome/android/java/src/org/chromium/chrome/browser/snackbar/undo/UndoBarPopupController.java
index 4b33b9bf1bec2616f1ff936d2b404908703f2e43..3336974c0d15e86c6ca1b9ea5325d06a67aeb7cc 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/snackbar/undo/UndoBarPopupController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/snackbar/undo/UndoBarPopupController.java
@@ -161,17 +161,19 @@ public class UndoBarPopupController implements SnackbarManager.SnackbarControlle
         RecordHistogram.recordEnumeratedHistogram("AndroidTabCloseUndo.Toast",
                 TAB_CLOSE_UNDO_TOAST_PRESSED, TAB_CLOSE_UNDO_TOAST_COUNT);
         if (actionData instanceof Integer) {
-            cancelTabClosure((Integer) actionData);
+            cancelTabClosure((Integer) actionData, false);

[Ted C, 2015/09/02 20:30:56]

somewhat like my other comment, what if we change cancelTabClosure to only
select a tab if none are currently selected.  This would change the behavior
slightly from this patch, but limiting the api complexity (and the ability for
calling sites to get the model into an unexpected state -- i.e. where no tabs
are selected) seems preferred to me than giving this amount of control.

[Jaekyun Seok (inactive), 2015/09/03 00:27:58]

Done.

         } else {
-            for (Integer id : (List<Integer>) actionData) {
-                cancelTabClosure(id);
+            List<Integer> idList = (List<Integer>) actionData;
+            for (int i = 0; i < idList.size(); i++) {
+                Integer id = idList.get(i);
+                cancelTabClosure(id, i != idList.size() - 1);
             }
         }
     }
 
-    private void cancelTabClosure(int tabId) {
+    private void cancelTabClosure(int tabId, boolean isHidden) {
         TabModel model = mTabModelSelector.getModelForTabId(tabId);
-        if (model != null) model.cancelTabClosure(tabId);
+        if (model != null) model.cancelTabClosure(tabId, isHidden);
     }
 
     /**