summaryrefslogtreecommitdiffstats
path: root/chrome/browser/password_manager
Commit message (Collapse)AuthorAgeFilesLines
* Move CancellationFlag and WaitableEvent to the synchronization subdirectory.brettw@chromium.org2011-01-024-4/+4
| | | | | | | | TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/5977010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70369 0039d316-1c4b-4281-b951-d872f2087c98
* Move base/mac_util.h to base/mac and use the base::mac namespace.brettw@chromium.org2011-01-011-3/+4
| | | | | | | | | | | Fix up callers to use the new location & namespace. Remove includes from  files where it wasn't necessary. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6046009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70359 0039d316-1c4b-4281-b951-d872f2087c98
* Remove base/platform_thread.h stub and fix up all callers to use the new ↵brettw@google.com2011-01-011-3/+4
| | | | | | | | | location and namespace. TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70346 0039d316-1c4b-4281-b951-d872f2087c98
* Move base/thread.h to base/threading, fix up callers to use the new location.brettw@chromium.org2011-01-012-2/+2
| | | | | | | | TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6028009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70345 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unneeded includes of notification_service.h.thestig@chromium.org2010-12-171-1/+0
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5875005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69507 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unneeded includes of notification_service.h in other headers.thestig@chromium.org2010-12-152-1/+2
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5783006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69321 0039d316-1c4b-4281-b951-d872f2087c98
* Last few NewRunnableMethod scoped_refptr params to fix.willchan@chromium.org2010-12-151-1/+1
| | | | | | | | | BUG=28083 TEST=builds Review URL: http://codereview.chromium.org/5870001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69229 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unneeded profile.h usage in headers.thestig@chromium.org2010-12-102-1/+6
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5762001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68809 0039d316-1c4b-4281-b951-d872f2087c98
* Add the "virtual" keyword on method overrides that are missing it.erg@google.com2010-12-081-14/+14
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5648004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68606 0039d316-1c4b-4281-b951-d872f2087c98
* Reduce the include footprint of pref_service.hmnissler@chromium.org2010-12-032-0/+3
| | | | | | | | | | | | | | | This removes the pref_value_store.h header from the pref_service.h include list and cleans up any users that were getting the MessageLoop and BrowserThread definitions indirectly through pref_service.h BUG=64893 TEST=compiles and passes tests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67997 Review URL: http://codereview.chromium.org/5461001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68157 0039d316-1c4b-4281-b951-d872f2087c98
* Move:ben@chromium.org2010-12-026-6/+6
| | | | | | | | | | | | | | | file_path_watcher into subdir profile* into profiles/ subdir login* into ui/login visitedlink* into subdir BUG=none TEST=none TBR=brettw Review URL: http://codereview.chromium.org/5606002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68069 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 67997 - Reduce the include footprint of pref_service.hmnissler@chromium.org2010-12-022-3/+0
| | | | | | | | | | | | | | | | This removes the pref_value_store.h header from the pref_service.h include list and cleans up any users that were getting the MessageLoop and BrowserThread definitions indirectly through pref_service.h BUG=64893 TEST=compiles and passes tests Review URL: http://codereview.chromium.org/5461001 TBR=mnissler@chromium.org Review URL: http://codereview.chromium.org/5575001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67998 0039d316-1c4b-4281-b951-d872f2087c98
* Reduce the include footprint of pref_service.hmnissler@chromium.org2010-12-022-0/+3
| | | | | | | | | | | | | This removes the pref_value_store.h header from the pref_service.h include list and cleans up any users that were getting the MessageLoop and BrowserThread definitions indirectly through pref_service.h BUG=64893 TEST=compiles and passes tests Review URL: http://codereview.chromium.org/5461001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67997 0039d316-1c4b-4281-b951-d872f2087c98
* Remove duplicate NotificationObserver mock.mnissler@chromium.org2010-12-012-7/+8
| | | | | | | | | BUG=none TEST=Compiles and passes tests. Review URL: http://codereview.chromium.org/5448001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67848 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 67326 - Re-landing original change since it didn't seem to fix the ↵brettw@chromium.org2010-11-242-0/+10
| | | | | | | | | | | | | | | | | | | | | failure. Revert 67281 to fix failing FullLogin test - The earlier TabContentsWrapper changes had incomplete support for password manager. This wires up Password Manager properly, making sure it observes all notifications that it needs to function. http://crbug.com/63664 TEST=see bug Review URL: http://codereview.chromium.org/5311006 TBR=ben@chromium.org Review URL: http://codereview.chromium.org/5291004 TBR=andybons@chromium.org Review URL: http://codereview.chromium.org/5347003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67337 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 67281 to fix failing FullLogin test - The earlier TabContentsWrapper ↵andybons@chromium.org2010-11-242-10/+0
| | | | | | | | | | | | | | | | changes had incomplete support for password manager. This wires up Password Manager properly, making sure it observes all notifications that it needs to function. http://crbug.com/63664 TEST=see bug Review URL: http://codereview.chromium.org/5311006 TBR=ben@chromium.org Review URL: http://codereview.chromium.org/5291004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67326 0039d316-1c4b-4281-b951-d872f2087c98
* The earlier TabContentsWrapper changes had incomplete support for password ↵ben@chromium.org2010-11-242-0/+10
| | | | | | | | | | | | | manager. This wires up Password Manager properly, making sure it observes all notifications that it needs to function. http://crbug.com/63664 TEST=see bug Review URL: http://codereview.chromium.org/5311006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67281 0039d316-1c4b-4281-b951-d872f2087c98
* Make pink's TabContentsWrapper change compile on Windows.ben@chromium.org2010-11-182-22/+25
| | | | | | | | | | Code by pinkerton@, with modifications by evanm and myself to get it to build on windows/linux/chromeos. BUG=none TEST=none Review URL: http://codereview.chromium.org/4694008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66626 0039d316-1c4b-4281-b951-d872f2087c98
* Implement device management policy providerdanno@chromium.org2010-11-092-14/+2
| | | | | | | | | BUG=62462 TEST=DeviceManagementPolicyProviderTest* Review URL: http://codereview.chromium.org/4382001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65520 0039d316-1c4b-4281-b951-d872f2087c98
* Convert LOG(INFO) to VLOG(1) - misc. chrome/browser/*/*.pkasting@chromium.org2010-10-211-1/+1
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/4009001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63420 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Even more ctor/virtual deinlining.erg@google.com2010-10-192-1/+3
| | | | | | | | | | | (Only 424k off Linux debug .a files). BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3859003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63059 0039d316-1c4b-4281-b951-d872f2087c98
* Integration tests for password sync.albertb@chromium.org2010-10-143-47/+85
| | | | | | | | | BUG=none TEST=self Review URL: http://codereview.chromium.org/3701007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62632 0039d316-1c4b-4281-b951-d872f2087c98
* Move Stats, histograms, and field trial into a metrics subdirectory of base andbrettw@chromium.org2010-10-142-2/+2
| | | | | | | | | put them in the base namespace. TEST=it compiles BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62510 0039d316-1c4b-4281-b951-d872f2087c98
* base: Move SplitString functions into the base namespace and update the callers.tfarina@chromium.org2010-10-141-2/+2
| | | | | | | | | BUG=None TEST=trybots Review URL: http://codereview.chromium.org/3750001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62483 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ChromeThread to BrowserThread Part21:tfarina@chromium.org2010-10-127-7/+7
| | | | | | | | | | | - Include browser_thread.h instead of chrome_thread.h in more 100 files. BUG=56926 TEST=trybots Review URL: http://codereview.chromium.org/3691006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62286 0039d316-1c4b-4281-b951-d872f2087c98
* Set the source of the password store notifications to the PasswordStorealbertb@chromium.org2010-10-114-23/+25
| | | | | | | | | | | | sending out the notification. This is needed for the password integration tests. BUG=none TEST=unit tests Review URL: http://codereview.chromium.org/3529027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62156 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ChromeThread to BrowserThread Part8:tfarina@chromium.org2010-10-0910-124/+150
| | | | | | | | | | | - Rename entries under net and password_manager and printing. BUG=56926 TEST=trybots Review URL: http://codereview.chromium.org/3645001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62062 0039d316-1c4b-4281-b951-d872f2087c98
* base: Finish moving the SplitString functions from string_util.h to ↵tfarina@chromium.org2010-09-241-1/+2
| | | | | | | | | | | string_split.h BUG=None TEST=trybos Review URL: http://codereview.chromium.org/3447008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60422 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Move virtual methods to implementation files.erg@google.com2010-09-232-3/+5
| | | | | | | | | | | Remove logging.h and other headers where possible. BUG=none TEST=none Review URL: http://codereview.chromium.org/3461019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60369 0039d316-1c4b-4281-b951-d872f2087c98
* Save passwords for HTTP+FTP authentication.cbentzel@chromium.org2010-09-222-11/+48
| | | | | | | | | | | | Rather than modifying the PasswordForm to have a username_element and password_element, change the logic in PasswordFormManager::HasValidPasswordForm to only care about empty username_element and password_element for HTML-form based PasswordForms. BUG=5236,49894 TEST=unit_tests --gtest_filter="*TestValidForms*". Manual: Start Chrome with fresh profile, go to a Digest authenticating proxy or server and enter a username+password. After page loads successfully, click "save password" in the infobar. Restart chrome with the same profile, and make sure username+password dialog prompt is filled in. Review URL: http://codereview.chromium.org/3419011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60171 0039d316-1c4b-4281-b951-d872f2087c98
* Add tests to verify that non-ASCII passwords (and other password form data) ↵mdm@chromium.org2010-09-141-23/+68
| | | | | | | | | | can be stored correctly. TEST=tests pass BUG=54065 Review URL: http://codereview.chromium.org/3358031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59382 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: fix retrieving passwords with non-ASCII characters from the default ↵mdm@chromium.org2010-09-121-1/+1
| | | | | | | | | | password store. TEST=save a password containing a non-ASCII character, it should autofill correctly BUG=54065 Review URL: http://codereview.chromium.org/3316014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59202 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 8)thestig@chromium.org2010-09-081-2/+1
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3232003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58794 0039d316-1c4b-4281-b951-d872f2087c98
* Add policy to control 'show password' facility in the password manager.mnissler@chromium.org2010-09-071-0/+1
| | | | | | | | | BUG=49609 TEST=Manual Review URL: http://codereview.chromium.org/3299010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58692 0039d316-1c4b-4281-b951-d872f2087c98
* Moving file_util::FileInfo to base::PlatformFileInfo, and adding thedumi@chromium.org2010-09-031-3/+3
| | | | | | | | | | | | last_accessed and creation_time fields. BUG=none TEST=none Review URL: http://codereview.chromium.org/3347005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58454 0039d316-1c4b-4281-b951-d872f2087c98
* Move prefs-related files under chrome/browser/ into a prefs/ subdir.evan@chromium.org2010-08-266-6/+6
| | | | | | | | | | | | Rename includes, resort header include order in places where the rename changed the order. BUG=50548 TEST=compiles Review URL: http://codereview.chromium.org/3203008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57434 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up unnecessary #includes of base/{scoped_ptr.h,string16.h}.viettrungluu@chromium.org2010-08-192-3/+1
| | | | | | | | | | | | Also fix a bunch of #includes. Folks, scoped_refptr<> is defined in base/ref_counted.h, not in base/scoped_ptr.h. BUG=none TEST=builds Review URL: http://codereview.chromium.org/3132024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56712 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 6)thestig@chromium.org2010-08-193-4/+1
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3093013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56641 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove "obviously" unneeded standard C++ library #includes.viettrungluu@chromium.org2010-08-152-4/+0
| | | | | | | | | BUG=none TEST=builds Review URL: http://codereview.chromium.org/3179017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56171 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Forward declare everything possible in testing_profile.herg@google.com2010-08-132-0/+4
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3129007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56053 0039d316-1c4b-4281-b951-d872f2087c98
* Add token serialization to token_service.chron@chromium.org2010-08-123-34/+5
| | | | | | | | | | | | | | | The token service now supports serialization of tokens into the web database. The token service doesn't actually know the email of the user though, and it's assumed that whoever is using the token service does. Not sure whether that's a bad choice or not, open to suggestions. This CL adds a new DB table. Design comments / corrections are welcome. Testing in Chromium OS is in progress and not yet done. This CL will not be submitted prior to more Chromium OS testing. BUG=47092,47093 TEST=Unit tests included Review URL: http://codereview.chromium.org/3061025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55835 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: fix several bugs in KWallet password store support.mdm@chromium.org2010-08-102-4/+36
| | | | | | | | | | | | * Don't disable kwallet support if the first site visited with a login form has no saved passwords. * Don't crash when there are certain types of invalid entries stored in the wallet that fail to deserialize. TEST=start with --password-store=detect in KDE and check that the above happens BUG=49968 Review URL: http://codereview.chromium.org/3048052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55535 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 4)thestig@chromium.org2010-08-071-1/+0
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/2819094 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55345 0039d316-1c4b-4281-b951-d872f2087c98
* Add #include utf_string_conversions.h to all files that use ASCIIToWide andbrettw@chromium.org2010-08-035-3/+7
| | | | | | | | | | | | | | | ASCIIToUTF16. I removed string_util includes from a few places where it obviously wasn't needed. In a separate pass, I'm going to remove ASCIITo* from string_util, then I'm going to do an even later pass to find the unnecessary string_util.h includes and remove them. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3058027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54746 0039d316-1c4b-4281-b951-d872f2087c98
* Convert more callers of the integer/string functions to usingbrettw@chromium.org2010-07-311-2/+2
| | | | | | | | | | string_number_conversions.h TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3013046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54454 0039d316-1c4b-4281-b951-d872f2087c98
* Move the number conversions from string_util to a new file.brettw@chromium.org2010-07-302-3/+5
| | | | | | | | | | | | | Use the base namespace in the new file. Update callers. I removed all wstring variants and also the string->number ones that ignore the return value. That encourages people to write code and forget about error handling. TEST=included unit tests BUG=none Review URL: http://codereview.chromium.org/3056029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54355 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unneeded headers from app/thestig@chromium.org2010-07-291-2/+0
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/2819063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54102 0039d316-1c4b-4281-b951-d872f2087c98
* `#pragma once` for app, base, chrome, gfx, ipc, net, skia, viewsthakis@chromium.org2010-07-2617-0/+17
| | | | | | | | | BUG=50273 TEST=everything still builds, build is 10% faster on windows, same speed on mac/linux TBR: erg git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53716 0039d316-1c4b-4281-b951-d872f2087c98
* Add DCHECKs to check PasswordStore's threadingdavidben@chromium.org2010-07-263-2/+11
| | | | | | | | | | | | | | PasswordStore can only be queried from one thread, although the implementation doesn't immediately look it. This adds DCHECKs to ensure we don't mess this up in future. R=wtc,stuartmorgan BUG=none TEST=PasswordManagerTest.* Review URL: http://codereview.chromium.org/3056008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53707 0039d316-1c4b-4281-b951-d872f2087c98
* Header Cleanup2: Remove another include of sqlite_utils.h that is not needed.tfarina@chromium.org2010-07-181-3/+6
| | | | | | | | | | | BUG=None TEST=trybots Signed-off-by: Thiago Farina <tfarina@chromium.org> Review URL: http://codereview.chromium.org/2843062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52837 0039d316-1c4b-4281-b951-d872f2087c98