AutoFill dialog UI polish. Based on feedback from thakis.

Change to AutoFillDialog.xib was window anchoring flags to better position dialog in different window sizes on first run.

> 1.) The initial position of the dialog is half off-screen on my 15'' MBP. It should probably be 
> more on the left.
> 

Done.  Auto centering now.

> 2.) Furthermore, the dialog should autosave its position. See e.g. 
> keyword_editor_cocoa_controller.mm on how to do this (the |if (g_browser_process...)| block in 
> |initWithProfile| – it takes only a couple lines).
> 

Done.

> 3.) When clicking "Add an address", I find it weird that the new address starts in the collapsed 
> state. That makes it hard to see how to delete it again, and chances are that when I'm adding a 
> new address I want to edit it immediately.
> 

Done.


BUG=36441
TEST=see manual repro steps above.



Review URL: http://codereview.chromium.org/660130

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40075 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 0 deletions

diff --git a/chrome/browser/autofill/autofill_manager.cc b/chrome/browser/autofill/autofill_manager.cc
index 1ede7bf..0d005aa 100644
--- a/chrome/browser/autofill/autofill_manager.cc
+++ b/chrome/browser/autofill/autofill_manager.cc
@@ -32,6 +32,11 @@ AutoFillManager::~AutoFillManager() {
 }
 
 // static
+void AutoFillManager::RegisterBrowserPrefs(PrefService* prefs) {
+  prefs->RegisterDictionaryPref(prefs::kAutoFillDialogPlacement);
+}
+
+// static
 void AutoFillManager::RegisterUserPrefs(PrefService* prefs) {
   prefs->RegisterBooleanPref(prefs::kAutoFillInfoBarShown, false);
   prefs->RegisterBooleanPref(prefs::kAutoFillEnabled, false);