Bug 88353

Summary: Delete dead code from RenderBox::computeLogicalWidthInRegion
Product: WebKit Reporter: Ojan Vafai <ojan>
Component: New BugsAssignee: Ojan Vafai <ojan>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, hyatt, jchaffraix, mitz, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch tony: review+

Description Ojan Vafai 2012-06-05 11:52:42 PDT
Delete dead code from RenderBox::computeLogicalWidthInRegion
Comment 1 Ojan Vafai 2012-06-05 11:53:13 PDT
Created attachment 145850 [details]
Patch
Comment 2 Tony Chang 2012-06-05 11:57:31 PDT
Comment on attachment 145850 [details]
Patch

Seems fine, but can you cc the original author?  Maybe they have a plan for this or know of some bug caused by not using this variable?
Comment 3 Ojan Vafai 2012-06-05 12:10:21 PDT
The usage of this variable was removed in http://trac.webkit.org/changeset/113885. Looks like it was just an oversight.
Comment 4 Ojan Vafai 2012-06-05 12:17:07 PDT
Committed r119513: <http://trac.webkit.org/changeset/119513>