Re-land 107645 with static initializers removed.

Re-land 107645 with static initializer removed.
Don't inherit from content::RenderViewObserver, instead just
create it on the fly when needed in tests.
BUG=69370, 66371, 63022
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107792

[kkania, 2011-10-28 00:22:59]

patch 1 is just a revert of the revert
#2 is the changes to remove the static initializer

Checked before and after, and static initializer count stays the same.

[Ryan Sleevi, 2011-10-28 00:38:03]

I'm not qualified for anything more than a rubberstamp - as I don't know what
the change actually does. In the past I've seen these minor tweaks landed with a
TBR of the original reviewer, if they truly are trivial. Otherwise, you'll
probably want to kick it to the original reviewers.

[kkania, 2011-10-28 01:47:52]

jamesr: please review the few changes in the automation renderer. The previous
CL was reverted because it increased static initializers. Inheriting from
content::RenderViewObserver<T> brought in an extra static LazyInstance. So in
this CL I just cut that out since it wasn't really necessary.

[jamesr, 2011-10-28 01:49:43]

So the delta between patch set #1 and #2 is the delta between what originally
landed and what you want to land?

[kkania, 2011-10-28 01:52:08]

On 2011/10/28 01:49:43, jamesr wrote:
> So the delta between patch set #1 and #2 is the delta between what originally
> landed and what you want to land?

Correct

[jamesr, 2011-10-28 01:54:16]

OK, in that case lgtm

[commit-bot: I haz the power, 2011-10-28 19:39:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kkania@chromium.org/8416024/2003

[commit-bot: I haz the power, 2011-10-28 21:09:08]

Change committed as 107792