MD Settings: Add MIDI devices to Content Settings

MD Settings: Add MIDI devices to Content Settings

BUG=691711
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2699793003
Cr-Commit-Position: refs/heads/master@{#451114}
Committed: https://chromium.googlesource.com/chromium/src/+/3d718c1107d57b641ae0a77e6c491c76c3c05fb3

[tommycli, 2017-02-15 23:09:12]

Description was changed from

==========
MD Settings: Add MIDI devices to Content Settings

BUG=691711
==========

to

==========
MD Settings: Add MIDI devices to Content Settings

BUG=691711
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tommycli, 2017-02-15 23:11:37]

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

[commit-bot: I haz the power, 2017-02-15 23:12:50]

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

[tommycli, 2017-02-15 23:40:15]

tommycli@chromium.org changed reviewers:
+ dschuyler@chromium.org

[tommycli, 2017-02-15 23:40:16]

dschuyler: PTAL. There's some UI clarifications needed (see the bug) but the
code itself is ready to review.

Thanks.

Tommy

[commit-bot: I haz the power, 2017-02-16 00:01:11]

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

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

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[tommycli, 2017-02-16 18:33:12]

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

[commit-bot: I haz the power, 2017-02-16 18:34:33]

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

[tommycli, 2017-02-16 19:43:25]

dschuyler: Hello, I have fixed the tests and updated the icon. UI has been
looked over by the PM and he seems satisfied. PTAL, thanks!

[dschuyler, 2017-02-16 20:11:33]

lgtm

[tommycli, 2017-02-16 20:12:32]

On 2017/02/16 20:11:33, dschuyler wrote:
> lgtm

alrighty thank you, submitting

[tommycli, 2017-02-16 20:12:36]

The CQ bit was checked by tommycli@chromium.org

[tommycli, 2017-02-16 20:12:37]

The CQ bit was unchecked by tommycli@chromium.org

[tommycli, 2017-02-16 20:30:05]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2017-02-16 20:31:32]

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-02-16 22:18:30]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487277005330760,
"parent_rev": "9b2bc7465055b5331166a247c6ee2d3842483094", "commit_rev":
"3d718c1107d57b641ae0a77e6c491c76c3c05fb3"}

[commit-bot: I haz the power, 2017-02-16 22:19:11]

Description was changed from

==========
MD Settings: Add MIDI devices to Content Settings

BUG=691711
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Add MIDI devices to Content Settings

BUG=691711
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2699793003
Cr-Commit-Position: refs/heads/master@{#451114}
Committed:
https://chromium.googlesource.com/chromium/src/+/3d718c1107d57b641ae0a77e6c49...
==========

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

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/3d718c1107d57b641ae0a77e6c49...