Bug 63702 - [Chromium] Autofill suggestions appear in upper left corner after input change
Summary: [Chromium] Autofill suggestions appear in upper left corner after input change
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P1 Normal
Assignee: Alexander Pavlov (apavlov)
URL:
Keywords: Regression
Depends on:
Blocks:
 
Reported: 2011-06-30 04:47 PDT by Alexander Pavlov (apavlov)
Modified: 2011-06-30 20:21 PDT (History)
5 users (show)

See Also:


Attachments
[PATCH] Suggested solution (5.67 KB, patch)
2011-06-30 06:00 PDT, Alexander Pavlov (apavlov)
no flags Details | Formatted Diff | Diff
Patch (1.93 KB, patch)
2011-06-30 15:12 PDT, John Bates
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Pavlov (apavlov) 2011-06-30 04:47:52 PDT
1. go to some page form field that will show autofill.
2. start typing, notice the popup appears in the upper left corner (after its contents have been updated due to the changed input).

The issue is mostly due to an unimplemented "IntPoint ChromeClientImpl::screenToWindow(const IntPoint&) const" method.

Upstreaming http://code.google.com/p/chromium/issues/detail?id=87972
Comment 1 Alexander Pavlov (apavlov) 2011-06-30 06:00:16 PDT
Created attachment 99279 [details]
[PATCH] Suggested solution
Comment 2 Kent Tamura 2011-06-30 06:51:22 PDT
Comment on attachment 99279 [details]
[PATCH] Suggested solution

Looks ok.
Comment 3 WebKit Review Bot 2011-06-30 07:45:35 PDT
Comment on attachment 99279 [details]
[PATCH] Suggested solution

Clearing flags on attachment: 99279

Committed r90119: <http://trac.webkit.org/changeset/90119>
Comment 4 WebKit Review Bot 2011-06-30 07:45:41 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 Ilya Sherman 2011-06-30 12:27:03 PDT
Thanks for the quick fix!
Comment 6 John Bates 2011-06-30 15:12:49 PDT
Created attachment 99372 [details]
Patch
Comment 7 Ilya Sherman 2011-06-30 15:35:47 PDT
(In reply to comment #6)
> Created an attachment (id=99372) [details]
> Patch

Methinks you attached this to the wrong bug ;)
Comment 8 John Bates 2011-06-30 15:49:30 PDT
Wow - my mistake (I didn't realize I had to be at the top of ChangeLogs). Please ignore the last patch.