Revert of Mojo: Eliminate duplicate C API symbols

Revert of Mojo: Eliminate duplicate C API symbols (patchset #3 id:80001 of https://codereview.chromium.org/2044023004/ )

Reason for revert:
tentative revert due to linker errors. this doesn't make sense. nothing makes sense. RIP sanity

Original issue's description:
> Mojo: Eliminate duplicate C API symbols
>
> This gets rid of the duplicate definitions of public
> Mojo C API symbols, instead making mojo/public/c/system
> the singular source of these definitions. This allows
> targets to be linked against mojo/public libraries
> without requiring any additional EDK dependencies at
> build time.
>
> To facilitate this the EDK uses the same set of API
> thunks as the public library, and the public library
> exposes a special embedder API (distinct from the DSO
> MojoSetSystemThunks call used by the native app loader)
> for setting these thunks. The MojoSetSystemThunks API
> is moved to a module which is only linked directly into
> app DSOs.
>
> Because mojo/public/c/system and mojo/edk/system are
> now both components, and because there are no longer
> redundant exports between the two, it's now safe for
> targets to depend on either one or both of them as
> needed without encountering duplicate definitions.
>
> Also some opportunistic cleanup of Mojo build rules.
>
> BUG=612500
>
> Committed: https://crrev.com/464e306a82ead839f02f1930a684400228b72495
> Cr-Commit-Position: refs/heads/master@{#398605}

TBR=jam@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=612500
Committed: https://crrev.com/e8e242161954d86ce8e064f8a6515bc818d5b600
Cr-Commit-Position: refs/heads/master@{#398614}

[Ken Rockot(use gerrit already), 2016-06-08 17:58:44]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-06-08 17:58:45]

Created Revert of Mojo: Eliminate duplicate C API symbols

[commit-bot: I haz the power, 2016-06-08 17:59:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2047813003/1

[commit-bot: I haz the power, 2016-06-08 18:01:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-08 18:09:25]

Description was changed from

==========
Revert of Mojo: Eliminate duplicate C API symbols (patchset #3 id:80001 of
https://codereview.chromium.org/2044023004/ )

Reason for revert:
tentative revert due to linker errors. this doesn't make sense. nothing makes
sense. RIP sanity

Original issue's description:
> Mojo: Eliminate duplicate C API symbols
> 
> This gets rid of the duplicate definitions of public
> Mojo C API symbols, instead making mojo/public/c/system
> the singular source of these definitions. This allows
> targets to be linked against mojo/public libraries
> without requiring any additional EDK dependencies at
> build time.
> 
> To facilitate this the EDK uses the same set of API
> thunks as the public library, and the public library
> exposes a special embedder API (distinct from the DSO
> MojoSetSystemThunks call used by the native app loader)
> for setting these thunks. The MojoSetSystemThunks API
> is moved to a module which is only linked directly into
> app DSOs.
> 
> Because mojo/public/c/system and mojo/edk/system are
> now both components, and because there are no longer
> redundant exports between the two, it's now safe for
> targets to depend on either one or both of them as
> needed without encountering duplicate definitions.
> 
> Also some opportunistic cleanup of Mojo build rules.
> 
> BUG=612500
> 
> Committed: https://crrev.com/464e306a82ead839f02f1930a684400228b72495
> Cr-Commit-Position: refs/heads/master@{#398605}

TBR=jam@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=612500
==========

to

==========
Revert of Mojo: Eliminate duplicate C API symbols (patchset #3 id:80001 of
https://codereview.chromium.org/2044023004/ )

Reason for revert:
tentative revert due to linker errors. this doesn't make sense. nothing makes
sense. RIP sanity

Original issue's description:
> Mojo: Eliminate duplicate C API symbols
>
> This gets rid of the duplicate definitions of public
> Mojo C API symbols, instead making mojo/public/c/system
> the singular source of these definitions. This allows
> targets to be linked against mojo/public libraries
> without requiring any additional EDK dependencies at
> build time.
>
> To facilitate this the EDK uses the same set of API
> thunks as the public library, and the public library
> exposes a special embedder API (distinct from the DSO
> MojoSetSystemThunks call used by the native app loader)
> for setting these thunks. The MojoSetSystemThunks API
> is moved to a module which is only linked directly into
> app DSOs.
>
> Because mojo/public/c/system and mojo/edk/system are
> now both components, and because there are no longer
> redundant exports between the two, it's now safe for
> targets to depend on either one or both of them as
> needed without encountering duplicate definitions.
>
> Also some opportunistic cleanup of Mojo build rules.
>
> BUG=612500
>
> Committed: https://crrev.com/464e306a82ead839f02f1930a684400228b72495
> Cr-Commit-Position: refs/heads/master@{#398605}

TBR=jam@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=612500

Committed: https://crrev.com/e8e242161954d86ce8e064f8a6515bc818d5b600
Cr-Commit-Position: refs/heads/master@{#398614}
==========

[commit-bot: I haz the power, 2016-06-08 18:09:27]

Patchset 1 (id:??) landed as
https://crrev.com/e8e242161954d86ce8e064f8a6515bc818d5b600
Cr-Commit-Position: refs/heads/master@{#398614}

[Ken Rockot(use gerrit already), 2016-06-08 18:20:22]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2048053003/ by rockot@chromium.org.

The reason for reverting is: Reverting this didn't fix the failure after all.