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

Issue 6735061: Correcting a TypeError in popups pyauto test. (Closed)

Created:
9 years, 8 months ago by sunandt
Modified:
9 years, 7 months ago
Reviewers:
Nirnimesh
CC:
chromium-reviews
Visibility:
Public.

Description

Correcting a TypeError in popups pyauto test. BUG=70659 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=80484

Patch Set 1 #

Total comments: 2

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M functional/popups.py View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
sunandt
9 years, 8 months ago (2011-04-05 17:41:58 UTC) #1
Nirnimesh
LGTM http://codereview.chromium.org/6735061/diff/1/functional/popups.py File functional/popups.py (right): http://codereview.chromium.org/6735061/diff/1/functional/popups.py#newcode76 functional/popups.py:76: logging.debug('Attempt %d' % attempts+1) put space around +
9 years, 8 months ago (2011-04-05 17:43:02 UTC) #2
sunandt
9 years, 8 months ago (2011-04-05 17:45:24 UTC) #3
http://codereview.chromium.org/6735061/diff/1/functional/popups.py
File functional/popups.py (right):

http://codereview.chromium.org/6735061/diff/1/functional/popups.py#newcode76
functional/popups.py:76: logging.debug('Attempt %d' % attempts+1)
On 2011/04/05 17:43:02, Nirnimesh wrote:
> put space around +

Done.

Powered by Google App Engine
This is Rietveld 408576698