[webkit-reviews] review granted: [Bug 194319] Web Inspector: "Worker not found" uncaught protocol errors : [Attachment 361248] [PATCH] Proposed Fix

[bugzilla-daemon at webkit.org]

Matt Baker <mattbaker at apple.com> has granted Joseph Pecoraro
<joepeck at webkit.org>'s request for review:
Bug 194319: Web Inspector: "Worker not found" uncaught protocol errors
https://bugs.webkit.org/show_bug.cgi?id=194319

Attachment 361248: [PATCH] Proposed Fix

https://bugs.webkit.org/attachment.cgi?id=361248&action=review




--- Comment #2 from Matt Baker <mattbaker at apple.com> ---
Comment on attachment 361248
  --> https://bugs.webkit.org/attachment.cgi?id=361248
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=361248&action=review

r=me

> Source/WebInspectorUI/ChangeLog:12
> +	   frame, but for now this is the only a page transition can kill

Grammar: "the only a page" -> "the only way a page"

> Source/WebInspectorUI/UserInterface/Base/Main.js:232
> +    let workerTargets = WI.targets.filter((x) => x.type ===
WI.Target.Type.Worker);

Nit: rename x to target.