Add module suffix in .mojom files for components/arc

Add module suffix in .mojom files for components/arc

'module arc;' --> 'module arc.mojom;'
This way generated bindings will consistently live in a mojom
sub-namespace and avoid collision with non-mojom types.

This CL is for these .mojom files:
components/arc/common/*.mojom

BUG=588964
TBR=chrisha@chromium.org,piman@chromium.org

Committed: https://crrev.com/890322916942fbdad77caea1ab9c6f1dc9eff459
Cr-Commit-Position: refs/heads/master@{#387823}

[leonhsl(Using Gerrit), 2016-04-14 06:08:05]

The CQ bit was checked by leon.han@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-04-14 06:08:10]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885683005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885683005/1

[commit-bot: I haz the power, 2016-04-14 06:42:39]

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

[commit-bot: I haz the power, 2016-04-14 06:42:40]

Dry run: This issue passed the CQ dry run.

[leonhsl(Using Gerrit), 2016-04-14 07:04:50]

leon.han@intel.com changed reviewers:
+ lhchavez@chromium.org, tsepez@chromium.org

[leonhsl(Using Gerrit), 2016-04-14 07:04:51]

Would you PTAL at this? Thanks~

+lhchavez@ for components/arc, chrome/browser/chromeos/arc/ and overall

+tsepez@ for components/arc/common/*.mojom

Will add other OWNERS for review after you think this is OK.

[Tom Sepez, 2016-04-14 15:59:21]

RS LGTM based on inspection of .mojom and a few other samples.

[Luis Héctor Chávez, 2016-04-14 17:26:55]

lgtm with nits.

https://codereview.chromium.org/1885683005/diff/1/chrome/browser/memory/tab_m...
File chrome/browser/memory/tab_manager_delegate_chromeos.cc (right):

https://codereview.chromium.org/1885683005/diff/1/chrome/browser/memory/tab_m...
chrome/browser/memory/tab_manager_delegate_chromeos.cc:141: VLOG(1) <<
"arc::mojom::ProcessInstance version < 2 does not "
nit: Can you remove the "arc::mojom::" prefix? Same in the comment below.

https://codereview.chromium.org/1885683005/diff/1/components/arc/input/arc_in...
File components/arc/input/arc_input_bridge.cc (right):

https://codereview.chromium.org/1885683005/diff/1/components/arc/input/arc_in...
components/arc/input/arc_input_bridge.cc:345: VLOG(1) << "ArcBridgeService
mojom::InputInstance disappeared.";
nit: Leave this log as-is.

[leonhsl(Using Gerrit), 2016-04-15 05:50:00]

The CQ bit was checked by leon.han@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-04-15 05:50:17]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885683005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885683005/20001

[leonhsl(Using Gerrit), 2016-04-15 06:22:11]

The CQ bit was checked by leon.han@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-04-15 06:22:28]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885683005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885683005/40001

[leonhsl(Using Gerrit), 2016-04-15 06:33:23]

leon.han@intel.com changed reviewers:
+ afakhry@chromium.org, chrisha@chromium.org, stevenjb@chromium.org

[leonhsl(Using Gerrit), 2016-04-15 06:33:25]

Would you PTAL for OWNERS review? Thanks~

+stevenjb@ for chrome/browser/ui/{app_list, ash}

+chrisha@ for chrome/browser/memory

+afakhry@ for chrome/browser/task_management

https://codereview.chromium.org/1885683005/diff/1/chrome/browser/memory/tab_m...
File chrome/browser/memory/tab_manager_delegate_chromeos.cc (right):

https://codereview.chromium.org/1885683005/diff/1/chrome/browser/memory/tab_m...
chrome/browser/memory/tab_manager_delegate_chromeos.cc:141: VLOG(1) <<
"arc::mojom::ProcessInstance version < 2 does not "
On 2016/04/14 17:26:54, Luis Héctor Chávez wrote:
> nit: Can you remove the "arc::mojom::" prefix? Same in the comment below.

Done. Also done for other places.

https://codereview.chromium.org/1885683005/diff/1/components/arc/input/arc_in...
File components/arc/input/arc_input_bridge.cc (right):

https://codereview.chromium.org/1885683005/diff/1/components/arc/input/arc_in...
components/arc/input/arc_input_bridge.cc:345: VLOG(1) << "ArcBridgeService
mojom::InputInstance disappeared.";
On 2016/04/14 17:26:54, Luis Héctor Chávez wrote:
> nit: Leave this log as-is.

Done.

[commit-bot: I haz the power, 2016-04-15 07:07:23]

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

[commit-bot: I haz the power, 2016-04-15 07:07:25]

Dry run: This issue passed the CQ dry run.

[stevenjb, 2016-04-15 15:43:57]

c/b/ui RS LGTM

[afakhry, 2016-04-15 16:34:51]

On 2016/04/15 15:43:57, stevenjb wrote:
> c/b/ui RS LGTM

c/b/task_management lgtm.

[leonhsl(Using Gerrit), 2016-04-16 13:32:20]

The CQ bit was checked by leon.han@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-04-16 13:32:28]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885683005/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885683005/60001

[leonhsl(Using Gerrit), 2016-04-16 15:26:30]

Description was changed from

==========
Add module suffix in .mojom files for components/arc

'module arc;' --> 'module arc.mojom;'
This way generated bindings will consistently live in a mojom
sub-namespace and avoid collision with non-mojom types.

This CL is for these .mojom files:
components/arc/common/*.mojom

BUG=588964
==========

to

==========
Add module suffix in .mojom files for components/arc

'module arc;' --> 'module arc.mojom;'
This way generated bindings will consistently live in a mojom
sub-namespace and avoid collision with non-mojom types.

This CL is for these .mojom files:
components/arc/common/*.mojom

BUG=588964
TBR=chrisha@chromium.org,piman@chromium.org
==========

[leonhsl(Using Gerrit), 2016-04-16 15:29:53]

leon.han@intel.com changed reviewers:
+ piman@chromium.org

[leonhsl(Using Gerrit), 2016-04-16 15:29:54]

Sorry I'd like to land this CL with TBR:
   chrisha@ for chrome/browser/memory
   piman@ for chrome/gpu

Thanks all~

[leonhsl(Using Gerrit), 2016-04-16 15:31:55]

The CQ bit was unchecked by leon.han@intel.com

[leonhsl(Using Gerrit), 2016-04-16 15:32:12]

The CQ bit was checked by leon.han@intel.com

[leonhsl(Using Gerrit), 2016-04-16 15:32:13]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
lhchavez@chromium.org, afakhry@chromium.org, stevenjb@chromium.org
Link to the patchset: https://codereview.chromium.org/1885683005/#ps60001
(title: "Rebase only")

[commit-bot: I haz the power, 2016-04-16 15:32:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1885683005/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1885683005/60001

[chrisha, 2016-04-16 15:57:31]

lgtm!

[commit-bot: I haz the power, 2016-04-16 23:23:04]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-04-16 23:24:32]

Description was changed from

==========
Add module suffix in .mojom files for components/arc

'module arc;' --> 'module arc.mojom;'
This way generated bindings will consistently live in a mojom
sub-namespace and avoid collision with non-mojom types.

This CL is for these .mojom files:
components/arc/common/*.mojom

BUG=588964
TBR=chrisha@chromium.org,piman@chromium.org
==========

to

==========
Add module suffix in .mojom files for components/arc

'module arc;' --> 'module arc.mojom;'
This way generated bindings will consistently live in a mojom
sub-namespace and avoid collision with non-mojom types.

This CL is for these .mojom files:
components/arc/common/*.mojom

BUG=588964
TBR=chrisha@chromium.org,piman@chromium.org

Committed: https://crrev.com/890322916942fbdad77caea1ab9c6f1dc9eff459
Cr-Commit-Position: refs/heads/master@{#387823}
==========

[commit-bot: I haz the power, 2016-04-16 23:24:34]

Patchset 4 (id:??) landed as
https://crrev.com/890322916942fbdad77caea1ab9c6f1dc9eff459
Cr-Commit-Position: refs/heads/master@{#387823}