| Summary: | Element::recalcStyle() needlessly null-checks the result of styleForElement() | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | mitz | ||||
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | ||||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Mac | ||||||
| OS: | OS X 10.5 | ||||||
| Attachments: |
|
||||||
|
Description
mitz
2008-02-09 10:56:44 PST
Created attachment 19019 [details]
Remove unneeded null checks
Comment on attachment 19019 [details]
Remove unneeded null checks
This looks good to me, but as discussed, a better word than redundant should be used to describe why the null checks aren't needed. Just stating that styleForElement() cannot return null should be sufficient.
|