Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(25)

Side by Side Diff: third_party/WebKit/LayoutTests/fast/canvas/OffscreenCanvas-multiple-worker-commit.html

Issue 2502853003: Make OffscreenCanvas histogram thread safe (Closed)
Patch Set: address comments Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 <!DOCTYPE>
2 <script src="../../resources/testharness.js"></script>
3 <script src="../../resources/testharnessreport.js"></script>
4 <script id="myWorker" type="text/worker">
5 self.onmessage = function(e) {
6 var offscreenCanvas = e.data;
7 var offscreen2d = offscreenCanvas.getContext("2d");
8 offscreen2d.commit();
9 self.postMessage("worker done");
10 };
11 </script>
12 <script>
13 function makeWorker(script) {
14 var blob = new Blob([script]);
15 return new Worker(URL.createObjectURL(blob));
16 }
17
18 // Regression test for crbug.com/663916
19 async_test(function(t) {
20 var canvas1 = document.createElement('canvas');
21 canvas1.width = canvas1.height = 10;
22 var offscreenCanvas1 = canvas1.transferControlToOffscreen();
23
24 var canvas2 = document.createElement('canvas');
25 canvas2.width = canvas2.height = 10;
26 var offscreenCanvas2 = canvas2.transferControlToOffscreen();
27
28 var worker1 = makeWorker(document.getElementById("myWorker").textContent);
Justin Novosad 2016/11/15 20:24:58 Nit: no need for an argument for makeWorker becaus
29 var worker2 = makeWorker(document.getElementById("myWorker").textContent);
30 var returnedWorkerCounter = 0;
31 worker1.onmessage = function (e) {
32 returnedWorkerCounter++;
33 if (returnedWorkerCounter == 2)
34 t.done();
35 };
36 worker2.onmessage = function (e) {
37 returnedWorkerCounter++;
38 if (returnedWorkerCounter == 2)
39 t.done();
40 };
41 worker1.postMessage(offscreenCanvas1, [offscreenCanvas1]);
42 worker2.postMessage(offscreenCanvas2, [offscreenCanvas2]);
43 }, "Calling OffscreenCanvas.commit() from two workers should not crash.");
44
45 </script>
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698