Reviewers: kpreid2,
Description:
Chrome testing is still failing randomly due to some chromedriver
issue where it hangs forever waiting for something.
This is a refactor that tries to avoid the problem, by reinstating
code to use fresh windows in the same browser instance, but rewritten
a little more cleanly. I've been using this to run Chrome tests
continually on my laptop for a while, and it hasn't failed.
I have low confidence this will actually solve the problem, because
my previous attempt at a fix also ran fine in continual testing on
my laptop. But this was easy to do, and it does have a chance of fixing
the problem, so why not try it.
(My current guess about the failure is that it started happening
because of the forced switch to chromedriver2 because of lack of
support for chromedriver1 in the newer versions of chrome, but I'm
having trouble reproducing a failure and/or capturing info from a
failure, so this is a lot of stumbling in the dark.)
Please review this at https://codereview.appspot.com/11790043/
Affected files:
M tests/com/google/caja/lang/css/CssPropertyPatternsTest.java
M tests/com/google/caja/plugin/BrowserTestCase.java
A tests/com/google/caja/plugin/Echo.java
M tests/com/google/caja/plugin/WebDriverHandle.java
M tests/com/google/caja/util/TestFlag.java
--
---
You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
