{"issue":{"user":{"id":741528,"login":"arturadib","url":"https://api.github.com/users/arturadib","gravatar_id":"1e79769843eb2b1146834c762ae5f3da","avatar_url":"https://secure.gravatar.com/avatar/1e79769843eb2b1146834c762ae5f3da?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"milestone":null,"title":"Fix worker_loader.js for examples/. Simpler code","created_at":"2011-12-22T14:47:39Z","labels":[],"updated_at":"2011-12-22T14:47:46Z","state":"open","id":2637847,"closed_at":null,"pull_request":{"patch_url":"https://github.com/mozilla/pdf.js/pull/969.patch","html_url":"https://github.com/mozilla/pdf.js/pull/969","diff_url":"https://github.com/mozilla/pdf.js/pull/969.diff"},"html_url":"https://github.com/mozilla/pdf.js/issues/969","body":"Our examples/ were broken due to a path resolution issue in `worker_loader.js`. \r\n\r\nThe spec for `importScripts()` is a [little cryptic](http://www.whatwg.org/specs/web-apps/current-work/multipage/workers.html#importing-scripts-and-libraries) about how it resolves URLs, but both Chrome and Firefox seem to resolve relative to where `worker_loader.js` is, so there's no need to add a `dir` prefix.\r\n\r\nAlso, since we don't need the path we also don't seem to need the `workerSrc` message.\r\n\r\nTested with both Chrome Dev and Firefox Nightly in:\r\n\r\n+ examples/helloworld\r\n+ examples/acroforms\r\n+ web/viewer.html\r\n+ web/viewer-production.html\r\n+ make test\r\n","number":969,"url":"https://api.github.com/repos/mozilla/pdf.js/issues/969","comments":1,"assignee":null},"comment":{"user":{"id":741528,"login":"arturadib","url":"https://api.github.com/users/arturadib","gravatar_id":"1e79769843eb2b1146834c762ae5f3da","avatar_url":"https://secure.gravatar.com/avatar/1e79769843eb2b1146834c762ae5f3da?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"created_at":"2011-12-22T14:47:46Z","updated_at":"2011-12-22T14:47:46Z","id":3249718,"body":"@pdfjsbot test","url":"https://api.github.com/repos/mozilla/pdf.js/issues/comments/3249718"},"action":"created"}