Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(221)

Issue 11486044: upgrade selenium/webdriver (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 7 months ago by felix8a
Modified:
12 years, 7 months ago
Reviewers:
kpreid2
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

this is a build from selenium trunk, because it includes a fix that allows js execution to work in firefox again. how to rebuild: git clone https://code.google.com/p/selenium/ cd selenium git checkout bb3450e3ab26 ./go selenium-server-standalone

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -4 lines) Patch
M build.xml View 1 chunk +1 line, -1 line 0 comments Download
M third_party/java/webdriver/README View 1 chunk +7 lines, -1 line 0 comments Download
M third_party/java/webdriver/VERSION View 1 chunk +2 lines, -2 lines 1 comment Download
D third_party/java/webdriver/selenium-server-standalone-2.33.0.jar View Binary file 0 comments Download
A third_party/java/webdriver/selenium-standalone.jar View Binary file 0 comments Download
third_party/java/webdriver/selenium-standalone.jar View 0 chunks +1 line, -0 lines 0 comments Download

Messages

Total messages: 3
felix8a
12 years, 7 months ago (2013-07-23 09:36:22 UTC) #1
kpreid2
LGTM https://codereview.appspot.com/11486044/diff/1/third_party/java/webdriver/VERSION File third_party/java/webdriver/VERSION (right): https://codereview.appspot.com/11486044/diff/1/third_party/java/webdriver/VERSION#newcode1 third_party/java/webdriver/VERSION:1: Version: bb3450e3ab26 (2013-07-22 trunk after 2.33.0) Maybe use ...
12 years, 7 months ago (2013-07-23 20:31:11 UTC) #2
felix8a
12 years, 7 months ago (2013-07-23 23:19:23 UTC) #3
@r5500

On 2013/07/23 20:31:11, kpreid2 wrote:
> LGTM
> 
>
https://codereview.appspot.com/11486044/diff/1/third_party/java/webdriver/VER...
> File third_party/java/webdriver/VERSION (right):
> 
>
https://codereview.appspot.com/11486044/diff/1/third_party/java/webdriver/VER...
> third_party/java/webdriver/VERSION:1: Version: bb3450e3ab26 (2013-07-22 trunk
> after 2.33.0)
> Maybe use `git describe`'s answer here, as a standard format? In this case it
> is:
> 
>     selenium-2.29.0a-830-gbb3450e

yeah, that's slightly odd and maybe misleading, 2.29.0a was a long time ago.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b