Bug 57244 - Fix call sites of rendererIsEditable that should call isContentEditable instead
Summary: Fix call sites of rendererIsEditable that should call isContentEditable instead
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
: 89380 (view as bug list)
Depends on: 52058
Blocks:
  Show dependency treegraph
 
Reported: 2011-03-28 09:57 PDT by Chang Shu
Modified: 2012-06-21 10:32 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Chang Shu 2011-03-28 09:57:40 PDT
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.
Comment 1 Ryosuke Niwa 2012-06-21 10:32:26 PDT
*** Bug 89380 has been marked as a duplicate of this bug. ***