Bug 56979 - Expose 'isUserEditable()' (conceptually) in Chromium WebKit API
Summary: Expose 'isUserEditable()' (conceptually) in Chromium WebKit API
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-23 17:32 PDT by Ilya Sherman
Modified: 2013-04-11 13:31 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ilya Sherman 2011-03-23 17:32:08 PDT
See https://bugs.webkit.org/show_bug.cgi?id=56809 for some backstory.

For Chromium Autofill, we need a concept akin to "Is this field user-editable?"  Currently, we approximate this by checking that the field is (a) enabled, (b) not read only, and (c) focusable.  (c) is particularly odd, as we call into HTMLFormControlElement::isFocusable(), which assumes that layout has been done -- but does not do the layout itself.  It might be wise to encapsulate this or similar "isUserEditable()" logic within WebKit, and expose that one method in the API.  Otherwise, we should re-evaluate whether we are exposing the right methods, especially with the implicit dependency on layout having already occurred.  James also mentioned that reentrancy might be an issue to watch out for here.