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

Unified Diff: chrome/chrome_tests.gypi

Issue 23622013: Move password_form_conversion_utils out of //content into //autofill. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@password_form_includes_removal
Patch Set: Created 7 years, 4 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 | components/autofill.gypi » ('j') | components/autofill/content/DEPS » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_tests.gypi
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi
index a090420ff7476fdc4e856a9073c920f35a214be1..66eb4209031d26e42fef33cda70b10efac535043 100644
--- a/chrome/chrome_tests.gypi
+++ b/chrome/chrome_tests.gypi
@@ -1165,6 +1165,10 @@
'sources': [
'../apps/app_restore_service_browsertest.cc',
'../apps/load_and_launch_browsertest.cc',
+ # TODO(blundell): Is this the right place to put this browsertest in
blundell 2013/08/30 14:03:23 Jói, I'd love your input here.
Jói 2013/08/30 14:08:17 Yes, this seems like the right way to do it.
+ # the short-term, coupled with creating a bug for bringing up a
+ # components_browsertests target?
+ '../components/autofill/content/renderer/password_form_conversion_utils_browsertest.cc',
'../components/autofill/content/renderer/test_password_autofill_agent.h',
'../components/autofill/content/renderer/test_password_autofill_agent.cc',
'../remoting/test/auth_browsertest.cc',
« no previous file with comments | « no previous file | components/autofill.gypi » ('j') | components/autofill/content/DEPS » ('J')

Powered by Google App Engine
This is Rietveld 408576698