Remove ui_unittests -> packed_resources dependency

Remove ui_unittests -> packed_resources dependency

The dependency is no longer required.

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

[tfarina, 2014-04-28 18:23:40]

Did you check in a clobber build, android_builder_tests still works?

[Xianzhu, 2014-04-28 18:55:52]

On 2014/04/28 18:23:40, tfarina wrote:
> Did you check in a clobber build, android_builder_tests still works?

Tried locally. The build succeeded. Several test cases failed or crashed when
running ui_unittests, but the they also failed or crashed without this change.

[tfarina, 2014-04-28 20:07:11]

On 2014/04/28 18:55:52, Xianzhu wrote:
> On 2014/04/28 18:23:40, tfarina wrote:
> > Did you check in a clobber build, android_builder_tests still works?
> 
> Tried locally. The build succeeded. Several test cases failed or crashed when
> running ui_unittests, but the they also failed or crashed without this change.

Could you file a bug?

[Xianzhu, 2014-04-28 20:14:35]

On 2014/04/28 20:07:11, tfarina wrote:
> On 2014/04/28 18:55:52, Xianzhu wrote:
> > On 2014/04/28 18:23:40, tfarina wrote:
> > > Did you check in a clobber build, android_builder_tests still works?
> > 
> > Tried locally. The build succeeded. Several test cases failed or crashed
when
> > running ui_unittests, but the they also failed or crashed without this
change.
> 
> Could you file a bug?

Didn't file a bug because the test passes on bots. I guess there is some local
configuration problem is causing the failures, but I haven't got time to
investigate it.

[Xianzhu, 2014-04-28 20:44:57]

On 2014/04/28 20:14:35, Xianzhu wrote:
> On 2014/04/28 20:07:11, tfarina wrote:
> > On 2014/04/28 18:55:52, Xianzhu wrote:
> > > On 2014/04/28 18:23:40, tfarina wrote:
> > > > Did you check in a clobber build, android_builder_tests still works?
> > > 
> > > Tried locally. The build succeeded. Several test cases failed or crashed
> when
> > > running ui_unittests, but the they also failed or crashed without this
> change.
> > 
> > Could you file a bug?
> 
> Didn't file a bug because the test passes on bots. I guess there is some local
> configuration problem is causing the failures, but I haven't got time to
> investigate it.

Run ui_unittests after content_unittests, and it succeeded. The failure is
related to ICU data file. Will fix that. It is not related to this change.

[tfarina, 2014-04-28 22:18:22]

On 2014/04/28 20:44:57, Xianzhu wrote:
> On 2014/04/28 20:14:35, Xianzhu wrote:
> > On 2014/04/28 20:07:11, tfarina wrote:
> > > On 2014/04/28 18:55:52, Xianzhu wrote:
> > > > On 2014/04/28 18:23:40, tfarina wrote:
> > > > > Did you check in a clobber build, android_builder_tests still works?
> > > > 
> > > > Tried locally. The build succeeded. Several test cases failed or crashed
> > when
> > > > running ui_unittests, but the they also failed or crashed without this
> > change.
> > > 
> > > Could you file a bug?
> > 
> > Didn't file a bug because the test passes on bots. I guess there is some
local
> > configuration problem is causing the failures, but I haven't got time to
> > investigate it.
> 
> Run ui_unittests after content_unittests, and it succeeded. The failure is
> related to ICU data file. Will fix that. It is not related to this change.

Is the icu issue related to crbug.com/351052?

lgtm for what we have here, providing you have tried this locally and bots are
green.

You can put this on CQ with my approval, since build/OWNERS has *.

[Xianzhu, 2014-04-28 22:23:45]

On 2014/04/28 22:18:22, tfarina wrote:
> On 2014/04/28 20:44:57, Xianzhu wrote:
> > On 2014/04/28 20:14:35, Xianzhu wrote:
> > > On 2014/04/28 20:07:11, tfarina wrote:
> > > > On 2014/04/28 18:55:52, Xianzhu wrote:
> > > > > On 2014/04/28 18:23:40, tfarina wrote:
> > > > > > Did you check in a clobber build, android_builder_tests still works?
> > > > > 
> > > > > Tried locally. The build succeeded. Several test cases failed or
crashed
> > > when
> > > > > running ui_unittests, but the they also failed or crashed without this
> > > change.
> > > > 
> > > > Could you file a bug?
> > > 
> > > Didn't file a bug because the test passes on bots. I guess there is some
> local
> > > configuration problem is causing the failures, but I haven't got time to
> > > investigate it.
> > 
> > Run ui_unittests after content_unittests, and it succeeded. The failure is
> > related to ICU data file. Will fix that. It is not related to this change.
> 
> Is the icu issue related to crbug.com/351052?
>

No. My encountered issue will be fixed by
https://codereview.chromium.org/259123002/.

> lgtm for what we have here, providing you have tried this locally and bots are
> green.
> 
> You can put this on CQ with my approval, since build/OWNERS has *.

[Xianzhu, 2014-04-28 22:23:51]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2014-04-28 22:25:59]

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

[commit-bot: I haz the power, 2014-04-28 22:28:42]

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

[commit-bot: I haz the power, 2014-04-28 22:28:42]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_clang_dbg

[Xianzhu, 2014-04-28 22:52:33]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2014-04-28 22:53:41]

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

[commit-bot: I haz the power, 2014-04-29 07:58:24]

Change committed as 266782