Summary: | Fix exception scope verification failures in JSScope.cpp | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Lam <mark.lam> | ||||
Component: | JavaScriptCore | Assignee: | Mark Lam <mark.lam> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, fpizlo, ggaren, jfbastien, keith_miller, msaboff, saam, ysuzuki | ||||
Priority: | P2 | ||||||
Version: | WebKit Local Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 162351 | ||||||
Attachments: |
|
Description
Mark Lam
2016-11-23 09:45:04 PST
Created attachment 295371 [details]
proposed patch.
Comment on attachment 295371 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=295371&action=review > Source/JavaScriptCore/runtime/JSScope.cpp:-244 > - ASSERT_WITH_MESSAGE_UNUSED(throwScope, !throwScope.exception(), "When an exception occurs, the result of isUnscopable becomes false"); Why get rid of this? Comment on attachment 295371 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=295371&action=review >> Source/JavaScriptCore/runtime/JSScope.cpp:-244 >> - ASSERT_WITH_MESSAGE_UNUSED(throwScope, !throwScope.exception(), "When an exception occurs, the result of isUnscopable becomes false"); > > Why get rid of this? Because it is now redundant. The assertion before the "if (!unscopable)" check replaces it. Thanks for the review. Landed in r209028: <http://trac.webkit.org/r209028>. |