//device/media_transfer_protocol depends on dbus

//device/media_transfer_protocol depends on dbus

The build should fail early if //device/media_transfer_protocol is included with dbus explicitly disabled.

BUG=632297
Committed: https://crrev.com/e5c9431b0a1d8714ecf2fe2659f740e5d2d61201
Cr-Commit-Position: refs/heads/master@{#418744}

[Mostyn Bramley-Moore, 2016-09-12 10:24:07]

The CQ bit was checked by mostynb@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-12 10:24:21]

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

[Mostyn Bramley-Moore, 2016-09-12 10:25:27]

Description was changed from

==========
//device/media_transfer_protocol depends on dbus
==========

to

==========
//device/media_transfer_protocol depends on dbus

The build should fail early if //device/media_transfer_protocol is included with
dbus explicitly disabled.
==========

[commit-bot: I haz the power, 2016-09-12 11:18:15]

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

[commit-bot: I haz the power, 2016-09-12 11:18:16]

Dry run: This issue passed the CQ dry run.

[Mostyn Bramley-Moore, 2016-09-12 12:28:13]

mostynb@opera.com changed reviewers:
+ brettw@chromium.org, mbjorge@chromium.org

[Mostyn Bramley-Moore, 2016-09-12 12:28:14]

@brettw: please take a look.

@mbjorge: is this OK for downstream chromecast builds?

[Mostyn Bramley-Moore, 2016-09-13 07:29:01]

Description was changed from

==========
//device/media_transfer_protocol depends on dbus

The build should fail early if //device/media_transfer_protocol is included with
dbus explicitly disabled.
==========

to

==========
//device/media_transfer_protocol depends on dbus

The build should fail early if //device/media_transfer_protocol is included with
dbus explicitly disabled.

BUG=632297
==========

[mbjorge, 2016-09-13 20:59:30]

On 2016/09/12 at 12:28:14, mostynb wrote:

> @mbjorge: is this OK for downstream chromecast builds?

lgtm. (Patched this is an did a desktop build and a device build. Also did `gn
gen` for 5 other variants, all worked fine.)

[mbjorge, 2016-09-13 20:59:54]

On 2016/09/13 at 20:59:30, mbjorge wrote:
> On 2016/09/12 at 12:28:14, mostynb wrote:
> 
> > @mbjorge: is this OK for downstream chromecast builds?
> 
> lgtm. (Patched this is an did a desktop build and a device build. Also did `gn
gen` for 5 other variants, all worked fine.)

Thanks for checking :)

[Mostyn Bramley-Moore, 2016-09-13 22:44:29]

np :)

@brettw: *ping* (not sure which IM to ping you on, I can't see you on
freenode...)

[brettw, 2016-09-14 22:46:28]

lgtm

[Mostyn Bramley-Moore, 2016-09-14 22:54:46]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2016-09-14 22:55:23]

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-09-15 01:22:52]

Description was changed from

==========
//device/media_transfer_protocol depends on dbus

The build should fail early if //device/media_transfer_protocol is included with
dbus explicitly disabled.

BUG=632297
==========

to

==========
//device/media_transfer_protocol depends on dbus

The build should fail early if //device/media_transfer_protocol is included with
dbus explicitly disabled.

BUG=632297
==========

[commit-bot: I haz the power, 2016-09-15 01:22:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-15 01:24:29]

Description was changed from

==========
//device/media_transfer_protocol depends on dbus

The build should fail early if //device/media_transfer_protocol is included with
dbus explicitly disabled.

BUG=632297
==========

to

==========
//device/media_transfer_protocol depends on dbus

The build should fail early if //device/media_transfer_protocol is included with
dbus explicitly disabled.

BUG=632297

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

[commit-bot: I haz the power, 2016-09-15 01:24:30]

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