Summary: | Web Inspector: [Canvas] do not store a dropped trace log in backend | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andrey Adaikin <aandrey> | ||||
Component: | Web Inspector (Deprecated) | Assignee: | Andrey Adaikin <aandrey> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, webkit.review.bot, yurys | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Andrey Adaikin
2013-02-01 01:56:51 PST
Created attachment 185983 [details]
Patch
Comment on attachment 185983 [details]
Patch
Missing a test
(In reply to comment #2) > (From update of attachment 185983 [details]) > Missing a test hm... this is quite hard to test (injected script), and IMO does not worth testing, in a sense, that the only thing that changed is nulling a pointer upon dropping a trace log. Comment on attachment 185983 [details]
Patch
At this point, I don't think proceeding with no tests at all is acceptable. We need at least some acceptance tests.
Comment on attachment 185983 [details] Patch Clearing flags on attachment: 185983 Committed r141884: <http://trac.webkit.org/changeset/141884> All reviewed patches have been landed. Closing bug. |