Move theme color logic to Tab

Move theme color logic to Tab

Starts caching the theme color to be used inside tab.
This value depends on whether the tab is incognito and also
the security state and whether the tab is showing an interstitial
page. Changes TabObserver#ondidChangeThemeColor so that in only
pushes actually changes from the cached value.

ChromeActivity starts listening for this signal and pushes any
color changes to ToolbarManager making all ChromeActivity
classes aware of theme color changes.

This should still not cause any visual changes.

BUG=507340
Committed: https://crrev.com/39b1ea70f1f923c763f31ce66dd783ed530dc39f
Cr-Commit-Position: refs/heads/master@{#341559}

[Yusuf, 2015-08-03 05:53:48]

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

[David Trainor- moved to gerrit, 2015-08-03 17:04:28]

lgtm with change to ChromeActivity and nit.

https://chromiumcodereview.appspot.com/1254783003/diff/1/chrome/android/java/...
File chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
(right):

https://chromiumcodereview.appspot.com/1254783003/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java:507:
public void onDidChangeThemeColor(Tab tab, int color) {
Won't this be called for every tab in the model?  Do we need to check if tab ==
the active tab?

https://chromiumcodereview.appspot.com/1254783003/diff/1/chrome/android/java/...
File chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java (right):

https://chromiumcodereview.appspot.com/1254783003/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java:809: if
(mNumPixel16DP == 0) mNumPixel16DP = 16;
Should this just go in the else block above?

[Yusuf, 2015-08-03 17:46:55]

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

https://codereview.chromium.org/1254783003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java:507:
public void onDidChangeThemeColor(Tab tab, int color) {
On 2015/08/03 17:04:28, David Trainor wrote:
> Won't this be called for every tab in the model?  Do we need to check if tab
==
> the active tab?

Done.

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

https://codereview.chromium.org/1254783003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java:809: if
(mNumPixel16DP == 0) mNumPixel16DP = 16;
On 2015/08/03 17:04:28, David Trainor wrote:
> Should this just go in the else block above?

Done.

[Yusuf, 2015-08-03 17:47:00]

The CQ bit was checked by yusufo@chromium.org

[Yusuf, 2015-08-03 17:47:00]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/1254783003/#ps20001
(title: "Nits")

[commit-bot: I haz the power, 2015-08-03 17:48:35]

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

[commit-bot: I haz the power, 2015-08-03 18:29:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-03 18:30:00]

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