[webkit-reviews] review granted: [Bug 62469] Fetching a Worker with url that isn't allowed from a file based test causes DRT to crash. : [Attachment 96779] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jun 10 14:21:47 PDT 2011


Dmitry Titov <dimich at chromium.org> has granted David Levin
<levin at chromium.org>'s request for review:
Bug 62469: Fetching a Worker with url that isn't allowed from a file based test
causes DRT to crash.
https://bugs.webkit.org/show_bug.cgi?id=62469

Attachment 96779: Patch
https://bugs.webkit.org/attachment.cgi?id=96779&action=review

------- Additional Comments from Dmitry Titov <dimich at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=96779&action=review

r=me. Nice.

>> Source/WebCore/workers/WorkerScriptLoader.h:44
>> +	class ResourceRequest;
> 
> Code inside a namespace should not be indented.  [whitespace/indent] [4]

Could be fixed as a separate patch...


More information about the webkit-reviews mailing list