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

Unified Diff: chrome/utility/media_galleries/image_metadata_extractor_unittest.cc

Issue 1803883002: Reland: Media Galleries Partial Deprecation: Remove image metadata & libexif. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove a remnant on MacOSX official build 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: chrome/utility/media_galleries/image_metadata_extractor_unittest.cc
diff --git a/chrome/utility/media_galleries/image_metadata_extractor_unittest.cc b/chrome/utility/media_galleries/image_metadata_extractor_unittest.cc
deleted file mode 100644
index 06206bb3aaed39c1ff8238379207633209032396..0000000000000000000000000000000000000000
--- a/chrome/utility/media_galleries/image_metadata_extractor_unittest.cc
+++ /dev/null
@@ -1,76 +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 "base/bind.h"
-#include "base/files/file_path.h"
-#include "base/path_service.h"
-#include "base/run_loop.h"
-#include "chrome/common/chrome_paths.h"
-#include "chrome/utility/media_galleries/image_metadata_extractor.h"
-#include "media/filters/file_data_source.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace metadata {
-
-void QuitLoop(base::RunLoop* loop, bool* output, bool success) {
- loop->Quit();
- *output = success;
-}
-
-base::FilePath GetTestDataFilePath(const std::string& filename) {
- base::FilePath path;
- EXPECT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &path));
- path = path.AppendASCII("extensions").AppendASCII("api_test")
- .AppendASCII("wallpaper").AppendASCII(filename);
- return path;
-}
-
-scoped_ptr<ImageMetadataExtractor> GetExtractor(
- const std::string& filename,
- bool expected_result) {
- EXPECT_TRUE(ImageMetadataExtractor::InitializeLibraryForTesting());
-
- media::FileDataSource source;
- base::FilePath test_path;
-
- EXPECT_TRUE(source.Initialize(GetTestDataFilePath(filename)));
-
- scoped_ptr<ImageMetadataExtractor> extractor(new ImageMetadataExtractor);
-
- base::RunLoop loop;
- bool extracted = false;
- extractor->Extract(&source, base::Bind(&QuitLoop, &loop, &extracted));
- EXPECT_EQ(expected_result, extracted);
-
- return extractor;
-}
-
-TEST(ImageMetadataExtractorTest, JPGFile) {
- scoped_ptr<ImageMetadataExtractor> extractor =
- GetExtractor("test.jpg", true);
-
- EXPECT_EQ(5616, extractor->width());
- EXPECT_EQ(3744, extractor->height());
- EXPECT_EQ(0, extractor->rotation());
- EXPECT_EQ(300.0, extractor->x_resolution());
- EXPECT_EQ(300.0, extractor->y_resolution());
- EXPECT_EQ("2012:03:01 17:06:07", extractor->date());
- EXPECT_EQ("Canon", extractor->camera_make());
- EXPECT_EQ("Canon EOS 5D Mark II", extractor->camera_model());
- EXPECT_EQ(0.01, extractor->exposure_time_sec());
- EXPECT_FALSE(extractor->flash_fired());
- EXPECT_EQ(3.2, extractor->f_number());
- EXPECT_EQ(100, extractor->focal_length_mm());
- EXPECT_EQ(1600, extractor->iso_equivalent());
-}
-
-TEST(ImageMetadataExtractorTest, PNGFile) {
- GetExtractor("test.png", false);
-}
-
-TEST(ImageMetadataExtractorTest, NonImageFile) {
- GetExtractor("test.js", false);
-}
-
-} // namespace metadata
« no previous file with comments | « chrome/utility/media_galleries/image_metadata_extractor.cc ('k') | chrome/utility/media_galleries/media_metadata_parser.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698