Bug 158353

Summary: Miscellaneous IDBBindingUtilities cleanup
Product: WebKit Reporter: Brady Eidson <beidson>
Component: WebCore Misc.Assignee: Brady Eidson <beidson>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, alecflett, commit-queue, jsbell
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 149117    
Attachments:
Description Flags
Patch thorton: review+

Brady Eidson
Reported 2016-06-03 13:40:43 PDT
Miscellaneous IDBBindingUtilities cleanup
Attachments
Patch (31.03 KB, patch)
2016-06-03 13:51 PDT, Brady Eidson 		thorton: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Brady Eidson
Comment 1 2016-06-03 13:51:23 PDT
Created attachment 280460 [details] Patch
Tim Horton
Comment 2 2016-06-03 14:29:54 PDT
Comment on attachment 280460 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=280460&action=review > Source/WebCore/ChangeLog:8 > + No new tests (Straight refactor, yo). Can't you test the new errors?
Brady Eidson
Comment 3 2016-06-03 14:39:14 PDT
(In reply to comment #2) > Comment on attachment 280460 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=280460&action=review > > > Source/WebCore/ChangeLog:8 > > + No new tests (Straight refactor, yo). > > Can't you test the new errors? Sadly, no. We know "ScriptExecutionContext without an ExecState" is a possibility, somehow, but it's not known to be purposefully triggerable... If I'm wrong and somebody lets me know, I'd happily add tests.
Brady Eidson
Comment 4 2016-06-03 14:41:09 PDT
http://trac.webkit.org/changeset/201660
Note You need to log in before you can comment on or make changes to this bug.