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

Issue 10821055: Rename renderer/autofill/autofill_browsertest.cc to be unique. (Closed)

Created:
8 years, 5 months ago by Andrew T Wilson (Slow)
Modified:
8 years, 4 months ago
Reviewers:
jam, dhollowa
CC:
chromium-reviews, dhollowa+watch_chromium.org, brettw-cc_chromium.org, Ilya Sherman, dyu1, darin-cc_chromium.org
Visibility:
Public.

Description

Rename renderer/autofill/autofill_browsertest.cc to be unique. BUG=139244 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148908

Patch Set 1 #

Patch Set 2 : Whitespace change to force autofill_browsertest recompilation. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -177 lines) Patch
M chrome/browser/autofill/autofill_browsertest.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
M chrome/chrome_tests.gypi View 1 chunk +1 line, -1 line 0 comments Download
D chrome/renderer/autofill/autofill_browsertest.cc View 1 chunk +0 lines, -176 lines 0 comments Download
A + chrome/renderer/autofill/autofill_renderer_browsertest.cc View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 12 (0 generated)
jam
lgtm seems like we should make that warning you saw an error?
8 years, 5 months ago (2012-07-27 00:21:36 UTC) #1
dhollowa
LGTM (for OWNERS). What was the error?
8 years, 5 months ago (2012-07-27 00:29:50 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/atwilson@chromium.org/10821055/1
8 years, 4 months ago (2012-07-27 15:59:41 UTC) #3
Andrew T Wilson (Slow)
On 2012/07/27 00:29:50, dhollowa wrote: > LGTM (for OWNERS). What was the error? The error ...
8 years, 4 months ago (2012-07-27 16:01:42 UTC) #4
commit-bot: I haz the power
Try job failure for 10821055-1 (retry) on win_rel for step "browser_tests". It's a second try, ...
8 years, 4 months ago (2012-07-27 18:11:14 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/atwilson@chromium.org/10821055/1
8 years, 4 months ago (2012-07-27 18:18:51 UTC) #6
commit-bot: I haz the power
Try job failure for 10821055-1 (retry) on win_rel for step "browser_tests". It's a second try, ...
8 years, 4 months ago (2012-07-27 20:59:53 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/atwilson@chromium.org/10821055/1
8 years, 4 months ago (2012-07-27 23:47:26 UTC) #8
commit-bot: I haz the power
Try job failure for 10821055-1 (retry) on win_rel for step "browser_tests". It's a second try, ...
8 years, 4 months ago (2012-07-28 01:54:41 UTC) #9
Andrew T Wilson (Slow)
Wow, ExtensionApiTest.Idle keeps failing with this patch, which is weird since I'm not touching that ...
8 years, 4 months ago (2012-07-28 02:56:06 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/atwilson@chromium.org/10821055/11004
8 years, 4 months ago (2012-07-28 21:47:19 UTC) #11
commit-bot: I haz the power
8 years, 4 months ago (2012-07-28 23:10:52 UTC) #12
Change committed as 148908

Powered by Google App Engine
This is Rietveld 408576698