udacity/web-workers-demo

Migrate Long Running JS onto a Web Worker

Web Workers Demo

Migrate Long Running JS onto a Web Worker

Relevant Quiz from Browser Rendering Optimization

Relevant solution from Browser Rendering Optimization

Working on the quiz? Start by examining index.html and the JavaScript files linked inside it.

web-workers-demo's Issues

security exception block web workers on Chrome

The improved solution can't work locally on Chrome.
Please warn the user about that and give him workarounds :

example :
npm install http-server -g
cd web-workers-demo
http-server .
127.0.0.1:8080//index-solution-improved.html

Links are deprecated

Links in Read.me give 404's.

ud860 - L4 - JavaScript - Web Workers Quiz: Linked Repo Contains Reference Error in worker.js File

Updated by Susan Smith July 1, 2015.

This is in regard to this web worker repo:
https://github.com/udacity/web-workers-demo/tree/solution/scripts

Which is referenced in the BRO Web Workers Quiz Here:

https://www.udacity.com/course/viewer#!/c-ud860/l-4138168623/e-4184098558/m-4150829139

In worker.js on MASTER branch, the error catching code defines a ManipulatorException but then attempts to throw an "InverterException" - this causes a ReferenceError.

Code:
catch (e) {
function ManipulatorException(message) {
this.name = "ManipulationException";
this.message = message;
};
throw new InverterException('Image manipulation error');
postMessage(undefined);
}

The worker-improved.js in the SOLUTION branch does not have this mismatch:

Code:
catch (e) {
function InverterException(message) {
this.name = "InverterException";
this.message = message;
};
throw new InverterException("Image manipulation error");
postMessage(undefined);
}

Should the error code in worker.js in the MASTER branch be updated to match the inverterException name?