WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
41286
[chromium] EditorClient should forward text field related calls to the WebViewClient
https://bugs.webkit.org/show_bug.cgi?id=41286
Summary
[chromium] EditorClient should forward text field related calls to the WebVie...
Jay Civelli
Reported
2010-06-28 10:53:15 PDT
In order to be able to move password autocomplete and field autocomplete code from EditorClientImpl.cpp to the Chromium code, the text field related methods should be exposed on the WebViewClient.
Attachments
First version of the patch
(10.54 KB, patch)
2010-06-28 13:25 PDT
,
Jay Civelli
fishd
: review-
Details
Formatted Diff
Diff
Updated with fishd's suggested changes
(11.13 KB, patch)
2010-07-02 15:11 PDT
,
Jay Civelli
no flags
Details
Formatted Diff
Diff
Patch for landing
(11.09 KB, patch)
2010-07-07 17:12 PDT
,
Jay Civelli
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Jay Civelli
Comment 1
2010-06-28 13:25:44 PDT
Created
attachment 59930
[details]
First version of the patch
Dirk Pranke
Comment 2
2010-07-02 12:27:28 PDT
Comment on
attachment 59930
[details]
First version of the patch Looks basically good to me, but I'm neither an expert on this area of the code nor a reviewer :)
Darin Fisher (:fishd, Google)
Comment 3
2010-07-02 13:38:35 PDT
Comment on
attachment 59930
[details]
First version of the patch WebKit/chromium/src/EditorClientImpl.cpp:823 + HTMLInputElement* inputElement = WebKit::toHTMLInputElement(element); EditorClientImpl is defined in the WebKit namespace, so you don't need to have the WebKit:: prefix before toHTMLInputElement. WebKit/chromium/public/WebViewClient.h:146 + virtual void textFieldHandlingKeyDown(const WebInputElement&, const WebKeyboardEvent&) { } textFieldDidReceiveKeyDown? WebKit/chromium/public/WebViewClient.h:144 + virtual void textDidChangeInTextField(const WebInputElement&) { } textFieldDidChange? WebKit/chromium/public/WebInputElement.h:83 + WEBKIT_API bool readOnly() const; hmm... maybe this should be named isReadOnly to match isEnabledFormControl. i know you are just replicating readOnly from the WebCore name. looks good otherwise.
Jay Civelli
Comment 4
2010-07-02 15:11:53 PDT
Created
attachment 60411
[details]
Updated with fishd's suggested changes
Darin Fisher (:fishd, Google)
Comment 5
2010-07-02 22:11:41 PDT
Comment on
attachment 60411
[details]
Updated with fishd's suggested changes WebKit/chromium/public/WebViewClient.h:142 + // Called when a key down happens in a text-field. this comment seems fairly redundant now with the method name, so you could leave it out. R=me
Jay Civelli
Comment 6
2010-07-07 17:12:45 PDT
Created
attachment 60808
[details]
Patch for landing
WebKit Commit Bot
Comment 7
2010-07-07 17:44:41 PDT
Comment on
attachment 60808
[details]
Patch for landing Rejecting patch 60808 from commit-queue.
jcivelli@chromium.org
does not have committer permissions according to
http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/webkitpy/common/config/committers.py
. - If you do not have committer rights please read
http://webkit.org/coding/contributing.html
for instructions on how to use bugzilla flags. - If you have committer rights please correct the error in WebKitTools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed). Due to
bug 30084
the commit-queue will require a restart after your change. Please contact
eseidel@chromium.org
to request a commit-queue restart. After restart the commit-queue will correctly respect your committer rights.
WebKit Commit Bot
Comment 8
2010-07-08 22:26:05 PDT
Comment on
attachment 60808
[details]
Patch for landing Clearing flags on attachment: 60808 Committed
r62893
: <
http://trac.webkit.org/changeset/62893
>
WebKit Commit Bot
Comment 9
2010-07-08 22:26:10 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug