[Presentation API] Move presentation.mojom to content/common/presentation

[Presentation API] Move presentation.mojom to content/common/presentation

To write Mojo typemaps for presentation.mojom using types in
content/public/common, we need to move presentation.mojom to content/.

See crbug.com/632623

BUG=679075

[mark a. foltz, 2017-01-10 21:32:30]

Description was changed from

==========
[Presentation API] Move presentation.mojom to content/common/presentation

To write Mojo typemaps for presentation.mojom using types in
content/public/common, we need to move presentation.mojom to content/.

BUG=679075
==========

to

==========
[Presentation API] Move presentation.mojom to content/common/presentation

To write Mojo typemaps for presentation.mojom using types in
content/public/common, we need to move presentation.mojom to content/.

See crbug.com/632623

BUG=679075
==========

[mark a. foltz, 2017-01-10 21:32:40]

The CQ bit was checked by mfoltz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-10 21:33:24]

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

[mark a. foltz, 2017-01-10 21:34:29]

mfoltz@chromium.org changed reviewers:
+ avayvod@chromium.org, avi@chromium.org, jochen@chromium.org,
nasko@chromium.org

[mark a. foltz, 2017-01-10 21:34:29]

avayvod@ for overall review
avi@ for content/
nasko@ for ipc/SECURITY_OWNERS
jochen@ for third_party/WebKit

[Avi (use Gerrit), 2017-01-10 21:37:58]

lgtm

[commit-bot: I haz the power, 2017-01-10 21:40:41]

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

[commit-bot: I haz the power, 2017-01-10 21:40:42]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[mark a. foltz, 2017-01-10 21:46:13]

The CQ bit was checked by mfoltz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-10 21:47:10]

Dry run: 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, 2017-01-10 22:22:21]

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

[commit-bot: I haz the power, 2017-01-10 22:22:21]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[jochen (gone - plz use gerrit), 2017-01-11 09:50:22]

webkit lgtm

[nasko, 2017-01-11 18:45:05]

LGTM

[whywhat, 2017-01-11 19:56:52]

lgtm although I've no idea why the browser test is failing...

https://codereview.chromium.org/2622993002/diff/80001/content/public/common/p...
File content/public/common/presentation_session.cc (right):

https://codereview.chromium.org/2622993002/diff/80001/content/public/common/p...
content/public/common/presentation_session.cc:14:
PresentationSessionInfo::~PresentationSessionInfo() {}
nit: these seem like unnecessary changes - is it what git cl format did? (I'm
fine either way).

[mark a. foltz, 2017-01-11 23:31:06]

The CQ bit was checked by mfoltz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-11 23:32:03]

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

[mark a. foltz, 2017-01-11 23:32:31]

On 2017/01/11 at 19:56:52, avayvod wrote:
> lgtm although I've no idea why the browser test is failing...
> 
>
https://codereview.chromium.org/2622993002/diff/80001/content/public/common/p...
> File content/public/common/presentation_session.cc (right):
> 
>
https://codereview.chromium.org/2622993002/diff/80001/content/public/common/p...
> content/public/common/presentation_session.cc:14:
PresentationSessionInfo::~PresentationSessionInfo() {}
> nit: these seem like unnecessary changes - is it what git cl format did? (I'm
fine either way).

Yes, I ran git cl format and it made those changes...

[commit-bot: I haz the power, 2017-01-12 00:56:58]

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

[commit-bot: I haz the power, 2017-01-12 00:56:59]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[mark a. foltz, 2017-01-24 00:36:14]

It looks like all I need is a one-line change to content/common/DEPS to avoid
having to do this.  I updated https://codereview.chromium.org/2613153003 and am
closing this.