diff options
| author | csharp@chromium.org <csharp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-09-07 20:53:00 +0000 |
|---|---|---|
| committer | csharp@chromium.org <csharp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-09-07 20:53:00 +0000 |
| commit | f57c90fa9a0916afd523287767d880499879b031 (patch) | |
| tree | 99ac73d9c72f5ed7d0b12040a9a95befac25c684 /chrome/browser/autofill | |
| parent | c58ec3c5bbb030259977645b463e865c11e053db (diff) | |
| download | chromium_src-f57c90fa9a0916afd523287767d880499879b031.zip chromium_src-f57c90fa9a0916afd523287767d880499879b031.tar.gz chromium_src-f57c90fa9a0916afd523287767d880499879b031.tar.bz2 | |
Start of New Autofill UI for Windows
Add some very basic functionality for the new Autofill UI system for windows. It currently just shows an empty popup in the correct location. Future cls will add the actual functionality.
BUG=51644
Review URL: https://chromiumcodereview.appspot.com/10825324
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155476 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/autofill')
| -rw-r--r-- | chrome/browser/autofill/autofill_external_delegate.cc | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/chrome/browser/autofill/autofill_external_delegate.cc b/chrome/browser/autofill/autofill_external_delegate.cc index a199b06..912892e 100644 --- a/chrome/browser/autofill/autofill_external_delegate.cc +++ b/chrome/browser/autofill/autofill_external_delegate.cc @@ -18,8 +18,7 @@ using content::RenderViewHost; using WebKit::WebAutofillClient; -AutofillExternalDelegate::~AutofillExternalDelegate() { -} +AutofillExternalDelegate::~AutofillExternalDelegate() {} AutofillExternalDelegate::AutofillExternalDelegate( TabContents* tab_contents, @@ -211,8 +210,12 @@ bool AutofillExternalDelegate::DidAcceptAutofillSuggestions( } void AutofillExternalDelegate::ClearPreviewedForm() { - RenderViewHost* host = tab_contents_->web_contents()->GetRenderViewHost(); - host->Send(new AutofillMsg_ClearPreviewedForm(host->GetRoutingID())); + if (tab_contents_ && tab_contents_->web_contents()) { + RenderViewHost* host = tab_contents_->web_contents()->GetRenderViewHost(); + + if (host) + host->Send(new AutofillMsg_ClearPreviewedForm(host->GetRoutingID())); + } } void AutofillExternalDelegate::HideAutofillPopup() { @@ -343,11 +346,8 @@ void AutofillExternalDelegate::InsertDataListValues( data_list_unique_ids_.end()); } -// Add a "!defined(OS_YOUROS) for each platform that implements this -// in an autofill_external_delegate_YOUROS.cc. Currently there are -// none, so all platforms use the default. -#if !defined(OS_ANDROID) && !defined(TOOLKIT_GTK) +#if defined(OS_MACOSX) AutofillExternalDelegate* AutofillExternalDelegate::Create( TabContents*, AutofillManager*) { |