diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-16 00:05:15 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-16 00:05:15 +0000 |
| commit | 01899c12fdad06f0e12667871be5ac99624256d8 (patch) | |
| tree | c98b6c59915dcb1cdc9ead4fafb828df08f4a12e /chrome/browser/render_widget_host.cc | |
| parent | c1bcb90272edd39e67f85877b58da4f28fef055e (diff) | |
| download | chromium_src-01899c12fdad06f0e12667871be5ac99624256d8.zip chromium_src-01899c12fdad06f0e12667871be5ac99624256d8.tar.gz chromium_src-01899c12fdad06f0e12667871be5ac99624256d8.tar.bz2 | |
Changes to the autofill popup:
- we now reuse the existing popup if there is one (instead of creating a new on with each key stoke), this prevents the flickering we used to see.
- we don't hide the popup on every key stroke, instead we rely on the client editor telling us the field is not being edited anymore.
TEST=Enter some text in a form's text field several times, the autofill should show and not flicker.
BUG=5258
Review URL: http://codereview.chromium.org/14100
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@7017 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/render_widget_host.cc')
| -rw-r--r-- | chrome/browser/render_widget_host.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/render_widget_host.cc b/chrome/browser/render_widget_host.cc index 41fd233..90aac3c 100644 --- a/chrome/browser/render_widget_host.cc +++ b/chrome/browser/render_widget_host.cc @@ -340,8 +340,8 @@ void RenderWidgetHost::OnMsgClose() { } void RenderWidgetHost::OnMsgRequestMove(const gfx::Rect& pos) { - // Don't allow renderer widgets to move themselves by default. Maybe this - // policy will change if we add more types of widgets. + // Note that we ignore the position. + view_->SetSize(pos.size()); } void RenderWidgetHost::OnMsgPaintRect( |