[Extensions Tests] Enable ExtensionServiceTest.UpdatePendingExternalCrx
This test was disabled by r81946, but, strangely, the linked bug lists
the failing test as PendingExternalFileOverSync, rather than
UpdatePendingExternalCrx. It's unclear if UpdatePendingExternalCrx
was actually failing.
Locally, the test passes on a run of 100+. Re-enable, and see what
happens.
BUG=79737
Review-Url: https://codereview.chromium.org/2767143002
Cr-Commit-Position: refs/heads/master@{#458877}
diff --git a/chrome/browser/extensions/extension_service_unittest.cc b/chrome/browser/extensions/extension_service_unittest.cc
index c3b026a..689f719a 100644
--- a/chrome/browser/extensions/extension_service_unittest.cc
+++ b/chrome/browser/extensions/extension_service_unittest.cc
@@ -2888,16 +2888,10 @@
EXPECT_TRUE(service()->IsExtensionEnabled(theme_crx));
}
-#if defined(OS_CHROMEOS)
-// Always fails on ChromeOS: http://crbug.com/79737
-#define MAYBE_UpdatePendingExternalCrx DISABLED_UpdatePendingExternalCrx
-#else
-#define MAYBE_UpdatePendingExternalCrx UpdatePendingExternalCrx
-#endif
// Test updating a pending CRX as if the source is an external extension
// with an update URL. In this case we don't know if the CRX is a theme
// or not.
-TEST_F(ExtensionServiceTest, MAYBE_UpdatePendingExternalCrx) {
+TEST_F(ExtensionServiceTest, UpdatePendingExternalCrx) {
InitializeEmptyExtensionService();
EXPECT_TRUE(service()->pending_extension_manager()->AddFromExternalUpdateUrl(
theme_crx,
