//media/midi: use top level namespace midi rather than media.midi

//media/midi: use top level namespace midi rather than media.midi

This is a preparation for mojofication that recommends us not to use
nested namespace for mojom. With this preparation, C++ code in midi
can access midi.mojom namespace conveniently.

BUG=582327
Committed: https://crrev.com/59e31971365b1085fc53a60f0c8228894869ae49
Cr-Commit-Position: refs/heads/master@{#424969}

[Takashi Toyoshima, 2016-10-12 05:32:20]

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

[commit-bot: I haz the power, 2016-10-12 05:32:34]

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

[Takashi Toyoshima, 2016-10-12 06:40:01]

toyoshim@chromium.org changed reviewers:
+ jochen@chromium.org, michaelbai@chromium.org, tsepez@chromium.org,
yhirano@chromium.org

[Takashi Toyoshima, 2016-10-12 06:40:02]

michaelbai@chromium.org for library_loader_hooks.cc, and Java package name
changes
jochen@chromium.org for content/
tsepez@chromium.org for content/common/media/midi_messages.h
yhirano@chromium.org for media/midi/

Hi reviewers, could you review this namespace change? This contains only
mechanical changes.

Historically midi was placed under media directory, but midi was changed to have
a separate build rules to be a separate component in terms of component build,
but at that point, code was remained in media namespace. But, now I want to
change it to midi for mojofication.

[jochen (gone - plz use gerrit), 2016-10-12 09:38:59]

lgtm

[yhirano, 2016-10-12 11:26:08]

https://codereview.chromium.org/2418493002/diff/20001/media/midi/java/src/org...
File media/midi/java/src/org/chromium/midi/MidiOutputPortAndroid.java (right):

https://codereview.chromium.org/2418493002/diff/20001/media/midi/java/src/org...
media/midi/java/src/org/chromium/midi/MidiOutputPortAndroid.java:39: private
static final String TAG = "media_midi";
Should this be fixed?

[Tom Sepez, 2016-10-12 16:10:07]

RS LGTM on removing namespace from midi_messages.h

[michaelbai, 2016-10-13 03:35:15]

library_loader_hooks.cc, and Java package name LGTM

[Takashi Toyoshima, 2016-10-13 05:28:40]

https://codereview.chromium.org/2418493002/diff/20001/media/midi/java/src/org...
File media/midi/java/src/org/chromium/midi/MidiOutputPortAndroid.java (right):

https://codereview.chromium.org/2418493002/diff/20001/media/midi/java/src/org...
media/midi/java/src/org/chromium/midi/MidiOutputPortAndroid.java:39: private
static final String TAG = "media_midi";
Good eyes, thanks!

[Takashi Toyoshima, 2016-10-13 05:28:57]

The CQ bit was checked by toyoshim@chromium.org

[Takashi Toyoshima, 2016-10-13 05:28:58]

The patchset sent to the CQ was uploaded after l-g-t-m from
michaelbai@chromium.org, tsepez@chromium.org, jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2418493002/#ps40001
(title: "TAG name change s/media_midi/midi/")

[commit-bot: I haz the power, 2016-10-13 05:29:13]

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

[yhirano, 2016-10-13 05:31:11]

lgtm

[commit-bot: I haz the power, 2016-10-13 06:43:56]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-13 06:46:01]

Description was changed from

==========
//media/midi: use top level namespace midi rather than media.midi

This is a preparation for mojofication that recommends us not to use
nested namespace for mojom. With this preparation, C++ code in midi
can access midi.mojom namespace conveniently.

BUG=582327
==========

to

==========
//media/midi: use top level namespace midi rather than media.midi

This is a preparation for mojofication that recommends us not to use
nested namespace for mojom. With this preparation, C++ code in midi
can access midi.mojom namespace conveniently.

BUG=582327

Committed: https://crrev.com/59e31971365b1085fc53a60f0c8228894869ae49
Cr-Commit-Position: refs/heads/master@{#424969}
==========

[commit-bot: I haz the power, 2016-10-13 06:46:02]

Patchset 3 (id:??) landed as
https://crrev.com/59e31971365b1085fc53a60f0c8228894869ae49
Cr-Commit-Position: refs/heads/master@{#424969}