components: Remove 'ui' dependencies from json_schema and user_prefs targets.

components: Remove 'ui' dependencies from json_schema and user_prefs targets.

BUG=302505
TEST=None
TBR=joi@chromium.org

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

[tfarina, 2013-12-14 20:05:37]

TBRing...

[commit-bot: I haz the power, 2013-12-14 20:06:10]

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

[commit-bot: I haz the power, 2013-12-14 21:02:45]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-12-15 00:20:36]

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

[commit-bot: I haz the power, 2013-12-15 00:51:56]

Retried try job too often on linux_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[Jói, 2013-12-16 12:28:12]

LGTM

On Sat, Dec 14, 2013 at 8:05 PM,  <tfarina@chromium.org> wrote:
> Reviewers: Jói,
>
> Message:
> TBRing...
>
> Description:
> components: Remove 'ui' dependencies from json_schema and user_prefs
> targets.
>
> BUG=302505
> TEST=None
> TBR=joi@chromium.org
>
> Please review this at https://codereview.chromium.org/106553004/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files (+0, -4 lines):
>   M components/json_schema.gypi
>   M components/json_schema/json_schema_validator.cc
>   M components/user_prefs.gypi
>
>
> Index: components/json_schema.gypi
> diff --git a/components/json_schema.gypi b/components/json_schema.gypi
> index
>
7218f9762c11a9fe198e132ddebc5383dfce130e..fd1675d70216303e8b729df2742ed647db9188db
> 100644
> --- a/components/json_schema.gypi
> +++ b/components/json_schema.gypi
> @@ -9,8 +9,6 @@
>        'type': 'static_library',
>        'dependencies': [
>          '../base/base.gyp:base',
> -        '../ui/gfx/gfx.gyp:gfx',
> -        '../ui/ui.gyp:ui',
>        ],
>        'include_dirs': [
>          '..',
> Index: components/json_schema/json_schema_validator.cc
> diff --git a/components/json_schema/json_schema_validator.cc
> b/components/json_schema/json_schema_validator.cc
> index
>
310975ae7ec7cc87c97b14c5f2e5fd7d760d9bbe..eadba17b06b8be6e2128015a6ae0fc066d02d0bf
> 100644
> --- a/components/json_schema/json_schema_validator.cc
> +++ b/components/json_schema/json_schema_validator.cc
> @@ -14,7 +14,6 @@
>  #include "base/strings/stringprintf.h"
>  #include "base/values.h"
>  #include "components/json_schema/json_schema_constants.h"
> -#include "ui/base/l10n/l10n_util.h"
>
>  namespace schema = json_schema_constants;
>
> Index: components/user_prefs.gypi
> diff --git a/components/user_prefs.gypi b/components/user_prefs.gypi
> index
>
5bb04c9ef0dd9c97721eae3544e6e6c4c9885d33..a556e708ce3366131aef71bfb7576215feb66e8a
> 100644
> --- a/components/user_prefs.gypi
> +++ b/components/user_prefs.gypi
> @@ -12,7 +12,6 @@
>          '../base/base.gyp:base_prefs',
>
>
'../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
>          '../content/content.gyp:content_browser',
> -        '../ui/gfx/gfx.gyp:gfx',
>          '../ui/ui.gyp:ui',
>        ],
>        'include_dirs': [
>
>

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-12-16 12:30:14]

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

[commit-bot: I haz the power, 2013-12-16 13:34:21]

Retried try job too often on linux_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-12-16 15:06:31]

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

[commit-bot: I haz the power, 2013-12-16 15:48:27]

Retried try job too often on linux_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-12-16 16:17:22]

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

[commit-bot: I haz the power, 2013-12-16 17:28:00]

Retried try job too often on linux_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-12-16 17:32:52]

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

[commit-bot: I haz the power, 2013-12-16 19:00:26]

Retried try job too often on linux_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-12-16 20:19:52]

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

[commit-bot: I haz the power, 2013-12-16 22:26:22]

Change committed as 241047