Fix failing the plugin test: plugins.PluginsTest.testDisableEnableAllPlugins

Fix failing the plugin test: plugins.PluginsTest.testDisableEnableAllPlugins

BUG=87811
Test=plugins.PluginsTest.testDisableEnableAllPlugins

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

[Venkat Yellapu, 2011-08-26 01:01:44]

This fixes:
1. AssertionError
self.assertFalse(self._GetPluginPID(plugin_name=plugin_name))
2. quicktime error : There is a mismatch in QuickTime plugin name. Just fixed
it.

Please note, now installed Realplayer plugin on the BOTs, and this plugin also
triggers permission inforbar. Hence included this plugin in the authorization
list.

[Nirnimesh, 2011-08-26 01:17:05]

http://codereview.chromium.org/7745041/diff/1/functional/PYAUTO_TESTS
File functional/PYAUTO_TESTS (left):

http://codereview.chromium.org/7745041/diff/1/functional/PYAUTO_TESTS#oldcode153
functional/PYAUTO_TESTS:153:
'-passwords.PasswordTest.testDisplayAndSavePasswordInfobar',
??

http://codereview.chromium.org/7745041/diff/1/functional/PYAUTO_TESTS#oldcode362
functional/PYAUTO_TESTS:362:
'-passwords.PasswordTest.testDisplayAndSavePasswordInfobar',
??

Why is this here?

http://codereview.chromium.org/7745041/diff/1/functional/plugins.py
File functional/plugins.py (right):

http://codereview.chromium.org/7745041/diff/1/functional/plugins.py#newcode151
functional/plugins.py:151: self.WaitUntil(
wrap inside assertTrue

[Venkat Yellapu, 2011-08-26 15:10:26]

Reflected the changes.

Thanks.

http://codereview.chromium.org/7745041/diff/1/functional/PYAUTO_TESTS
File functional/PYAUTO_TESTS (left):

http://codereview.chromium.org/7745041/diff/1/functional/PYAUTO_TESTS#oldcode153
functional/PYAUTO_TESTS:153:
'-passwords.PasswordTest.testDisplayAndSavePasswordInfobar',
On 2011/08/26 01:17:05, Nirnimesh wrote:
> ??

Done.

http://codereview.chromium.org/7745041/diff/1/functional/PYAUTO_TESTS#oldcode362
functional/PYAUTO_TESTS:362:
'-passwords.PasswordTest.testDisplayAndSavePasswordInfobar',
On 2011/08/26 01:17:05, Nirnimesh wrote:
> ??
> 
> Why is this here?
Bcoz of the patch http://codereview.chromium.org/7744035/
Anyhow, I get it back.

http://codereview.chromium.org/7745041/diff/1/functional/plugins.py
File functional/plugins.py (right):

http://codereview.chromium.org/7745041/diff/1/functional/plugins.py#newcode151
functional/plugins.py:151: self.WaitUntil(
On 2011/08/26 01:17:05, Nirnimesh wrote:
> wrap inside assertTrue

Done.

[Nirnimesh, 2011-08-27 08:56:12]

LGTM.

Please watch the bots after you commit.

[commit-bot: I haz the power, 2011-08-30 02:30:36]

Change committed as 98761