diff options
| author | nirnimesh@chromium.org <nirnimesh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-09 05:16:29 +0000 |
|---|---|---|
| committer | nirnimesh@chromium.org <nirnimesh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-09 05:16:29 +0000 |
| commit | f2a6ec64e7ee1bd87111919344094f86e0602c52 (patch) | |
| tree | 36a117c9a3779fa7a62f7aa217415da53aa86c2c /chrome/browser/automation/automation_provider_observers.h | |
| parent | a3f4684f67993ab52f7bee525436c77e2eb826b4 (diff) | |
| download | chromium_src-f2a6ec64e7ee1bd87111919344094f86e0602c52.zip chromium_src-f2a6ec64e7ee1bd87111919344094f86e0602c52.tar.gz chromium_src-f2a6ec64e7ee1bd87111919344094f86e0602c52.tar.bz2 | |
[pyauto] FillAutofillProfile should not crash with Debug builds
Delete NotificationRegistar on the DB thread.
Also fix AddSavedPassword/RemoveSavedPassword which follow the same model.
BUG=127261
TEST=autofill.py
NOTRY=true
R=dennisjeffrey@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10388038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135987 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/automation/automation_provider_observers.h')
| -rw-r--r-- | chrome/browser/automation/automation_provider_observers.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/chrome/browser/automation/automation_provider_observers.h b/chrome/browser/automation/automation_provider_observers.h index 2b3c7d2..ea0e1d6 100644 --- a/chrome/browser/automation/automation_provider_observers.h +++ b/chrome/browser/automation/automation_provider_observers.h @@ -1183,7 +1183,6 @@ class PasswordStoreLoginsChangedObserver IPC::Message* reply_message, PasswordStoreChange::Type expected_type, const std::string& result_key); - virtual ~PasswordStoreLoginsChangedObserver(); // Schedules a task on the DB thread to register the appropriate observers. virtual void Init(); @@ -1196,6 +1195,7 @@ class PasswordStoreLoginsChangedObserver private: friend struct content::BrowserThread::DeleteOnThread< content::BrowserThread::UI>; + ~PasswordStoreLoginsChangedObserver(); friend class base::DeleteHelper<PasswordStoreLoginsChangedObserver>; // Registers the appropriate observers. Called on the DB thread. @@ -1210,7 +1210,7 @@ class PasswordStoreLoginsChangedObserver base::WeakPtr<AutomationProvider> automation_; scoped_ptr<IPC::Message> reply_message_; - content::NotificationRegistrar registrar_; + scoped_ptr<content::NotificationRegistrar> registrar_; PasswordStoreChange::Type expected_type_; std::string result_key_; @@ -1407,7 +1407,6 @@ class AutofillChangedObserver IPC::Message* reply_message, int num_profiles, int num_credit_cards); - virtual ~AutofillChangedObserver(); // Schedules a task on the DB thread to register the appropriate observers. virtual void Init(); @@ -1420,6 +1419,7 @@ class AutofillChangedObserver private: friend struct content::BrowserThread::DeleteOnThread< content::BrowserThread::UI>; + ~AutofillChangedObserver(); friend class base::DeleteHelper<AutofillChangedObserver>; // Registers the appropriate observers. Called on the DB thread. @@ -1431,7 +1431,7 @@ class AutofillChangedObserver base::WeakPtr<AutomationProvider> automation_; scoped_ptr<IPC::Message> reply_message_; - content::NotificationRegistrar registrar_; + scoped_ptr<content::NotificationRegistrar> registrar_; int num_profiles_; int num_credit_cards_; |