Bug 57244
| Summary: | Fix call sites of rendererIsEditable that should call isContentEditable instead | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Chang Shu <cshu> |
| Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | darin, rniwa, shinyak |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | PC | ||
| OS: | All | ||
| Bug Depends on: | 52058 | ||
| Bug Blocks: | |||
Chang Shu
Bug 52058 provides function isContentEditable in addition to rendererIsEditable. rendererIsEditable is an internal function that retrieve render style info to determine the editability of a node. However, the render style info may not sync up with DOM contenteditable attribute. isContentEditable fixes this issue by calling udpate layout. But the performance isn't great. For all call sites that are currently all calling rendererIsEditable, some of them have to call isContentEditable for the sake of code correctness.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ryosuke Niwa
*** Bug 89380 has been marked as a duplicate of this bug. ***