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

Unified Diff: content/common/dwrite_font_platform_win_unittest.cc

Issue 1847173002: Revert "Revert of Remove font cache code (patchset #3 id:40001 of https://codereview.chromium.org/1… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add test expectations to rebaseline tests on Win10 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
« no previous file with comments | « content/common/dwrite_font_platform_win.cc ('k') | content/common/sandbox_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/dwrite_font_platform_win_unittest.cc
diff --git a/content/common/dwrite_font_platform_win_unittest.cc b/content/common/dwrite_font_platform_win_unittest.cc
deleted file mode 100644
index 4e517feedf6a9c0413c1ab2bdc02c680920e9417..0000000000000000000000000000000000000000
--- a/content/common/dwrite_font_platform_win_unittest.cc
+++ /dev/null
@@ -1,71 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "content/public/common/dwrite_font_platform_win.h"
-
-#include "base/files/file_util.h"
-#include "base/files/scoped_temp_dir.h"
-#include "base/logging.h"
-#include "base/path_service.h"
-#include "content/public/common/content_paths.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "ui/gfx/win/direct_write.h"
-
-namespace content {
-
-class DWriteFontCacheTest : public testing::Test {
- public:
- DWriteFontCacheTest() { }
-
- void SetUp() override {
- base::FilePath data_path;
- ASSERT_TRUE(PathService::Get(content::DIR_TEST_DATA, &data_path));
- arial_cache_path_ =
- data_path.AppendASCII("font/dwrite_font_cache_arial.dat");
-
- corrupt_cache_path_ =
- data_path.AppendASCII("font/dwrite_font_cache_corrupt.dat");
-
- ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
- cache_file_path_ = temp_dir_.path().AppendASCII("dwrite_font_cache.dat");
- }
-
- protected:
- base::FilePath arial_cache_path_;
- base::FilePath corrupt_cache_path_;
- base::ScopedTempDir temp_dir_;
- base::FilePath cache_file_path_;
-};
-
-TEST_F(DWriteFontCacheTest, BuildCacheTest) {
- if (gfx::win::ShouldUseDirectWrite()) {
- DLOG(INFO) << __FUNCTION__ << ": " << cache_file_path_.value().c_str();
- EXPECT_TRUE(BuildFontCache(cache_file_path_));
- ASSERT_TRUE(base::PathExists(cache_file_path_));
- }
-}
-
-TEST_F(DWriteFontCacheTest, ValidCacheTest) {
- if (gfx::win::ShouldUseDirectWrite()) {
- DLOG(INFO) << __FUNCTION__ << ": " << arial_cache_path_.value().c_str();
- scoped_ptr<base::File> file(new base::File(arial_cache_path_,
- base::File::FLAG_OPEN | base::File::FLAG_READ));
- ASSERT_TRUE(file->IsValid());
-
- EXPECT_TRUE(ValidateFontCacheFile(file.get()));
- }
-}
-
-TEST_F(DWriteFontCacheTest, CorruptCacheTest) {
- if (gfx::win::ShouldUseDirectWrite()) {
- DLOG(INFO) << __FUNCTION__ << ": " << corrupt_cache_path_.value().c_str();
- scoped_ptr<base::File> file(new base::File(corrupt_cache_path_,
- base::File::FLAG_OPEN | base::File::FLAG_READ));
- ASSERT_TRUE(file->IsValid());
-
- EXPECT_FALSE(ValidateFontCacheFile(file.get()));
- }
-}
-
-} // content
« no previous file with comments | « content/common/dwrite_font_platform_win.cc ('k') | content/common/sandbox_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698