[Android O] Split out browser notification channels

[Android O] Split out browser notification channels

- Incognito, Media, and Download notifications are all now posted
to their own special channels, which are created on startup.

- The Browser channel remains for all other misc browser notifications.

BUG=704978
Review-Url: https://codereview.chromium.org/2814603003
Cr-Commit-Position: refs/heads/master@{#464079}
Committed: https://chromium.googlesource.com/chromium/src/+/0ecaeb804b4aab2c9aad6953954705499d2ad4a1

[awdf, 2017-04-11 16:53:33]

awdf@chromium.org changed reviewers:
+ peter@chromium.org

[awdf, 2017-04-11 16:53:33]

Told you this would be simple!

Note this depends on the two channels initialization reviews I sent you.

[awdf, 2017-04-11 16:53:59]

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

[commit-bot: I haz the power, 2017-04-11 16:54:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Peter Beverloo, 2017-04-11 17:49:07]

lgtm!

https://codereview.chromium.org/2814603003/diff/1/chrome/android/java/strings...
File chrome/android/java/strings/android_chrome_strings.grd (right):

https://codereview.chromium.org/2814603003/diff/1/chrome/android/java/strings...
chrome/android/java/strings/android_chrome_strings.grd:256: <message
name="IDS_NOTIFICATION_CATEGORY_MEDIA" desc="Label for notifications shown when
media is playing, within a list of notification categories. [CHAR-LIMIT=32]">
or recording

[commit-bot: I haz the power, 2017-04-11 17:59:05]

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

[commit-bot: I haz the power, 2017-04-11 17:59:06]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[awdf, 2017-04-11 18:59:04]

awdf@chromium.org changed reviewers:
+ nyquist@chromium.org

[awdf, 2017-04-11 18:59:04]

+nyquist@ for owners review - Tommy please take a look, thanks!

https://codereview.chromium.org/2814603003/diff/1/chrome/android/java/strings...
File chrome/android/java/strings/android_chrome_strings.grd (right):

https://codereview.chromium.org/2814603003/diff/1/chrome/android/java/strings...
chrome/android/java/strings/android_chrome_strings.grd:256: <message
name="IDS_NOTIFICATION_CATEGORY_MEDIA" desc="Label for notifications shown when
media is playing, within a list of notification categories. [CHAR-LIMIT=32]">
On 2017/04/11 17:49:07, Peter Beverloo wrote:
> or recording

Done.

[nyquist, 2017-04-11 19:36:37]

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

[nyquist, 2017-04-11 19:36:38]

lgtm!

[David Trainor- moved to gerrit, 2017-04-11 20:43:40]

lgtm!?

[awdf, 2017-04-12 16:02:48]

The CQ bit was checked by awdf@chromium.org

[awdf, 2017-04-12 16:02:49]

The patchset sent to the CQ was uploaded after l-g-t-m from peter@chromium.org,
nyquist@chromium.org, dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/2814603003/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2017-04-12 16:04:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[awdf, 2017-04-12 17:09:58]

The CQ bit was checked by awdf@chromium.org

[awdf, 2017-04-12 17:09:58]

The patchset sent to the CQ was uploaded after l-g-t-m from peter@chromium.org,
nyquist@chromium.org, dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/2814603003/#ps60001
(title: "Fix up ChannelsUpdaterTest expectations")

[commit-bot: I haz the power, 2017-04-12 17:10:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-12 17:55:00]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492016998369500,
"parent_rev": "e4b2c606f4fcdfc6d535165bef73bf9d0a9eb157", "commit_rev":
"0ecaeb804b4aab2c9aad6953954705499d2ad4a1"}

[commit-bot: I haz the power, 2017-04-12 17:55:57]

Description was changed from

==========
[Android O] Split out browser notification channels

- Incognito, Media, and Download notifications are all now posted
to their own special channels, which are created on startup.

- The Browser channel remains for all other misc browser notifications.

BUG=704978
==========

to

==========
[Android O] Split out browser notification channels

- Incognito, Media, and Download notifications are all now posted
to their own special channels, which are created on startup.

- The Browser channel remains for all other misc browser notifications.

BUG=704978

Review-Url: https://codereview.chromium.org/2814603003
Cr-Commit-Position: refs/heads/master@{#464079}
Committed:
https://chromium.googlesource.com/chromium/src/+/0ecaeb804b4aab2c9aad69539547...
==========

[commit-bot: I haz the power, 2017-04-12 17:56:01]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/0ecaeb804b4aab2c9aad69539547...