Fix DCHECK, memory leak, and refactor PasswordStore to use CancelableRequest

chrome/test/live_sync/live_passwords_sync_test.h

Index: chrome/test/live_sync/live_passwords_sync_test.h
diff --git a/chrome/test/live_sync/live_passwords_sync_test.h b/chrome/test/live_sync/live_passwords_sync_test.h
index 81fe21512298abbb03121c77c3a56dbd08957a08..03522c3b24820ee19c091b8ae2ec28203ae00ec8 100644
--- a/chrome/test/live_sync/live_passwords_sync_test.h
+++ b/chrome/test/live_sync/live_passwords_sync_test.h
@@ -64,7 +64,8 @@ class LivePasswordsSyncTest : public LiveSyncTest {
         : PasswordStoreConsumer(), result_(result) {}
 
     virtual void OnPasswordStoreRequestDone(
-        int handle, const std::vector<webkit_glue::PasswordForm*>& result) {
+        PasswordStore::Handle handle,
+        const std::vector<webkit_glue::PasswordForm*>& result) {
       result_.clear();
       for (std::vector<webkit_glue::PasswordForm*>::const_iterator it =
            result.begin(); it != result.end(); ++it) {