UI review thread:
https://groups.google.com/a/google.com/forum/#!topic/chrome-ui-review/tEIF6tvoxxU

Menu mocks
https://folio.googleplex.com/chrome-ux/mocks/293-web-view-read-only-mode/120516_Menus#%2F02_AddAllPageActions.png

Bookmarks flow
https://folio.googleplex.com/chrome-ux/mocks/293-web-view-read-only-mode/120516_Menus#%2F03_BookmarksInCCT.png

Downloads flow
https://folio.googleplex.com/chrome-ux/mocks/293-web-view-read-only-mode/120516_Menus#%2F04_DownloadsInCCT.png

Changes:
1) Add "Find in page," RDS, A2HS (persistently), Bookmark and Download
2) Append "in Chrome" to "Downloaded" and "Bookmarked" snackbars

I don't think you need anything else from UX but let me know if otherwise. Reassigning ownership to sbirch@ for triage.

ianwen@ had mentioned, he might have cycles to handle this. sbirch@ if this one is good to go, feel free to assign it to him with a milestone.

Just to throw one more consideration in here that dewittj brought up, it would be great to be able to differentiate in UMA whether the downloaded page was taken from CCT or from a regular Chrome tab just to measure the impact of the change. 

Sam, is this something we'd want for all of the actions? We're happy to help with this for download action!

It should be possible with sequence analysis automatically since we log when a CCT is opened/closed. If you prefer, adding an explicit split sounds fine to me too. Bonus points if there's a way to do it for all the menu items consistently :).

One addition: we should also update the "Download link" option in the context menu to have the same snackbar behavior. And probably append "in Chrome" to the text.

sbirch@, should #6 be a part of Issue 565505 rather than this, since that will handle context menu changes?

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5abc9695b5db94035f872aaaea6c833e70971e2d

commit 5abc9695b5db94035f872aaaea6c833e70971e2d
Author: ltian <ltian@chromium.org>
Date: Thu Jan 05 20:43:05 2017

[Android]Update CCT/Herb menu item

Add more items to CCT/Herb menu including:
1. Bookmark action & Download action to the top bar
2. "Add to home screen" and "Request desktop site" to the menu
3. The return of "find in page"

Udpate text message in the snackbar for Bookmark action and Download
action to display which app the action is execucted. For example, if CCT
is opened in Chrome Dev, the text in the snackbar for Bookmark should be
"Bookmarked in Chrome Dev".

BUG= 671797 

Review-Url: https://codereview.chromium.org/2596973002
Cr-Commit-Position: refs/heads/master@{#441743}

[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/java/res/menu/custom_tabs_menu.xml
[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuPropertiesDelegate.java
[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/java/src/org/chromium/chrome/browser/bookmarks/BookmarkUtils.java
[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabActivity.java
[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabAppMenuPropertiesDelegate.java
[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadSnackbarController.java
[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/java/strings/android_chrome_strings.grd
[modify] https://crrev.com/5abc9695b5db94035f872aaaea6c833e70971e2d/chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/CustomTabActivityTest.java

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1c53fd3cf915d7700bbb2702061e90845c6a120d

commit 1c53fd3cf915d7700bbb2702061e90845c6a120d
Author: tedchoc <tedchoc@chromium.org>
Date: Fri Jan 06 17:26:47 2017

Fix misnamed CCT find in page UMA action.

TBR=yusufo@chromium.org
BUG= 671797 

Review-Url: https://codereview.chromium.org/2611163005
Cr-Commit-Position: refs/heads/master@{#441961}

[modify] https://crrev.com/1c53fd3cf915d7700bbb2702061e90845c6a120d/chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabActivity.java

Woohoo! To Tal's request in #3, it would be easier to see these actions in sequence if we had a user action for the CCT menu being opened initially. "CustomTabsMenuOpened" perhaps?

Verified on 57.0.2977.0. Thank you!

Still pending Sam's query in comment#11.