[webkit-reviews] review granted: [Bug 84542] [chromium][workers] setTargetType(ResourceRequest::TargetIsWorker) is repeatedly called in chromium : [Attachment 138740] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 24 23:03:40 PDT 2012


David Levin <levin at chromium.org> has granted Li Yin <li.yin at intel.com>'s
request for review:
Bug 84542: [chromium][workers] setTargetType(ResourceRequest::TargetIsWorker)
is repeatedly called in chromium
https://bugs.webkit.org/show_bug.cgi?id=84542

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

------- Additional Comments from David Levin <levin at chromium.org>
I wonder why
http://code.google.com/searchframe#OAMlx_jo-ck/src/third_party/WebKit/Source/We
bCore/workers/WorkerScriptLoader.cpp&type=cs&l=55 isn't changed to be "||
PLATFORM(BLACKBERRY)" and then just get rid of this.

But this is fine as is.  Thanks!


More information about the webkit-reviews mailing list