Fix ChromeShell TabObservers and upstream more things

Fix ChromeShell TabObservers and upstream more things

* Fixes various things that were wrong with ChromeShell and Observers,
  including having its ChromeShellTabChromeWebContentsDelegateAndroid
  properly call the super class methods instead of flat-out
  ignoring them.

* Adds a pair of tests for the ChromeShell tab switcher to confirm
  that it is brought forth and sent away properly and properly
  monitoring tab title changes.  The latter test catches a case
  where the TabObserver was permanently being discarded after
  entering and exiting the tab switcher once.

* Upstreams logic required for tab title changing.

* Upstreams more logic required for unfreezing frozen tabs,
  which resulted in bringing in a truckload of other functions
  like restoreIfNeeded() and the various initialize() functions.

* Upstreams random functions from ChromeTab where ChromeTab was
  using Tab fields indirectly to react to events that the Tab
  should have been monitoring (e.g. didStartPageLoad).
  Also includes some calls to update the FullscreenManager about
  events, since the FullscreenManager was previously upstreamed.

BUG=443773
TEST=AccessibilityTabSwitcherTest

Committed: https://crrev.com/c0ed134137c98c2935bf32e85f74d4e94c2b980d
Cr-Commit-Position: refs/heads/master@{#309847}

[gone, 2014-12-31 03:09:55]

dfalcantara@chromium.org changed reviewers:
+ dtrainor@chromium.org, tedchoc@chromium.org

[Ted C, 2015-01-02 18:14:45]

https://codereview.chromium.org/834723002/diff/1/chrome/android/java/src/org/...
File chrome/android/java/src/org/chromium/chrome/browser/Tab.java (right):

https://codereview.chromium.org/834723002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:1158:
initializeNative();
this should go in internalInit per offline discussion

https://codereview.chromium.org/834723002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:1185: 
Looks like an extra blank line here

https://codereview.chromium.org/834723002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:1243: public void
initializeNative() {
make protected per offline discussion

https://codereview.chromium.org/834723002/diff/1/chrome/android/javatests/src...
File
chrome/android/javatests/src/org/chromium/chrome/browser/widget/accessibility/AccessibilityTabSwitcherTest.java
(right):

https://codereview.chromium.org/834723002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/widget/accessibility/AccessibilityTabSwitcherTest.java:1:
// Copyright 2012 The Chromium Authors. All rights reserved.
2014

https://codereview.chromium.org/834723002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/widget/accessibility/AccessibilityTabSwitcherTest.java:40:
assertTrue(CriteriaHelper.pollForCriteria(new Criteria() {
Would pollForUIThreadCriteria be better for this (and pretty much most of the
others)?

https://codereview.chromium.org/834723002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/widget/accessibility/AccessibilityTabSwitcherTest.java:55:
TestTouchUtils.performClickOnMainSync(getInstrumentation(),
Use TouchCommon if at all possible.

[gone, 2015-01-02 18:23:02]

https://codereview.chromium.org/834723002/diff/1/chrome/android/java/src/org/...
File chrome/android/java/src/org/chromium/chrome/browser/Tab.java (right):

https://codereview.chromium.org/834723002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:1158:
initializeNative();
On 2015/01/02 18:14:45, Ted C wrote:
> this should go in internalInit per offline discussion

Done.

https://codereview.chromium.org/834723002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:1185: 
On 2015/01/02 18:14:45, Ted C wrote:
> Looks like an extra blank line here

Done.

https://codereview.chromium.org/834723002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:1243: public void
initializeNative() {
On 2015/01/02 18:14:45, Ted C wrote:
> make protected per offline discussion

Added TODO since DocumentTabModelImpl needs access to it.

https://codereview.chromium.org/834723002/diff/1/chrome/android/javatests/src...
File
chrome/android/javatests/src/org/chromium/chrome/browser/widget/accessibility/AccessibilityTabSwitcherTest.java
(right):

https://codereview.chromium.org/834723002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/widget/accessibility/AccessibilityTabSwitcherTest.java:1:
// Copyright 2012 The Chromium Authors. All rights reserved.
On 2015/01/02 18:14:45, Ted C wrote:
> 2014

Done.

https://codereview.chromium.org/834723002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/widget/accessibility/AccessibilityTabSwitcherTest.java:40:
assertTrue(CriteriaHelper.pollForCriteria(new Criteria() {
On 2015/01/02 18:14:45, Ted C wrote:
> Would pollForUIThreadCriteria be better for this (and pretty much most of the
> others)?

Done.

https://codereview.chromium.org/834723002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/widget/accessibility/AccessibilityTabSwitcherTest.java:55:
TestTouchUtils.performClickOnMainSync(getInstrumentation(),
On 2015/01/02 18:14:45, Ted C wrote:
> Use TouchCommon if at all possible.

Done.

[Ted C, 2015-01-02 18:38:44]

lgtm

[gone, 2015-01-02 18:39:33]

The CQ bit was checked by dfalcantara@chromium.org

[commit-bot: I haz the power, 2015-01-02 18:40:09]

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

[commit-bot: I haz the power, 2015-01-02 19:20:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-02 19:21:23]

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