Revert of [Presentation API] Convert presentation.mojom to new wrapper types.

Revert of [Presentation API] Convert presentation.mojom to new wrapper types. (patchset #5 id:80001 of https://codereview.chromium.org/2181163003/ )

Reason for revert:
Suspect (not the first) for widespread NaCl test failures since
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/builds/55504

Errors like

[4224:3200:0729/162247:ERROR:nacl_process_host.cc(337)] NaCl process exited with status -1073741674 (0xc0000096)
...
  Actual: "Plugin crashed. 'NaCl module crashed'"

Wasn't r408782 or r408787. Running out of candidates..

Original issue's description:
> [Presentation API] Convert presentation.mojom to new wrapper types.
>
> This converts presentation.mojom to use the new (std::) Mojo wrapper types:
> mojo::String => std::string
> mojo::Array => std::vector
>
> No behavior changes should ensue.
>
> Optional values are represented as base::Optional<Foo> with the new wrapper
> types requiring some additional changes.
>
> This work will simplify further changes for crbug.com/627655.
>
> BUG=627655
>
> Committed: https://crrev.com/59e7b40948030815a49af887b922b370ba048b8b
> Cr-Commit-Position: refs/heads/master@{#408781}

TBR=dcheng@chromium.org,imcheng@chromium.org,mfoltz@chromium.org
NOTRY=true
BUG=627655
Committed: https://crrev.com/43c746be32b2bfce5c3fd654894e4b7f7bf6387f
Cr-Commit-Position: refs/heads/master@{#408905}

[tapted, 2016-08-01 03:12:03]

The CQ bit was checked by tapted@chromium.org

[tapted, 2016-08-01 03:12:03]

Created Revert of [Presentation API] Convert presentation.mojom to new wrapper
types.

[commit-bot: I haz the power, 2016-08-01 03:12:13]

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

[tapted, 2016-08-01 03:23:42]

Description was changed from

==========
Revert of [Presentation API] Convert presentation.mojom to new wrapper types.
(patchset #5 id:80001 of https://codereview.chromium.org/2181163003/ )

Reason for revert:
Suspect (not the first) for widespread NaCl test failures since
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Errors like

[4224:3200:0729/162247:ERROR:nacl_process_host.cc(337)] NaCl process exited with
status -1073741674 (0xc0000096)
...
  Actual: "Plugin crashed. 'NaCl module crashed'"

Wasn't r408782 or r408787. Running out of candidates..

Original issue's description:
> [Presentation API] Convert presentation.mojom to new wrapper types.
> 
> This converts presentation.mojom to use the new (std::) Mojo wrapper types:
> mojo::String => std::string
> mojo::Array => std::vector
> 
> No behavior changes should ensue.
> 
> Optional values are represented as base::Optional<Foo> with the new wrapper
> types requiring some additional changes.
> 
> This work will simplify further changes for crbug.com/627655.
> 
> BUG=627655
> 
> Committed: https://crrev.com/59e7b40948030815a49af887b922b370ba048b8b
> Cr-Commit-Position: refs/heads/master@{#408781}

TBR=dcheng@chromium.org,imcheng@chromium.org,mfoltz@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=627655
==========

to

==========
Revert of [Presentation API] Convert presentation.mojom to new wrapper types.
(patchset #5 id:80001 of https://codereview.chromium.org/2181163003/ )

Reason for revert:
Suspect (not the first) for widespread NaCl test failures since
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Errors like

[4224:3200:0729/162247:ERROR:nacl_process_host.cc(337)] NaCl process exited with
status -1073741674 (0xc0000096)
...
  Actual: "Plugin crashed. 'NaCl module crashed'"

Wasn't r408782 or r408787. Running out of candidates..

Original issue's description:
> [Presentation API] Convert presentation.mojom to new wrapper types.
> 
> This converts presentation.mojom to use the new (std::) Mojo wrapper types:
> mojo::String => std::string
> mojo::Array => std::vector
> 
> No behavior changes should ensue.
> 
> Optional values are represented as base::Optional<Foo> with the new wrapper
> types requiring some additional changes.
> 
> This work will simplify further changes for crbug.com/627655.
> 
> BUG=627655
> 
> Committed: https://crrev.com/59e7b40948030815a49af887b922b370ba048b8b
> Cr-Commit-Position: refs/heads/master@{#408781}

TBR=dcheng@chromium.org,imcheng@chromium.org,mfoltz@chromium.org
NOTRY=true
BUG=627655
==========

[tapted, 2016-08-01 03:23:46]

The CQ bit was unchecked by tapted@chromium.org

[tapted, 2016-08-01 03:23:47]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2016-08-01 03:23:53]

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

[commit-bot: I haz the power, 2016-08-01 03:31:19]

Description was changed from

==========
Revert of [Presentation API] Convert presentation.mojom to new wrapper types.
(patchset #5 id:80001 of https://codereview.chromium.org/2181163003/ )

Reason for revert:
Suspect (not the first) for widespread NaCl test failures since
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Errors like

[4224:3200:0729/162247:ERROR:nacl_process_host.cc(337)] NaCl process exited with
status -1073741674 (0xc0000096)
...
  Actual: "Plugin crashed. 'NaCl module crashed'"

Wasn't r408782 or r408787. Running out of candidates..

Original issue's description:
> [Presentation API] Convert presentation.mojom to new wrapper types.
> 
> This converts presentation.mojom to use the new (std::) Mojo wrapper types:
> mojo::String => std::string
> mojo::Array => std::vector
> 
> No behavior changes should ensue.
> 
> Optional values are represented as base::Optional<Foo> with the new wrapper
> types requiring some additional changes.
> 
> This work will simplify further changes for crbug.com/627655.
> 
> BUG=627655
> 
> Committed: https://crrev.com/59e7b40948030815a49af887b922b370ba048b8b
> Cr-Commit-Position: refs/heads/master@{#408781}

TBR=dcheng@chromium.org,imcheng@chromium.org,mfoltz@chromium.org
NOTRY=true
BUG=627655
==========

to

==========
Revert of [Presentation API] Convert presentation.mojom to new wrapper types.
(patchset #5 id:80001 of https://codereview.chromium.org/2181163003/ )

Reason for revert:
Suspect (not the first) for widespread NaCl test failures since
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Errors like

[4224:3200:0729/162247:ERROR:nacl_process_host.cc(337)] NaCl process exited with
status -1073741674 (0xc0000096)
...
  Actual: "Plugin crashed. 'NaCl module crashed'"

Wasn't r408782 or r408787. Running out of candidates..

Original issue's description:
> [Presentation API] Convert presentation.mojom to new wrapper types.
> 
> This converts presentation.mojom to use the new (std::) Mojo wrapper types:
> mojo::String => std::string
> mojo::Array => std::vector
> 
> No behavior changes should ensue.
> 
> Optional values are represented as base::Optional<Foo> with the new wrapper
> types requiring some additional changes.
> 
> This work will simplify further changes for crbug.com/627655.
> 
> BUG=627655
> 
> Committed: https://crrev.com/59e7b40948030815a49af887b922b370ba048b8b
> Cr-Commit-Position: refs/heads/master@{#408781}

TBR=dcheng@chromium.org,imcheng@chromium.org,mfoltz@chromium.org
NOTRY=true
BUG=627655
==========

[commit-bot: I haz the power, 2016-08-01 03:31:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-01 03:33:30]

Description was changed from

==========
Revert of [Presentation API] Convert presentation.mojom to new wrapper types.
(patchset #5 id:80001 of https://codereview.chromium.org/2181163003/ )

Reason for revert:
Suspect (not the first) for widespread NaCl test failures since
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Errors like

[4224:3200:0729/162247:ERROR:nacl_process_host.cc(337)] NaCl process exited with
status -1073741674 (0xc0000096)
...
  Actual: "Plugin crashed. 'NaCl module crashed'"

Wasn't r408782 or r408787. Running out of candidates..

Original issue's description:
> [Presentation API] Convert presentation.mojom to new wrapper types.
> 
> This converts presentation.mojom to use the new (std::) Mojo wrapper types:
> mojo::String => std::string
> mojo::Array => std::vector
> 
> No behavior changes should ensue.
> 
> Optional values are represented as base::Optional<Foo> with the new wrapper
> types requiring some additional changes.
> 
> This work will simplify further changes for crbug.com/627655.
> 
> BUG=627655
> 
> Committed: https://crrev.com/59e7b40948030815a49af887b922b370ba048b8b
> Cr-Commit-Position: refs/heads/master@{#408781}

TBR=dcheng@chromium.org,imcheng@chromium.org,mfoltz@chromium.org
NOTRY=true
BUG=627655
==========

to

==========
Revert of [Presentation API] Convert presentation.mojom to new wrapper types.
(patchset #5 id:80001 of https://codereview.chromium.org/2181163003/ )

Reason for revert:
Suspect (not the first) for widespread NaCl test failures since
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...

Errors like

[4224:3200:0729/162247:ERROR:nacl_process_host.cc(337)] NaCl process exited with
status -1073741674 (0xc0000096)
...
  Actual: "Plugin crashed. 'NaCl module crashed'"

Wasn't r408782 or r408787. Running out of candidates..

Original issue's description:
> [Presentation API] Convert presentation.mojom to new wrapper types.
>
> This converts presentation.mojom to use the new (std::) Mojo wrapper types:
> mojo::String => std::string
> mojo::Array => std::vector
>
> No behavior changes should ensue.
>
> Optional values are represented as base::Optional<Foo> with the new wrapper
> types requiring some additional changes.
>
> This work will simplify further changes for crbug.com/627655.
>
> BUG=627655
>
> Committed: https://crrev.com/59e7b40948030815a49af887b922b370ba048b8b
> Cr-Commit-Position: refs/heads/master@{#408781}

TBR=dcheng@chromium.org,imcheng@chromium.org,mfoltz@chromium.org
NOTRY=true
BUG=627655

Committed: https://crrev.com/43c746be32b2bfce5c3fd654894e4b7f7bf6387f
Cr-Commit-Position: refs/heads/master@{#408905}
==========

[commit-bot: I haz the power, 2016-08-01 03:33:31]

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

[tapted, 2016-08-01 04:50:32]

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

The reason for reverting is: Didn't work. NaCl still crashing. Sadness :(.