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

Unified Diff: chrome/browser/autofill/android/auxiliary_profile_loader_android.h

Issue 12282004: Added personal_data_manager android implementation for auto-populating auto-fill on android builds … (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed assortment of nits Created 7 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: chrome/browser/autofill/android/auxiliary_profile_loader_android.h
diff --git a/chrome/browser/autofill/android/auxiliary_profile_loader_android.h b/chrome/browser/autofill/android/auxiliary_profile_loader_android.h
new file mode 100644
index 0000000000000000000000000000000000000000..6ac5a8064ee44458ff27c85b3bcb62f19bfaba87
--- /dev/null
+++ b/chrome/browser/autofill/android/auxiliary_profile_loader_android.h
@@ -0,0 +1,54 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_AUTOFILL_ANDROID_AUXILIARY_PROFILE_LOADER_ANDROID_H_
+#define CHROME_BROWSER_AUTOFILL_ANDROID_AUXILIARY_PROFILE_LOADER_ANDROID_H_
+
+#include <vector>
+
+#include "base/string16.h"
+
+namespace autofill {
+
+class AuxiliaryProfileLoader {
+ // Abstract interface class provides interface for loading
+ // contact information.
Ilya Sherman 2013/03/05 00:42:38 Why do you need an abstract interface for this? T
Ilya Sherman 2013/03/05 00:42:38 nit: Comment goes above the class.
apiccion 2013/03/09 00:53:41 Done.
+ public:
+ virtual ~AuxiliaryProfileLoader() {}
+
+ // Returns address street.
+ virtual string16 GetStreet() const = 0;
+ // Returns address post office box.
+ virtual string16 GetPostOfficeBox() const = 0;
+ // Returns address neighborhood (e.g. Noe Valley, Nob Hill, Twin Peaks, ...).
+ virtual string16 GetNeighborhood() const = 0;
+ // Returns address region such as state or province information
+ // (e.g. Ontario, California, Hubei).
+ virtual string16 GetRegion() const = 0;
+ // Returns address city.
+ virtual string16 GetCity() const = 0;
+ // Returns address postal code or zip code.
+ virtual string16 GetPostalCode() const = 0;
+ // Returns address country.
+ virtual string16 GetCountry() const = 0;
+
+ // Returns contact's first name.
+ virtual string16 GetFirstName() const = 0;
+ // Returns contact's middle name.
+ virtual string16 GetMiddleName() const = 0;
+ // Returns contact's last name.
+ virtual string16 GetLastName() const = 0;
+ // Returns contact's suffix (e.g. Ph.D, M.D., ...).
+ virtual string16 GetSuffix() const = 0;
+
+ // Populates string vector parameter with contact's email addresses.
+ virtual void GetEmailAddresses(std::vector<string16>*) const = 0;
Ilya Sherman 2013/03/05 00:42:38 nit: Parameters should always be named.
apiccion 2013/03/09 00:53:41 Done.
+
+ // Populates string vector parameter with contact's phones numbers.
+ virtual void GetPhoneNumbers(std::vector<string16>*) const = 0;
Ilya Sherman 2013/03/05 00:42:38 nit: Parameters should always be named.
apiccion 2013/03/09 00:53:41 Done.
+};
+
+} // namespace
+
+#endif // CHROME_BROWSER_AUTOFILL_ANDROID_AUXILIARY_PROFILE_LOADER_ANDROID_H_

Powered by Google App Engine
This is Rietveld 408576698