Web MIDI: use midi::TaskService in MidiManagerAlsa

Web MIDI: use midi::TaskService in MidiManagerAlsa

This patch replaces existing MidiManagerAlsa implementation
to support asynchronous method posts with midi::TaskService
that should be used on all platforms now to support such
functionalities.

BUG=672793
Review-Url: https://codereview.chromium.org/2923163003
Cr-Commit-Position: refs/heads/master@{#479374}
Committed: https://chromium.googlesource.com/chromium/src/+/c73f9dc095133071e0d3dd918a9135443aeb8032

[Takashi Toyoshima, 2017-06-06 12:03:14]

Description was changed from

==========
Web MIDI: Use midi::TaskService to run member methods asynchronously

This patch replaces existing MidiManagerAlsa port implementation
to support running asynchronous methods with midi::TaskService
that should be used on all platforms from now on.

BUG=672793
==========

to

==========
Web MIDI: use midi::TaskService in MidiManagerAlsa

This patch replaces existing MidiManagerAlsa implementation
to support asynchronous method posts with midi::TaskService
that should be used on all platforms now to support such
functionalities.

BUG=672793
==========

[Takashi Toyoshima, 2017-06-06 12:03:14]

toyoshim@chromium.org changed reviewers:
+ agoode@chromium.org, yhirano@chromium.org

[Takashi Toyoshima, 2017-06-06 12:03:27]

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

[commit-bot: I haz the power, 2017-06-06 12:03:38]

Dry run: CQ is trying da patch.

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

[Takashi Toyoshima, 2017-06-06 12:06:44]

ptal

https://codereview.chromium.org/2923163003/diff/1/media/midi/midi_manager_als...
File media/midi/midi_manager_alsa.cc (left):

https://codereview.chromium.org/2923163003/diff/1/media/midi/midi_manager_als...
media/midi/midi_manager_alsa.cc:183: // Take lock to ensure that the members
initialized on the IO thread
We should not need this lock any more because now it is always constructed and
destructed on the I/O thread.

[commit-bot: I haz the power, 2017-06-06 14:18:05]

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

[commit-bot: I haz the power, 2017-06-06 14:18:06]

Dry run: This issue passed the CQ dry run.

[Adam Goode, 2017-06-06 14:44:26]

lgtm

This is a nice cleanup to this code.

[yhirano, 2017-06-14 11:58:27]

lgtm

[Takashi Toyoshima, 2017-06-14 12:16:06]

The CQ bit was checked by toyoshim@chromium.org

[Takashi Toyoshima, 2017-06-14 12:16:07]

The patchset sent to the CQ was uploaded after l-g-t-m from agoode@chromium.org
Link to the patchset: https://codereview.chromium.org/2923163003/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2017-06-14 12:16: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-06-14 14:14:41]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1497442566786740,
"parent_rev": "3e63fd07b05beac183215d7907176d10ce62a486", "commit_rev":
"c73f9dc095133071e0d3dd918a9135443aeb8032"}

[commit-bot: I haz the power, 2017-06-14 14:14:52]

Description was changed from

==========
Web MIDI: use midi::TaskService in MidiManagerAlsa

This patch replaces existing MidiManagerAlsa implementation
to support asynchronous method posts with midi::TaskService
that should be used on all platforms now to support such
functionalities.

BUG=672793
==========

to

==========
Web MIDI: use midi::TaskService in MidiManagerAlsa

This patch replaces existing MidiManagerAlsa implementation
to support asynchronous method posts with midi::TaskService
that should be used on all platforms now to support such
functionalities.

BUG=672793

Review-Url: https://codereview.chromium.org/2923163003
Cr-Commit-Position: refs/heads/master@{#479374}
Committed:
https://chromium.googlesource.com/chromium/src/+/c73f9dc095133071e0d3dd918a91...
==========

[commit-bot: I haz the power, 2017-06-14 14:14:55]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/c73f9dc095133071e0d3dd918a91...