[Webkit-unassigned] [Bug 123868] New: Change the order of conditions to avoid computing rendererIsEditable()

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 5 22:08:01 PST 2013


https://bugs.webkit.org/show_bug.cgi?id=123868

           Summary: Change the order of conditions to avoid computing
                    rendererIsEditable()
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: BlinkMergeCandidate
          Severity: Normal
          Priority: P2
         Component: HTML Editing
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: darin at apple.com, koivisto at iki.fi, benjamin at webkit.org,
                    enrica at apple.com, akling at apple.com


Merge https://chromium.googlesource.com/chromium/blink/+/c89b413ff0fc4aafa0c71d180b0b1e131bb37707

When we need both Node::renderer() and Node::rendererIsEditable()
conditions to be true to perform some operation, it is more effective
to check for renderer() first, so that if this condition fails we can
avoid unnecessary computation of rendererIsEditable().

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list