Bug 61907 - Remove superfluous renderer() test
Summary: Remove superfluous renderer() test
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P4 Trivial
Assignee: Roland Steiner
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-01 23:13 PDT by Roland Steiner
Modified: 2011-12-08 17:40 PST (History)
0 users

See Also:


Attachments
Patch (1.26 KB, patch)
2011-06-01 23:16 PDT, Roland Steiner
tkent: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Roland Steiner 2011-06-01 23:13:04 PDT
Document::recalcStyle() at one point verifies that a renderer is present or bails out ('goto bail_out').

The code after that still contains a redundant check "if (renderer() && ..." that can be removed.
Comment 1 Roland Steiner 2011-06-01 23:16:43 PDT
Created attachment 95727 [details]
Patch
Comment 2 Kent Tamura 2011-06-01 23:23:40 PDT
Comment on attachment 95727 [details]
Patch

ok
Comment 3 Roland Steiner 2011-06-01 23:30:41 PDT
Committed r87879: <http://trac.webkit.org/changeset/87879>