{"comment":{"created_at":"2011-10-08T21:15:49Z","body":"Processing command **test** by user _jviereck_. Queue size: 0\n\nLive script output is available at: http://184.73.87.52:8989/2333905.txt\n\n[bot:processed:2333905]","updated_at":"2011-10-08T21:15:49Z","url":"https://api.github.com/repos/andreasgal/pdf.js/issues/comments/2333908","id":2333908,"user":{"avatar_url":"https://secure.gravatar.com/avatar/566da92c81e5d7bb420c76a0bb966ac9?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/pdfjsbot","id":1036361,"login":"pdfjsbot"}},"action":"created","issue":{"number":635,"created_at":"2011-10-08T21:14:13Z","pull_request":{"diff_url":"https://github.com/andreasgal/pdf.js/pull/635.diff","patch_url":"https://github.com/andreasgal/pdf.js/pull/635.patch","html_url":"https://github.com/andreasgal/pdf.js/pull/635"},"body":"This is a merged version of the worker branch that hopefully works. After the merge, the fonts weren't working for some pages so I've redone all the font changes, started from a scratch version of fonts.js from master again an build from there. The font's are currently processed (new Font(...)) on the main thread and not in a worker as well, but let's keep it simple for now and get something landed.\r\n\r\nCompared to the wish list from Chris on the other pull request, I've addressed some of the points, but some still stay opened:\r\n\r\n* The worker functionality is disabled right now as the feature detection needs to get implemented\r\n* The \"can we use UInt8Array as image data\" isn't implemented due to missing feature detection\r\n* The PDFDoc and WorkerPDFDoc as well as the Page and WorkerPage object haven't merge yet, as this requires a complete refactoring of these objects and I don't want to break more stuff/refactore more code then I've already done with these patches. However, note that the current `PDFDoc` got renamed to `PDFDocModel` and `WorkerPDFDoc` is now `PDFDoc`. That way, we don't have to change any of the APIs as you just can call the new PDFDoc as you did before.\r\n\r\nLet's see what the bot says to this patches. Can someone please take a look at the last changes if they make sense?\r\n","comments":2,"title":"Worker implementation #3","updated_at":"2011-10-08T21:15:49Z","url":"https://api.github.com/repos/andreasgal/pdf.js/issues/635","id":1853919,"assignee":null,"milestone":null,"closed_at":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/15354055d20592036cf71d436762ee5b?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/jviereck","id":191719,"login":"jviereck"},"html_url":"https://github.com/andreasgal/pdf.js/issues/635","labels":[],"state":"open"},"legacy":{"issue_id":1853919,"comment_id":2333908}}