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

Unified Diff: chrome/browser/component_updater/component_installers_unittest.cc

Issue 10702188: Fix unit test to allow repeated successful runs by avoiding a function static variable in tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add DEPS file Created 8 years, 5 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 | « no previous file | chrome/browser/component_updater/component_updater_service_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/component_updater/component_installers_unittest.cc
diff --git a/chrome/browser/component_updater/component_installers_unittest.cc b/chrome/browser/component_updater/component_installers_unittest.cc
deleted file mode 100644
index 1b11497865540f0fd4603635c9212113e1adc02e..0000000000000000000000000000000000000000
--- a/chrome/browser/component_updater/component_installers_unittest.cc
+++ /dev/null
@@ -1,67 +0,0 @@
-// Copyright (c) 2012 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 "chrome/browser/component_updater/flash_component_installer.h"
-
-#include "base/file_path.h"
-#include "base/file_util.h"
-#include "base/json/json_file_value_serializer.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/message_loop.h"
-#include "base/path_service.h"
-#include "base/version.h"
-#include "build/build_config.h"
-#include "chrome/common/chrome_paths.h"
-#include "content/public/test/test_browser_thread.h"
-
-#include "testing/gtest/include/gtest/gtest.h"
-
-using content::BrowserThread;
-
-namespace {
-// File name of the Pepper Flash plugin on different platforms.
-const FilePath::CharType kDataPath[] =
-#if defined(OS_MACOSX)
- FILE_PATH_LITERAL("components/flapper/mac");
-#elif defined(OS_WIN)
- FILE_PATH_LITERAL("components\\flapper\\windows");
-#else // OS_LINUX, etc.
-#if defined(ARCH_CPU_X86)
- FILE_PATH_LITERAL("components/flapper/linux");
-#elif defined(ARCH_CPU_X86_64)
- FILE_PATH_LITERAL("components/flapper/linux_x64");
-#else
- FILE_PATH_LITERAL("components/flapper/NONEXISTENT");
-#endif
-#endif
-}
-
-// TODO(viettrungluu): Separate out into two separate tests; use a test fixture.
-TEST(ComponentInstallerTest, PepperFlashCheck) {
- MessageLoop message_loop;
- content::TestBrowserThread ui_thread(BrowserThread::UI, &message_loop);
-
- // The test directory is chrome/test/data/components/flapper.
- FilePath manifest;
- PathService::Get(chrome::DIR_TEST_DATA, &manifest);
- manifest = manifest.Append(kDataPath);
- manifest = manifest.AppendASCII("manifest.json");
-
- if (!file_util::PathExists(manifest)) {
- LOG(WARNING) << "No test manifest available. Skipping.";
- return;
- }
-
- JSONFileValueSerializer serializer(manifest);
- std::string error;
- scoped_ptr<base::Value> root(serializer.Deserialize(NULL, &error));
- ASSERT_TRUE(root.get() != NULL);
- ASSERT_TRUE(root->IsType(base::Value::TYPE_DICTIONARY));
-
- // This checks that the whole manifest is compatible.
- Version version;
- EXPECT_TRUE(CheckPepperFlashManifest(
- static_cast<base::DictionaryValue*>(root.get()), &version));
- EXPECT_TRUE(version.IsValid());
-}
« no previous file with comments | « no previous file | chrome/browser/component_updater/component_updater_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698