Bug 97331

Summary: Web Inspector: [Canvas] set CanvasAgent in InstrumentingAgents upon calling enable command
Product: WebKit Reporter: Andrey Adaikin <aandrey>
Component: Web Inspector (Deprecated)Assignee: Andrey Adaikin <aandrey>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 97203    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Andrey Adaikin
Reported 2012-09-21 05:43:23 PDT
We should set CanvasAgent in InstrumentingAgents from enable() command and remove from disable(), instead of doing this in constructor and destructor. Patch to follow.
Attachments
Patch (5.51 KB, patch)
2012-09-24 08:19 PDT, Andrey Adaikin 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andrey Adaikin
Comment 1 2012-09-24 08:19:11 PDT
Created attachment 165386 [details] Patch
WebKit Review Bot
Comment 2 2012-09-24 08:47:00 PDT
Comment on attachment 165386 [details] Patch Clearing flags on attachment: 165386 Committed r129374: <http://trac.webkit.org/changeset/129374>
WebKit Review Bot
Comment 3 2012-09-24 08:47:03 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.