IIRC mojo/android is EDK runtime support for Java. It's required for Java
code to use mojo primitives and bindings.

build/ can go away

I'd like to lump common and utils together (public/cpp/utils?)

test/ should at best move to public/cpp/test_support

logging/ can go away - its only users are apptests (dead) and mojo:media,
but either way it doesn't belong in //mojo

platform_handle/ is tricky. It's like a client library, but it has a
dependency on non-public EDK API surface. This makes linking against it...
interesting. As such it's a bit more special than the rest of the client
libs which are all implemented entirely on public mojo APIs. We ought to
add similar support for wrapping/unwrapping base::SharedMemoryHandle, and
that will also require non-public EDK API usage. I'd be interested to hear
proposals on how to lump these things together in some meaningful way.

I agree on converters/ being in the wrong place.

gpu and gles2 seem like they belong together, but i don't see a great place
to combine them (other than gpu itself.) gles2 does depend on
platform_handle which makes it special as far as things in //mojo go.

The README is also conveying the wrong information at this point :)

Given your description, should mojo/android live under mojo/edk/android?

Yes, I think that makes sense

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/110b797e9ba759d96e5c44f665259b4fa51b440a

commit 110b797e9ba759d96e5c44f665259b4fa51b440a
Author: sadrul <sadrul@chromium.org>
Date: Tue Nov 29 03:27:12 2016

mojo/gpu: Remove dead gpu code.

This code is no longer used. So remove this.

BUG= 608017 
TBR=piman@ because presubmit checks after the patch.

Review-Url: https://codereview.chromium.org/2535993002
Cr-Commit-Position: refs/heads/master@{#434867}

[delete] https://crrev.com/3cb41a69e160c567b859273092576d65235f3968/mojo/gpu/BUILD.gn
[delete] https://crrev.com/3cb41a69e160c567b859273092576d65235f3968/mojo/gpu/DEPS
[delete] https://crrev.com/3cb41a69e160c567b859273092576d65235f3968/mojo/gpu/OWNERS
[delete] https://crrev.com/3cb41a69e160c567b859273092576d65235f3968/mojo/gpu/mojo_gles2_impl_autogen.cc
[delete] https://crrev.com/3cb41a69e160c567b859273092576d65235f3968/mojo/gpu/mojo_gles2_impl_autogen.h

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/23ffd510d988a0f266f30fb8ccdf70cbc77da1e9

commit 23ffd510d988a0f266f30fb8ccdf70cbc77da1e9
Author: sammc <sammc@chromium.org>
Date: Tue Jan 03 22:45:01 2017

Remove unused parts of //mojo/common.

BUG= 608017 

Review-Url: https://codereview.chromium.org/2608173002
Cr-Commit-Position: refs/heads/master@{#441228}

[modify] https://crrev.com/23ffd510d988a0f266f30fb8ccdf70cbc77da1e9/mojo/common/BUILD.gn
[modify] https://crrev.com/23ffd510d988a0f266f30fb8ccdf70cbc77da1e9/mojo/common/DEPS
[delete] https://crrev.com/599c47bd57be5be9b13126ca4c7678fa3f067481/mojo/common/data_pipe_file_utils.cc
[modify] https://crrev.com/23ffd510d988a0f266f30fb8ccdf70cbc77da1e9/mojo/common/data_pipe_utils.cc
[modify] https://crrev.com/23ffd510d988a0f266f30fb8ccdf70cbc77da1e9/mojo/common/data_pipe_utils.h
[delete] https://crrev.com/599c47bd57be5be9b13126ca4c7678fa3f067481/mojo/common/user_agent.cc
[delete] https://crrev.com/599c47bd57be5be9b13126ca4c7678fa3f067481/mojo/common/user_agent.h

Calling this fixed. All that's left is mojo/common which is all base:: typemapping stuff, and there's already a separate bug to track moving this to //base/mojom.