Add some docs about mojom (files/IDL).

Add some docs about mojom (files/IDL).

Only "1/3" done, but it's already a lot.

R=vardhan@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/7ac75cd786a9cbba79b2b073656f1c15f0226e9f

[vardhan, 2016-03-03 00:55:38]

lgtm w/ corrections

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md
File docs/intro/mojom_idl.md (right):

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md#new...
docs/intro/mojom_idl.md:22: (which receives request messages and receives
response messages).
and *sends* response messages

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md#new...
docs/intro/mojom_idl.md:59: Names in Mojom are not important. Except in
affecting compatibility at level of
should you mention that naming in the [ServiceName] attribute matters?  not sure
if this attribute is a first-class mojom construct though..

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md#new...
docs/intro/mojom_idl.md:61: arbirarily without any effect on the "meaning" of
the Mojom file (subject to
(sp) arbitrarily

[viettrungluu, 2016-03-03 01:01:54]

Thanks.

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md
File docs/intro/mojom_idl.md (right):

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md#new...
docs/intro/mojom_idl.md:22: (which receives request messages and receives
response messages).
On 2016/03/03 00:55:38, vardhan wrote:
> and *sends* response messages

Done.

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md#new...
docs/intro/mojom_idl.md:59: Names in Mojom are not important. Except in
affecting compatibility at level of
On 2016/03/03 00:55:38, vardhan wrote:
> should you mention that naming in the [ServiceName] attribute matters?  not
sure
> if this attribute is a first-class mojom construct though..

I'll have to mention it somewhere. I'll have to think about where. (I could
discuss it under "annotations". But it may also be worth mentioning here as a
practical matter.)

Let me add a TODO (the solution for everything!) about this.

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md#new...
docs/intro/mojom_idl.md:61: arbirarily without any effect on the "meaning" of
the Mojom file (subject to
On 2016/03/03 00:55:38, vardhan wrote:
> (sp) arbitrarily

Done.

[viettrungluu, 2016-03-03 01:02:31]

Description was changed from

==========
Add some docs about mojom (files/IDL).

Only "1/3" done, but it's already a lot.

R=vardhan@google.com
==========

to

==========
Add some docs about mojom (files/IDL).

Only "1/3" done, but it's already a lot.

R=vardhan@google.com

Committed:
https://chromium.googlesource.com/external/mojo/+/7ac75cd786a9cbba79b2b073656...
==========

[viettrungluu, 2016-03-03 01:02:33]

Committed patchset #2 (id:20001) manually as
7ac75cd786a9cbba79b2b073656f1c15f0226e9f (presubmit successful).

[rudominer, 2016-03-03 01:04:32]

rudominer@chromium.org changed reviewers:
+ rudominer@chromium.org

[rudominer, 2016-03-03 01:04:33]

I pointed out a place where you missed a "the"

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md
File docs/intro/mojom_idl.md (right):

https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md#new...
docs/intro/mojom_idl.md:59: Names in Mojom are not important. Except in
affecting compatibility at level of
at the level of

[viettrungluu, 2016-03-03 17:34:37]

Thanks, will include the fix in the next CL.

On 2016/03/03 01:04:33, rudominer wrote:
> I pointed out a place where you missed a "the"
> 
> https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md
> File docs/intro/mojom_idl.md (right):
> 
>
https://codereview.chromium.org/1762503002/diff/1/docs/intro/mojom_idl.md#new...
> docs/intro/mojom_idl.md:59: Names in Mojom are not important. Except in
> affecting compatibility at level of
> at the level of

[jln (very slow on Chromium), 2016-07-26 23:20:22]

jln@chromium.org changed reviewers:
+ jln@chromium.org

[jln (very slow on Chromium), 2016-07-26 23:20:23]

https://codereview.chromium.org/1762503002/diff/20001/docs/intro/message_pipe...
File docs/intro/message_pipes.md (right):

https://codereview.chromium.org/1762503002/diff/20001/docs/intro/message_pipe...
docs/intro/message_pipes.md:41: combined wait-then-read may form an asynchronous
pattern.)
didn't you mean "may form a synchronous pattern" here?

[vardhan, 2016-07-26 23:36:44]

https://codereview.chromium.org/1762503002/diff/20001/docs/intro/message_pipe...
File docs/intro/message_pipes.md (right):

https://codereview.chromium.org/1762503002/diff/20001/docs/intro/message_pipe...
docs/intro/message_pipes.md:41: combined wait-then-read may form an asynchronous
pattern.)
On 2016/07/26 23:20:23, jln (very slow on Chromium) wrote:
> didn't you mean "may form a synchronous pattern" here?

yeah, i think synchronous pattern is right