Fix except clauses with multiple browser names. The scripts parse the list on...

tests/selenium/sample_list.txt

 #
 # Test Requirements:
 #
 #   Each sample is expected to have a global variable called g_finished
 #   that is set to true when the sample has finish initalizing and is ready
 #   for a screenshot.
 #
 # Line Format:
 #
 #   TestType test-name options
@@ skipping 31 matching lines @@
 #
 #   pdiff_threshold_mac(number_of_pixels_allowed)
 #   pdiff_threshold_win(number_of_pixels_allowed)
 #   pdiff_threshold_linux(number_of_pixels_allowed)
 #   pdiff_threshold(number_of_pixels_allowed) : Number of pixels
 #       allowed to be perceptually different before the test fails.
 #       "pdiff_threshold" specifies the threshold for all
 #       platforms.  Platform specific versions override the
 #       threshold for that platform.  The default threshold is 10 pixels.
 #
-#   except(*firefox, *iexplore, *googlechrome) : Name of the browser
-#       environment(s) where the test should be skipped.
+#   except(*firefox,*iexplore,*googlechrome) : Name of the browser
+#       environment(s) where the test should be skipped. This list should
+#       not have spaces between the browser names.
 #       Default = ""
 #
 #   colorfactor(0 to 1) : Determines how important color is for comparison. A
 #       value of 1 means they must be exactly the same; a value of 0 means it
 #       won't factor in at all.
 #
 #   downsample(number_of_times_to_halve_resolution) : The resolution will be
 #       cut in half this number of times before comparing. It is useful for
 #       eliminating aliasing artifacts since such differences are less
 #       noticeable at the lower resolution. Be careful with using this parameter
 #       as it may mask other problems.
 #
 #   pdiff_edge_ignore_off : Turn off edge detection function in pdiff.
 #       By default, it's on with a default edge detect threshold 5.
 #
 #   pdiff_edge_threshold(0 to 7) : Set edge detect threshold in pdiff.
 #       This threshold indicates the max number of neighbors the pixel can
 #       have to be considered as edge. 0 means only ignore scattered pixels.
 # NOTE! ----------------------------------------------------------------------
 #
 # Read the sample guidelines
 # http://wiki.corp.google.com/twiki/bin/view/Main/ClientThreeDSampleGuidelines
 #
 #
 medium  2d                                screenshot timeout(30000) pdiff_threshold(200) pdiff_threshold_mac(41200) colorfactor(0.8) downsample(1)
 medium  animation
 large   animated-scene                    screenshot timeout(55000) pdiff_threshold(200)
-large   beachdemo/beachdemo               screenshot timeout(120000) pdiff_threshold(200) pdiff_threshold_mac(2100) downsample(1) except(*iexplore, *googlechrome)
+large   beachdemo/beachdemo               screenshot timeout(120000) pdiff_threshold(200) pdiff_threshold_mac(2100) downsample(1) except(*iexplore,*googlechrome)
 medium  billboards                        screenshot pdiff_threshold(200)
 medium  bitmap-draw-image                 screenshot pdiff_threshold(200)
 medium  canvas                            screenshot pdiff_threshold(200) pdiff_threshold_mac(14600)
 medium  canvas-fonts                      screenshot pdiff_threshold(200) pdiff_threshold_mac(21900)
 medium  canvas-texturedraw
 medium  checkers                          screenshot pdiff_threshold(200)
 medium  convolution                       screenshot pdiff_threshold(200)
 medium  culling                           screenshot(9) pdiff_threshold(450) downsample(1)
 medium  debugging                         screenshot pdiff_threshold(200) pdiff_threshold_mac(3000)
 medium  displayfps
@@ skipping 31 matching lines @@
 medium  GoogleIO-2009/step09ex            screenshot pdiff_threshold(200)
 large   GoogleIO-2009/step14ex            screenshot pdiff_threshold(200) timeout(45000) downsample(1)
 
 # -- tests below this line are tests for which there is a python
 #    function to custom run the test. As such, only the 'except' and
 #    pdiff_threshold options have any meaning
 
 small   TestSampleErrorTextureSmall       pdiff_threshold(200) screenshots(5)
 small   TestSampleHelloCube_TexturesSmall pdiff_threshold(450) screenshot
 # IE and chrome hang on this test.
-small   TestSampleRefreshPageLoad_Small   except(*iexplore, *googlechrome)
+small   TestSampleRefreshPageLoad_Small   except(*iexplore,*googlechrome)
 medium  TestSampleCustomCamera            pdiff_threshold(200) pdiff_threshold_win(200) screenshot run_time(180000)
 medium  TestSamplePicking                 run_time(60000)
 medium  TestSampleRenderMode              run_time(60000)
 medium  TestSampleRotateModel             pdiff_threshold(200) screenshots(2) run_time(100000)
 medium  TestSampleShader_Test             pdiff_threshold(200) pdiff_threshold_win(200) screenshots(13)
 # IE takes several minutes just loading this test.
 large   TestSampleMultipleClientsLarge    run_time(180000) except(*iexplore)
 large   TestSamplePingPongLarge
 # This test currently fails on IE as it considers localhost: to be a trusted
 # domain.
 # Do not run this test until get rid of scons.  It assumes a particular
 # directory structure to find an asset.  Need to change to reflect new gyp
 # directory structure.  Should be changed when scons is gone.
 #small   TestLoadTextureFromFileSmall      except(*iexplore)