Removes PluginInfoMessageFilter dependency on PluginGroup.

Removes PluginInfoMessageFilter dependency on PluginGroup.   


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

[Bernhard Bauer, 2012-09-20 17:09:02]

LGTM

Just so I understand, this is essentially a re-land of http://crrev.com/155529,
right?

[ibraaaa, 2012-09-20 17:13:20]

Yes.

On 2012/09/20 17:09:02, Bernhard Bauer wrote:
> LGTM
> 
> Just so I understand, this is essentially a re-land of
http://crrev.com/155529,
> right?

[commit-bot: I haz the power, 2012-09-20 17:14:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ibraaaa@google.com/10951029/1

[commit-bot: I haz the power, 2012-09-20 17:14:59]

Failed to apply patch for chrome/renderer/chrome_content_renderer_client.cc:
While running patch -p1 --forward --force;
patching file chrome/renderer/chrome_content_renderer_client.cc
Hunk #1 FAILED at 311.
Hunk #2 succeeded at 369 (offset 2 lines).
Hunk #3 succeeded at 395 (offset 2 lines).
Hunk #4 succeeded at 428 (offset 2 lines).
Hunk #5 succeeded at 483 (offset 2 lines).
Hunk #6 succeeded at 495 (offset 2 lines).
Hunk #7 succeeded at 506 (offset 2 lines).
Hunk #8 succeeded at 526 (offset 2 lines).
1 out of 8 hunks FAILED -- saving rejects to file
chrome/renderer/chrome_content_renderer_client.cc.rej

[commit-bot: I haz the power, 2012-09-20 17:30:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ibraaaa@google.com/10951029/10001

[commit-bot: I haz the power, 2012-09-20 17:30:16]

Presubmit check for 10951029-10001 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome

[ibraaaa, 2012-09-20 17:34:25]

Hi James,

Can I have an OWNERS review for this CL?

[James Hawkins, 2012-09-20 17:36:17]

lgtm

[commit-bot: I haz the power, 2012-09-20 17:38:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ibraaaa@google.com/10951029/10001

[commit-bot: I haz the power, 2012-09-20 18:44:33]

Retried try job too often for step(s) content_browsertests

[commit-bot: I haz the power, 2012-09-21 08:56:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ibraaaa@google.com/10951029/13003

[commit-bot: I haz the power, 2012-09-21 09:48:22]

Retried try job too often for step(s) crypto_unittests, unit_tests,
cacheinvalidation_unittests, remoting_unittests, jingle_unittests,
nacl_integration, ipc_tests, interactive_ui_tests, browser_tests, net_unittests,
content_unittests, gpu_unittests, base_unittests, sync_integration_tests,
media_unittests, safe_browsing_tests, content_browsertests, sql_unittests,
printing_unittests, sync_unit_tests, check_deps

[commit-bot: I haz the power, 2012-09-21 10:44:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ibraaaa@google.com/10951029/13003

[commit-bot: I haz the power, 2012-09-21 12:53:07]

Change committed as 157966