Same comments as previous CL. Looks good otherwise. On 2011/08/07 21:19:25, juantrujillo wrote:
http://codereview.appspot.com/4855043/diff/5001/android/src/java/org/openqa/s... File android/src/java/org/openqa/selenium/android/AndroidTouchScreen.java (right): http://codereview.appspot.com/4855043/diff/5001/android/src/java/org/openqa/s... android/src/java/org/openqa/selenium/android/AndroidTouchScreen.java:41: public AndroidTouchScreen(ActivityController controller, AndroidDriver driver) { Please update your CLs according to the changes we made yesterday. For instance this should not take the controller as a constructor parameter.
LGTM , submitted in r13452 On Thu, Aug 18, 2011 at 4:54 PM, <juantrujillo@google.com> wrote: > http://codereview.appspot.com/**4855043/<http://codereview.appspot.com/4855043/> > -- Dounia