Add additional closeAllTabs(boolean uponExit) methods

Add TabModel#closeAllTabs(boolean allowDelegation, boolean uponExit) and TabModelSelector#closeAllTabs(boolean uponExit)

BUG=428084
Committed: https://crrev.com/a46e183d0f98713e8c4921dcff4b74301ddeaf5c
Cr-Commit-Position: refs/heads/master@{#302293}

[Theresa, 2014-10-29 16:29:58]

twellington@chromium.org changed reviewers:
+ dtrainor@chromium.org

[David Trainor- moved to gerrit, 2014-10-29 17:48:54]

lgtm

https://codereview.chromium.org/689693002/diff/1/chrome/android/java/src/org/...
File chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModel.java
(right):

https://codereview.chromium.org/689693002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModel.java:99:
public void closeAllTabs(boolean allowDelegation, boolean uponExit);
Hmm we should get rid of some of these other closeAllTabs() methods.  I'd rather
have one and a utility method that handles passing default values that we can
use for convenience.

Also javadoc!

[Theresa, 2014-10-29 18:00:17]

https://codereview.chromium.org/689693002/diff/1/chrome/android/java/src/org/...
File chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModel.java
(right):

https://codereview.chromium.org/689693002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModel.java:99:
public void closeAllTabs(boolean allowDelegation, boolean uponExit);
On 2014/10/29 17:48:54, David Trainor wrote:
> Hmm we should get rid of some of these other closeAllTabs() methods.  I'd
rather
> have one and a utility method that handles passing default values that we can
> use for convenience.
> 
> Also javadoc!

I'll get rid of the closeAllTabs(boolean allowDelegation) and refactor the
classes that used it. I think we should leave closeAllTabs() for now, and look
into creating a utility method and refactoring in a separate CL since that
change will touch a lot code.

[Theresa, 2014-10-31 17:11:31]

The CQ bit was checked by twellington@chromium.org

[commit-bot: I haz the power, 2014-10-31 17:12:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/689693002/20001

[commit-bot: I haz the power, 2014-10-31 18:56:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-31 18:57:49]

Patchset 2 (id:??) landed as
https://crrev.com/a46e183d0f98713e8c4921dcff4b74301ddeaf5c
Cr-Commit-Position: refs/heads/master@{#302293}