[chromecast] Removes redundant DEPS rules.

[chromecast] Removes redundant DEPS rules.

BUG=665489
Committed: https://crrev.com/4941d96885a43a53b6ae84eaea345f61339bbcef
Cr-Commit-Position: refs/heads/master@{#433678}

[alokp, 2016-11-20 23:08:42]

alokp@chromium.org changed reviewers:
+ slan@chromium.org

[slan, 2016-11-21 22:08:21]

lgtm % 1 question

https://codereview.chromium.org/2517013002/diff/1/chromecast/crash/linux/DEPS
File chromecast/crash/linux/DEPS (left):

https://codereview.chromium.org/2517013002/diff/1/chromecast/crash/linux/DEPS...
chromecast/crash/linux/DEPS:2: "+chromecast/crash",
Does this work? I see some files that need includes from the parent dir. One
example:

https://cs.chromium.org/chromium/src/chromecast/crash/linux/minidump_uploader...

[alokp, 2016-11-21 22:34:23]

https://codereview.chromium.org/2517013002/diff/1/chromecast/crash/linux/DEPS
File chromecast/crash/linux/DEPS (left):

https://codereview.chromium.org/2517013002/diff/1/chromecast/crash/linux/DEPS...
chromecast/crash/linux/DEPS:2: "+chromecast/crash",
On 2016/11/21 22:08:21, slan wrote:
> Does this work? I see some files that need includes from the parent dir. One
> example:
> 
>
https://cs.chromium.org/chromium/src/chromecast/crash/linux/minidump_uploader...

You do not need to explicitly include parent directories unless the parent
directory DEPS disallows it.

[alokp, 2016-11-21 22:34:26]

The CQ bit was checked by alokp@chromium.org

[commit-bot: I haz the power, 2016-11-21 22:34:59]

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-11-21 22:46:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479767666978970, "parent_rev":
"88ed01cf201c3f6930eb1d4fd944ece5850836f6", "commit_rev":
"55759833bdfd83c98cd20334894f6909fc7fdc37"}

[commit-bot: I haz the power, 2016-11-21 22:47:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-21 22:49:18]

Description was changed from

==========
[chromecast] Removes redundant DEPS rules.

BUG=665489
==========

to

==========
[chromecast] Removes redundant DEPS rules.

BUG=665489

Committed: https://crrev.com/4941d96885a43a53b6ae84eaea345f61339bbcef
Cr-Commit-Position: refs/heads/master@{#433678}
==========

[commit-bot: I haz the power, 2016-11-21 22:49:19]

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

[slan, 2016-11-21 23:07:15]

On 2016/11/21 22:34:23, alokp wrote:
> https://codereview.chromium.org/2517013002/diff/1/chromecast/crash/linux/DEPS
> File chromecast/crash/linux/DEPS (left):
> 
>
https://codereview.chromium.org/2517013002/diff/1/chromecast/crash/linux/DEPS...
> chromecast/crash/linux/DEPS:2: "+chromecast/crash",
> On 2016/11/21 22:08:21, slan wrote:
> > Does this work? I see some files that need includes from the parent dir. One
> > example:
> > 
> >
>
https://cs.chromium.org/chromium/src/chromecast/crash/linux/minidump_uploader...
> 
> You do not need to explicitly include parent directories unless the parent
> directory DEPS disallows it.

TIL; thanks

[slan, 2016-11-21 23:07:15]

On 2016/11/21 22:34:23, alokp wrote:
> https://codereview.chromium.org/2517013002/diff/1/chromecast/crash/linux/DEPS
> File chromecast/crash/linux/DEPS (left):
> 
>
https://codereview.chromium.org/2517013002/diff/1/chromecast/crash/linux/DEPS...
> chromecast/crash/linux/DEPS:2: "+chromecast/crash",
> On 2016/11/21 22:08:21, slan wrote:
> > Does this work? I see some files that need includes from the parent dir. One
> > example:
> > 
> >
>
https://cs.chromium.org/chromium/src/chromecast/crash/linux/minidump_uploader...
> 
> You do not need to explicitly include parent directories unless the parent
> directory DEPS disallows it.

TIL; thanks