ericmckean / chromedriver

Automatically exported from code.google.com/p/chromedriver
0 stars 0 forks source link

GoBack command times out on all platforms #980

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
This is causing testGoBackAndGoForward to fail on all platforms. Here are the 
first failed builds for each platform:

http://build.chromium.org/p/chromium.chromedriver/builders/Win7/builds/7434
http://build.chromium.org/p/chromium.chromedriver/builders/Linux/builds/9314
http://build.chromium.org/p/chromium.chromedriver/builders/Linux32/builds/10848

The test was disabled on Mac due to issue 304, which has already been fixed, so 
it should be reenabled once this regression is fixed.

Original issue reported on code.google.com by samu...@chromium.org on 25 Nov 2014 at 10:46

GoogleCodeExporter commented 9 years ago
culprit cl - https://codereview.chromium.org/744653002

Original comment by samu...@chromium.org on 25 Nov 2014 at 11:23

GoogleCodeExporter commented 9 years ago
The issue is that DevTools no longer returns a response for the 
Runtime.evaluate("window.history.back();") command (as of r305619, see culprit 
cl in #1). This is because the RenderViewHost is replaced after the navigation 
event, and the response message from this RenderViewHost is no longer being 
passed back to ChromeDriver.

Original comment by samu...@chromium.org on 3 Dec 2014 at 12:47

GoogleCodeExporter commented 9 years ago
Fixed in crrev.com/780463003 (except on chromedriver_webview_shell...)

Original comment by samu...@chromium.org on 4 Dec 2014 at 9:33

GoogleCodeExporter commented 9 years ago

Original comment by samu...@chromium.org on 8 Dec 2014 at 7:13

GoogleCodeExporter commented 9 years ago
This is now fixed on WebView as well.

Original comment by samu...@chromium.org on 8 Dec 2014 at 7:14

GoogleCodeExporter commented 9 years ago

Original comment by samu...@chromium.org on 12 Dec 2014 at 10:03

GoogleCodeExporter commented 9 years ago

Original comment by samu...@chromium.org on 21 Feb 2015 at 12:25