Automated chrome driver tests for about:plugins functionality tests

functional/PYAUTO_TESTS

Index: functional/PYAUTO_TESTS
===================================================================
--- functional/PYAUTO_TESTS	(revision 119922)
+++ functional/PYAUTO_TESTS	(working copy)
@@ -34,6 +34,7 @@
   'CONTINUOUS': {
     'all': [
       'autofill',
+      'about_plugins_ui.AboutPluginsTest.testAboutPluginDetailInfo',
       'bookmark_bar',
       'bookmarks',
       'browser',
@@ -289,6 +290,8 @@
       '-policy_prefs_ui',
       # No Protector on ChromeOS.
       '-protector',
+      # No Web Driver test for ChromeOS.
+      '-aboutplugins_ui',

[Nirnimesh, 2012/02/03 09:05:31]

this is wrong and has caused pyauto tests to fail on chromeos

It should be: about_plugins_ui

[Nirnimesh, 2012/02/03 09:31:40]

You've added only one test in line 37. Why attempt to exclude all here?

[Nirnimesh, 2012/02/03 09:31:40]

I've fixed it in http://src.chromium.org/viewvc/chrome?view=rev&revision=120303

 
       # ==================================================
       # Disabled tests that need to be investigated/fixed.
@@ -432,6 +435,7 @@
       'media.media_basic_playback',
       'nacl_sdk',
       'translate',
+      'about_plugins_ui',

[Nirnimesh, 2012/02/03 09:31:40]

FULL = CONTINUOUS + extras

You've already specified one test in CONTINUOUS. So now FULL suite will run
testAboutPluginDetailInfo twice!! Not what you want.


It's better to: include 'about_plugins_ui' in CONTINUOUS, and specifically
disable that one test there, and re-add it in FULL

But why did you pick PDF as a part of this UI test?

 
       # Remove when r119687 reaches official builds.  Revisit 01/31/2012
       '-special_tabs.SpecialTabsTest.testSpecialAcceratorTabs',
@@ -554,6 +558,8 @@
       # ===========================
       # No NaCl support on ChromeOS.
       '-nacl_sdk',
+      # No Web Driver test for ChromeOS.
+      '-aboutplugins_ui',

[Nirnimesh, 2012/02/03 09:31:40]

FWIW, webdriver works on chromeos now. So this should not be necessary.

 
       # ==================================================
       # Disabled tests that need to be investigated/fixed.