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

Issue 151046: Remove cache_uittest.cc.... (Closed)

Created:
11 years, 5 months ago by eroman
Modified:
9 years, 6 months ago
Reviewers:
huanr
CC:
chromium-reviews_googlegroups.com, darin (slow to review), willchan no longer on Chromium
Visibility:
Public.

Description

Remove cache_uittest.cc. This test is flaky, and doesn't add anything on top of what the net/http/http_network_transaciton_unittest.cc or net/http/http_cache_unittest.cc unit-tests already check for. BUG=6491 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=19572

Patch Set 1 #

Patch Set 2 : Add purify exception list to CL #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -187 lines) Patch
M chrome/chrome.gyp View 2 chunks +0 lines, -3 lines 0 comments Download
D chrome/common/net/cache_uitest.cc View 1 chunk +0 lines, -179 lines 0 comments Download
M chrome/test/data/purify/ui_tests.gtest.txt View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
eroman
11 years, 5 months ago (2009-06-29 22:12:50 UTC) #1
huanr
11 years, 5 months ago (2009-06-29 22:24:58 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698