Bug 63702

Summary: [Chromium] Autofill suggestions appear in upper left corner after input change
Product: WebKit Reporter: Alexander Pavlov (apavlov) <apavlov>
Component: FormsAssignee: Alexander Pavlov (apavlov) <apavlov>
Status: RESOLVED FIXED    
Severity: Normal CC: honten, isherman, jbates, tkent, webkit.review.bot
Priority: P1 Keywords: Regression
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
[PATCH] Suggested solution
none
Patch none

Alexander Pavlov (apavlov)
Reported 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
Attachments
[PATCH] Suggested solution (5.67 KB, patch)
2011-06-30 06:00 PDT, Alexander Pavlov (apavlov) 		no flags
DetailsFormatted DiffDiff
Patch (1.93 KB, patch)
2011-06-30 15:12 PDT, John Bates 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Alexander Pavlov (apavlov)
Comment 1 2011-06-30 06:00:16 PDT
Created attachment 99279 [details] [PATCH] Suggested solution
Kent Tamura
Comment 2 2011-06-30 06:51:22 PDT
Comment on attachment 99279 [details] [PATCH] Suggested solution Looks ok.
WebKit Review Bot
Comment 3 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>
WebKit Review Bot
Comment 4 2011-06-30 07:45:41 PDT
All reviewed patches have been landed. Closing bug.
Ilya Sherman
Comment 5 2011-06-30 12:27:03 PDT
Thanks for the quick fix!
John Bates
Comment 6 2011-06-30 15:12:49 PDT
Created attachment 99372 [details] Patch
Ilya Sherman
Comment 7 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 ;)
John Bates
Comment 8 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.
Note You need to log in before you can comment on or make changes to this bug.