Remove unnecessary DEPS rules since by default top level directories can't include content or chrom…

Remove unnecessary DEPS rules since by default top level directories can't include content or chrome.

R=joi@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=226754

[Jói, 2013-10-03 00:24:28]

LGTM

On Wed, Oct 2, 2013 at 11:09 PM,  <jam@chromium.org> wrote:
> Reviewers: Jói,
>
> Description:
> Remove unnecessary DEPS rules since by default top level directories can't
> include content or chrome.
>
> Please review this at https://codereview.chromium.org/25727006/
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files (+1, -4 lines):
>   M     components/DEPS
>
>
> Index: components/DEPS
> ===================================================================
> --- components/DEPS     (revision 226603)
> +++ components/DEPS     (working copy)
> @@ -1,7 +1,5 @@
>  include_rules = [
> -  # Do not add chrome/ as an allowed include.  Components MUST NOT
> -  # depend on anything under src/chrome.
> -  "-chrome",
> +  # Do not add chrome/ as an allowed include.
>
>    # Components should only depend on the public Content API, and on
>    # layers below the Content Module. They must not depend on the
> @@ -11,7 +9,6 @@
>    # the additional parts of the Content API that they need,
>    # e.g. components/component_name/browser/DEPS would add a
>    # "+content/public/browser" rule.
> -  "-content",
>    "+content/public/common",
>
>    # Dependencies of variations component.
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-10-03 00:38:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jam@chromium.org/25727006/1

[commit-bot: I haz the power, 2013-10-03 05:07:42]

Retried try job too often on win_rel for step(s) browser_tests,
chrome_frame_net_tests, chrome_frame_tests, chrome_frame_unittests,
content_browsertests, mini_installer_test, nacl_integration,
sync_integration_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...