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

Unified Diff: third_party/WebKit/Source/platform/fonts/SymbolsIteratorTest.cpp

Issue 1837923003: Remove font logging in unit tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove all LogFonts Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/fonts/SymbolsIteratorTest.cpp
diff --git a/third_party/WebKit/Source/platform/fonts/SymbolsIteratorTest.cpp b/third_party/WebKit/Source/platform/fonts/SymbolsIteratorTest.cpp
index 02b6cb73b746c5e282bc1a5269ab317206988200..2c4f0ce6091072e7bf2d4a34cac9817eab73793d 100644
--- a/third_party/WebKit/Source/platform/fonts/SymbolsIteratorTest.cpp
+++ b/third_party/WebKit/Source/platform/fonts/SymbolsIteratorTest.cpp
@@ -28,13 +28,6 @@ struct ExpectedRun {
class SymbolsIteratorTest : public testing::Test {
protected:
-#if !LOG_DISABLED
- static void SetUpTestCase()
- {
- LogFonts = { WTFLogChannelOn };
- }
-#endif
-
void CheckRuns(const Vector<TestRun>& runs)
{
String text(emptyString16Bit());
@@ -59,7 +52,6 @@ protected:
ASSERT_EQ(expect[runCount].fontFallbackPriority, fontFallbackPriority);
++runCount;
}
- WTF_LOG(Fonts, "Expected %zu runs, got %lu ", expect.size(), runCount);
ASSERT_EQ(expect.size(), runCount);
}
};

Powered by Google App Engine
This is Rietveld 408576698