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

Unified Diff: components/password_manager/core/browser/password_manager_test_utils.h

Issue 1668523002: [Password Manager] Switch password manager code to use the Feature framework. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addresses the review inputs. Created 4 years, 10 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: components/password_manager/core/browser/password_manager_test_utils.h
diff --git a/components/password_manager/core/browser/password_manager_test_utils.h b/components/password_manager/core/browser/password_manager_test_utils.h
index 93f004ff4cbd86d1024bf7c6b0943d8fa0f35116..39f9c6c5c162e74b664ba79601fe9534ce19c4f5 100644
--- a/components/password_manager/core/browser/password_manager_test_utils.h
+++ b/components/password_manager/core/browser/password_manager_test_utils.h
@@ -8,6 +8,7 @@
#include <iosfwd>
#include <vector>
+#include "base/feature_list.h"
#include "base/memory/ref_counted.h"
#include "components/autofill/core/common/password_form.h"
#include "components/password_manager/core/browser/password_store.h"
@@ -73,6 +74,13 @@ MATCHER_P(UnorderedPasswordFormElementsAre, expectations, "") {
result_listener->stream());
}
+// Helper function to set command-line |feature|. It |appends_to_list| of
Bernhard Bauer 2016/02/11 11:12:40 Nit: The description of the |appends_to_list| para
Pritam Nikam 2016/02/25 14:16:06 Done.
+// --enable-features if |set_enabled| is set to true; otherwise to that of
+// --disable-features.
+void EnableFeature(const base::Feature& feature,
+ bool set_enabled,
+ bool appends_to_list);
+
class MockPasswordStoreObserver : public PasswordStore::Observer {
public:
MockPasswordStoreObserver();

Powered by Google App Engine
This is Rietveld 408576698