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

Unified Diff: chrome/browser/sync/glue/extension_util_unittest.cc

Issue 6852029: [Sync] Move some extension-sync-related logic to ExtensionService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address asargent's comments Created 9 years, 8 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/browser/sync/glue/extension_util_unittest.cc
diff --git a/chrome/browser/sync/glue/extension_util_unittest.cc b/chrome/browser/sync/glue/extension_util_unittest.cc
index 4df21b9f99901c6f8f0f6404cf57a8bdfd9e7757..35a40651eee29620e9c19569e0c3298cfb9d55aa 100644
--- a/chrome/browser/sync/glue/extension_util_unittest.cc
+++ b/chrome/browser/sync/glue/extension_util_unittest.cc
@@ -420,22 +420,6 @@ TEST_F(ExtensionUtilTest, GetExtensionSpecifics) {
EXPECT_EQ(kName, specifics.name());
}
-TEST_F(ExtensionUtilTest, IsExtensionOutdated) {
- FilePath file_path(kExtensionFilePath);
- scoped_refptr<Extension> extension(
- MakeSyncableExtension(kVersion2, kValidUpdateUrl1, kName, file_path));
- sync_pb::ExtensionSpecifics specifics;
- specifics.set_id(kValidId);
- specifics.set_update_url(kValidUpdateUrl1);
-
- specifics.set_version(kVersion1);
- EXPECT_FALSE(IsExtensionOutdated(*extension, specifics));
- specifics.set_version(kVersion2);
- EXPECT_FALSE(IsExtensionOutdated(*extension, specifics));
- specifics.set_version(kVersion3);
- EXPECT_TRUE(IsExtensionOutdated(*extension, specifics));
-}
-
// TODO(akalin): Make ExtensionService/ExtensionUpdater testable
// enough to be able to write a unittest for SetExtensionProperties().
« no previous file with comments | « chrome/browser/sync/glue/extension_util.cc ('k') | chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller_unittest.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698