Bug 128516

Summary: [WebGL] Remove state restorer code
Product: WebKit Reporter: Dean Jackson <dino>
Component: WebGLAssignee: Dean Jackson <dino>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, esprehn+autocc, gyuyoung.kim, kondapallykalyan, roger_fong, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch krit: review+

Description Dean Jackson 2014-02-09 21:02:00 PST 
I'm not sure what the point of the WebGLStateRestorer class was. It's always constructed with a false parameter, and then isn't even assigned to a local variable so wouldn't have worked anyway. Meanwhile there are also LOTS of calls to cleanupAfterGraphicsCall, the majority of which pass in false, making them a no-op. I replaced the handful of cleanupAfterGraphicsCall(true) with the call to markContextChanged().
Comment 1 Radar WebKit Bug Importer 2014-02-09 21:02:30 PST 
<rdar://problem/16022970>
Comment 2 Dean Jackson 2014-02-09 21:08:32 PST 
Created attachment 223673 [details]
Patch
Comment 3 Dirk Schulze 2014-02-09 21:26:07 PST 
Comment on attachment 223673 [details]
Patch

Looks reasonable and definitely cleaner. r=me
Comment 4 Dean Jackson 2014-02-09 21:37:37 PST 
Committed r163763: <http://trac.webkit.org/changeset/163763>