Bug 160350

Summary: Remove an unused line of code.
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: SVGAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: ggaren, mmaxfield, sabouhallawa, zimmermann
Priority: P2    
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 160344    
Attachments:
Description Flags
proposed patch. simon.fraser: review+

Description Mark Lam 2016-07-29 11:39:12 PDT
SVGInlineTextBox::paintText() instantiates a local variable selectionTextRun.

Thanks to analysis by Said Abou-Hallawa, we know that selectionTextRun was introduced in https://trac.webkit.org/changeset/61393, but was later rendered unused in https://trac.webkit.org/changeset/62099 because we switched to using paintTextWithShadows() with the selectionStyle.  Hence, selectionTextRun can be removed.
Comment 1 Mark Lam 2016-07-29 11:43:23 PDT
Created attachment 284877 [details]
proposed patch.
Comment 2 Mark Lam 2016-07-29 11:49:48 PDT
Thanks for the review.  Landed in r203910: <http://trac.webkit.org/r203910>.