Tests in browser_tests target should use ChromeRenderViewTest as the base class.

Tests in browser_tests target should use ChromeRenderViewTest as the base class.

Currently these tests are utilizing the content::RenderViewTest as the base class
whereas all other *_browsertest.cc are using ChromeRenderViewTest as the base class.

Due to this, the migration of inline CSS to GRD being done in [1] is failing
for browsertests target as listed here [2] in the build log.

In order to fix this, these tests should use the ChromeRenderViewTest as their base class.
In doing so, the ChromeRenderViewTest loads ChromeContentClient which in turn loads the correct
pak file i.e. resources.pak required for the migration.

This CL adds the extensions/DEPS rule to allow the inclusion of chrome_render_view_test.h.

[1] https://codereview.chromium.org/573553002
[2] http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/14184

Committed: https://crrev.com/d155cb80c7b61e65f3999df1718c3be6e3cce0b4
Cr-Commit-Position: refs/heads/master@{#309424}

[vivekg, 2014-12-19 04:44:26]

vivekg@chromium.org changed reviewers:
+ benwells@chromium.org, jochen@chromium.org, vivekg@chromium.org

[vivekg, 2014-12-19 04:44:27]

PTAL, thanks!

@jochen: chrome/*
@benwells: extensions/*

[benwells, 2014-12-19 05:46:48]

benwells@chromium.org changed reviewers:
+ yoz@chromium.org

[benwells, 2014-12-19 05:46:49]

-me, +yoz

[Yoyo Zhou, 2014-12-19 23:08:38]

LGTM

https://codereview.chromium.org/818563003/diff/1/extensions/DEPS
File extensions/DEPS (right):

https://codereview.chromium.org/818563003/diff/1/extensions/DEPS#newcode51
extensions/DEPS:51: "+chrome/test/base/chrome_render_view_test.h",
I don't like this change, but we can reassess whether this test needs to be in
chrome/ instead of extensions/ later.

[vivekg, 2014-12-22 05:25:24]

@jochen, wdyt?

[jochen (gone - plz use gerrit), 2014-12-22 14:19:07]

lgtm

[vivekg, 2014-12-22 14:22:32]

The CQ bit was checked by vivekg@chromium.org

[commit-bot: I haz the power, 2014-12-22 14:23:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/818563003/1

[commit-bot: I haz the power, 2014-12-22 14:37:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-12-22 14:39:09]

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