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

Unified Diff: extensions/common/manifest_handlers/oauth2_manifest_unittest.cc

Issue 671553002: Move extensions manifest handler for OAuth2 to //extensions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 | « extensions/common/manifest_handlers/oauth2_manifest_handler.cc ('k') | extensions/extensions.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/common/manifest_handlers/oauth2_manifest_unittest.cc
diff --git a/chrome/common/extensions/api/identity/extension_manifests_auth_unittest.cc b/extensions/common/manifest_handlers/oauth2_manifest_unittest.cc
similarity index 98%
rename from chrome/common/extensions/api/identity/extension_manifests_auth_unittest.cc
rename to extensions/common/manifest_handlers/oauth2_manifest_unittest.cc
index 0606c320055f838385dc6ea0fc147d849165ed4b..bd22eb53bbe7e6182bda06e55d038945132906f0 100644
--- a/chrome/common/extensions/api/identity/extension_manifests_auth_unittest.cc
+++ b/extensions/common/manifest_handlers/oauth2_manifest_unittest.cc
@@ -3,9 +3,9 @@
// found in the LICENSE file.
#include "base/test/values_test_util.h"
-#include "chrome/common/extensions/api/identity/oauth2_manifest_handler.h"
-#include "chrome/common/extensions/manifest_tests/chrome_manifest_test.h"
#include "extensions/common/manifest_constants.h"
+#include "extensions/common/manifest_handlers/oauth2_manifest_handler.h"
+#include "extensions/common/manifest_test.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace extensions {
@@ -25,7 +25,7 @@ const char kAutoApproveNotAllowedWarning[] =
} // namespace
-class OAuth2ManifestTest : public ChromeManifestTest {
+class OAuth2ManifestTest : public ManifestTest {
protected:
enum AutoApproveValue {
AUTO_APPROVE_NOT_SET,
@@ -296,7 +296,6 @@ TEST_F(OAuth2ManifestTest, ComponentWithChromeClientId) {
scoped_refptr<extensions::Extension> extension =
LoadAndExpectSuccess(manifest, extensions::Manifest::COMPONENT);
EXPECT_TRUE(OAuth2Info::GetOAuth2Info(extension.get()).client_id.empty());
-
}
{
@@ -306,7 +305,6 @@ TEST_F(OAuth2ManifestTest, ComponentWithChromeClientId) {
scoped_refptr<extensions::Extension> extension =
LoadAndExpectSuccess(manifest, extensions::Manifest::COMPONENT);
EXPECT_TRUE(OAuth2Info::GetOAuth2Info(extension.get()).client_id.empty());
-
}
}
« no previous file with comments | « extensions/common/manifest_handlers/oauth2_manifest_handler.cc ('k') | extensions/extensions.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698