Plumb the anonymize rect color to toolbar_layer from Tab#getThemeColor

Plumb the anonymize rect color to toolbar_layer from Tab#getThemeColor

Makes LayoutTab aware of the textbox background color so that it can
be used for anonymizeRect background. Changes the boolean used to
set the color for anonymizeRect to be an int and signal the actual
color. Adds tinting to toolbarPhone animation drawables so that
fullscreen works properly.

No visual changes. Yet.

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

[Yusuf, 2015-08-03 20:06:34]

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

[David Trainor- moved to gerrit, 2015-08-03 20:12:44]

lgtm

https://codereview.chromium.org/1268483004/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/document/BrandColorUtils.java
(right):

https://codereview.chromium.org/1268483004/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/document/BrandColorUtils.java:56:
private static int getColorWithOverlay(int baseColor, int overlayColor, float
overlayAlpha) {
cache 1.f - overlayAlpha to baseAlpha IMO.

https://codereview.chromium.org/1268483004/diff/1/chrome/browser/android/comp...
File chrome/browser/android/compositor/layer/toolbar_layer.cc (right):

https://codereview.chromium.org/1268483004/diff/1/chrome/browser/android/comp...
chrome/browser/android/compositor/layer/toolbar_layer.cc:14: const SkColor
kNormalAnonymizeContentColor = SK_ColorWHITE;
Do we still need this one?  Let's just leave the color as default, as whenever
it's shown we'll have a valid color right?

[Yusuf, 2015-08-03 20:39:08]

https://codereview.chromium.org/1268483004/diff/1/chrome/browser/android/comp...
File chrome/browser/android/compositor/layer/toolbar_layer.cc (right):

https://codereview.chromium.org/1268483004/diff/1/chrome/browser/android/comp...
chrome/browser/android/compositor/layer/toolbar_layer.cc:14: const SkColor
kNormalAnonymizeContentColor = SK_ColorWHITE;
On 2015/08/03 20:12:44, David Trainor wrote:
> Do we still need this one?  Let's just leave the color as default, as whenever
> it's shown we'll have a valid color right?

We use it on construction, but yeah we will always update to the valid color.

[Yusuf, 2015-08-25 20:55:25]

Rebased this and fixed a NPE. Also moved all theme color related controls to Tab
so that we don't have to worry about enabling/disabling in multiple places.

This is now ready to land. Again no visual changes, but adds essential plumbing.

https://codereview.chromium.org/1268483004/diff/1/chrome/browser/android/comp...
File chrome/browser/android/compositor/layer/toolbar_layer.cc (right):

https://codereview.chromium.org/1268483004/diff/1/chrome/browser/android/comp...
chrome/browser/android/compositor/layer/toolbar_layer.cc:14: const SkColor
kNormalAnonymizeContentColor = SK_ColorWHITE;
On 2015/08/03 20:39:08, Yusuf wrote:
> On 2015/08/03 20:12:44, David Trainor wrote:
> > Do we still need this one?  Let's just leave the color as default, as
whenever
> > it's shown we'll have a valid color right?
> 
> We use it on construction, but yeah we will always update to the valid color.

Updated to remove this in the last patch.

[David Trainor- moved to gerrit, 2015-08-26 00:16:40]

lgtm

https://codereview.chromium.org/1268483004/diff/60001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/document/BrandColorUtils.java
(right):

https://codereview.chromium.org/1268483004/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/document/BrandColorUtils.java:51:
boolean shouldUseOpaqueTextboxBackground =
shouldUseOpaqueTextboxBackground(color);
do we need this boolean?

https://codereview.chromium.org/1268483004/diff/60001/chrome/browser/android/...
File chrome/browser/android/compositor/compositor_view.cc (right):

https://codereview.chromium.org/1268483004/diff/60001/chrome/browser/android/...
chrome/browser/android/compositor/compositor_view.cc:264: resource,
SK_ColorWHITE, false, false, brightness);
This is the default toolbar texture shown in the static layout (surprise!). 
You'll probably want to eventually pipe the active tab's color here.

[Yusuf, 2015-08-26 20:06:49]

https://codereview.chromium.org/1268483004/diff/60001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/document/BrandColorUtils.java
(right):

https://codereview.chromium.org/1268483004/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/document/BrandColorUtils.java:51:
boolean shouldUseOpaqueTextboxBackground =
shouldUseOpaqueTextboxBackground(color);
On 2015/08/26 00:16:40, David Trainor wrote:
> do we need this boolean?

Done.

https://codereview.chromium.org/1268483004/diff/60001/chrome/browser/android/...
File chrome/browser/android/compositor/compositor_view.cc (right):

https://codereview.chromium.org/1268483004/diff/60001/chrome/browser/android/...
chrome/browser/android/compositor/compositor_view.cc:264: resource,
SK_ColorWHITE, false, false, brightness);
On 2015/08/26 00:16:40, David Trainor wrote:
> This is the default toolbar texture shown in the static layout (surprise!). 
> You'll probably want to eventually pipe the active tab's color here.

Oh. I see. I agree, since we don't show the anonymize rect in here it works OK
now, but for using this color for tinting the texture itself on the cc side, the
color should also be piped here.

[Yusuf, 2015-08-26 20:06:59]

The CQ bit was checked by yusufo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-08-26 20:08:10]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1268483004/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1268483004/80001

[commit-bot: I haz the power, 2015-08-26 21:04:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-26 21:04:46]

Dry run: This issue passed the CQ dry run.

[Yusuf, 2015-08-26 21:10:17]

The CQ bit was checked by yusufo@chromium.org

[Yusuf, 2015-08-26 21:10:17]

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/1268483004/#ps80001
(title: "Removed unnecessary boolean")

[commit-bot: I haz the power, 2015-08-26 21:11:14]

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

[commit-bot: I haz the power, 2015-08-26 21:19:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-26 21:19:25]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Yusuf, 2015-08-26 21:52:03]

The CQ bit was checked by yusufo@chromium.org

[Yusuf, 2015-08-26 21:52:03]

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/1268483004/#ps100001
(title: "Rebased")

[commit-bot: I haz the power, 2015-08-26 21:53:04]

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

[commit-bot: I haz the power, 2015-08-26 22:13:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-26 22:13:06]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_arm64_d...)
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[Yusuf, 2015-08-26 22:19:40]

The CQ bit was checked by yusufo@chromium.org

[Yusuf, 2015-08-26 22:19:40]

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/1268483004/#ps120001
(title: "Renamed BrandColorUtils in implementation")

[commit-bot: I haz the power, 2015-08-26 22:19:57]

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

[commit-bot: I haz the power, 2015-08-26 23:10:54]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2015-08-26 23:11:36]

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