Update demo with features from support lib 23.2.0

Update demo with features from support lib 23.2.0

- Added demo for default share menu action
- Added demo secondary toolbar color change
- Added demo toolbar button


BUG=

Committed: https://github.com/GoogleChrome/custom-tabs-client/commit/30e16577e696b635bfc5f66aafadc8a8033b8082

[andreban, 2016-03-01 11:24:41]

Description was changed from

==========
Update demo with features from support lib 23.2.0

- Added demo for default share menu action
- Added demo secondary toolbar color change
- Added demo toolbar button

Change-Id: I4951d5f310593f0dcdef3fb18b15095695e3a9e6

BUG=
==========

to

==========
Update demo with features from support lib 23.2.0

- Added demo for default share menu action
- Added demo secondary toolbar color change
- Added demo toolbar button


BUG=
==========

[andreban, 2016-03-01 11:24:41]

andreban@google.com changed reviewers:
+ pasko@chromium.org

[pasko, 2016-03-01 14:28:28]

lgtm, thank you

https://codereview.chromium.org/1750963002/diff/1/demos/src/main/java/org/chr...
File
demos/src/main/java/org/chromium/customtabsdemos/ActionBroadcastReceiver.java
(right):

https://codereview.chromium.org/1750963002/diff/1/demos/src/main/java/org/chr...
demos/src/main/java/org/chromium/customtabsdemos/ActionBroadcastReceiver.java:36:
String toastText = getToastText(context, intent.getIntExtra(KEY_ACTION_SOURCE,
-1), url);
nit: line longer than 100 chars

https://codereview.chromium.org/1750963002/diff/1/demos/src/main/java/org/chr...
File demos/src/main/java/org/chromium/customtabsdemos/CustomUIActivity.java
(right):

https://codereview.chromium.org/1750963002/diff/1/demos/src/main/java/org/chr...
demos/src/main/java/org/chromium/customtabsdemos/CustomUIActivity.java:114:
//Generally you do not want to decode bitmaps in the UI thread.
is this a TODO? Why is it OK to decode bitmaps in our case? Small bitmaps?

[andreban, 2016-03-01 15:03:03]

Addressed the comments

https://codereview.chromium.org/1750963002/diff/1/demos/src/main/java/org/chr...
File
demos/src/main/java/org/chromium/customtabsdemos/ActionBroadcastReceiver.java
(right):

https://codereview.chromium.org/1750963002/diff/1/demos/src/main/java/org/chr...
demos/src/main/java/org/chromium/customtabsdemos/ActionBroadcastReceiver.java:36:
String toastText = getToastText(context, intent.getIntExtra(KEY_ACTION_SOURCE,
-1), url);
On 2016/03/01 14:28:28, pasko wrote:
> nit: line longer than 100 chars

Done.

https://codereview.chromium.org/1750963002/diff/1/demos/src/main/java/org/chr...
File demos/src/main/java/org/chromium/customtabsdemos/CustomUIActivity.java
(right):

https://codereview.chromium.org/1750963002/diff/1/demos/src/main/java/org/chr...
demos/src/main/java/org/chromium/customtabsdemos/CustomUIActivity.java:114:
//Generally you do not want to decode bitmaps in the UI thread.
On 2016/03/01 14:28:28, pasko wrote:
> is this a TODO? Why is it OK to decode bitmaps in our case? Small bitmaps?

Done.

[andreban, 2016-03-01 15:28:36]

The CQ bit was checked by andreban@google.com

[andreban, 2016-03-01 15:28:37]

The patchset sent to the CQ was uploaded after l-g-t-m from pasko@chromium.org
Link to the patchset: https://codereview.chromium.org/1750963002/#ps40001
(title: "Fix comment on decoding Bitmpa on UI thread")

[commit-bot: I haz the power, 2016-03-01 15:28:40]

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

[commit-bot: I haz the power, 2016-03-01 15:30:14]

Description was changed from

==========
Update demo with features from support lib 23.2.0

- Added demo for default share menu action
- Added demo secondary toolbar color change
- Added demo toolbar button


BUG=
==========

to

==========
Update demo with features from support lib 23.2.0

- Added demo for default share menu action
- Added demo secondary toolbar color change
- Added demo toolbar button


BUG=

Committed:
https://github.com/GoogleChrome/custom-tabs-client/commit/30e16577e696b635bfc...
==========

[commit-bot: I haz the power, 2016-03-01 15:30:15]

Committed patchset #3 (id:40001) as
https://github.com/GoogleChrome/custom-tabs-client/commit/30e16577e696b635bfc...