Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(61)

Issue 3066017: Test to verify the 'multiple downloads' infobar (Closed)

Created:
10 years, 4 months ago by Nirnimesh
Modified:
9 years, 7 months ago
Reviewers:
John Grabowski
CC:
chromium-reviews, Paul Godavari, Paweł Hajdan Jr.
Visibility:
Public.

Description

Test to verify the 'multiple downloads' infobar Verifies the multiple downloads infobar. Implemented by Venkat, who figured out how to reliably trigger the multiple downloads infobar. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=54163

Patch Set 1 #

Patch Set 2 : os.remove #

Total comments: 2

Patch Set 3 : en-us #

Total comments: 2

Patch Set 4 : jrg #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -4 lines) Patch
M chrome/test/functional/infobars.py View 1 2 3 2 chunks +6 lines, -4 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Nirnimesh
10 years, 4 months ago (2010-07-29 18:45:11 UTC) #1
John Grabowski
LGTM http://codereview.chromium.org/3066017/diff/2001/3002 File chrome/test/functional/infobars.py (right): http://codereview.chromium.org/3066017/diff/2001/3002#newcode128 chrome/test/functional/infobars.py:128: self.assertTrue(infobars, 'Expected the multiple downloads infobar') Wrap english ...
10 years, 4 months ago (2010-07-29 18:49:56 UTC) #2
Nirnimesh
http://codereview.chromium.org/3066017/diff/2001/3002 File chrome/test/functional/infobars.py (right): http://codereview.chromium.org/3066017/diff/2001/3002#newcode128 chrome/test/functional/infobars.py:128: self.assertTrue(infobars, 'Expected the multiple downloads infobar') On 2010/07/29 18:49:57, ...
10 years, 4 months ago (2010-07-29 19:14:23 UTC) #3
John Grabowski
http://codereview.chromium.org/3066017/diff/7001/8002 File chrome/test/functional/infobars.py (right): http://codereview.chromium.org/3066017/diff/7001/8002#newcode113 chrome/test/functional/infobars.py:113: assert pyauto.PyUITest.IsEnUS() What I meant was: ... if pyauto.PyUITest.IsEnUS(): ...
10 years, 4 months ago (2010-07-29 19:48:55 UTC) #4
Nirnimesh
10 years, 4 months ago (2010-07-29 20:02:21 UTC) #5
(sorry I had committed. will recommit after this fix)

http://codereview.chromium.org/3066017/diff/7001/8002
File chrome/test/functional/infobars.py (right):

http://codereview.chromium.org/3066017/diff/7001/8002#newcode113
chrome/test/functional/infobars.py:113: assert pyauto.PyUITest.IsEnUS()
On 2010/07/29 19:48:55, John Grabowski wrote:
> What I meant was:
> 
> ...
> if pyauto.PyUITest.IsEnUS():
>   self.assertEqual(match_text, infobars[0]['text'])
>   self.assertEqual('Allow', infobars[0]['buttons'][0])
>   self.assertEqual('Deny', infobars[0]['buttons'][1])
> else:
>   # log warning about part of test being disabled
> 

Done.

Powered by Google App Engine
This is Rietveld 408576698