chrome/test/data/nacl/nacl_load_test.html - Issue 10837207: Add NaCl smoke test to browser_tests.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: chrome/test/data/nacl/nacl_load_test.html

Issue 10837207: Add NaCl smoke test to browser_tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Edits Created 8 years, 4 months 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 | Annotate | Revision Log

OLD	NEW
(Empty)
	1 <html>
	Mark Seaborn 2012/08/13 22:16:59 Shouldn't this file have a copyright message? The Shouldn't this file have a copyright message? The codebase doesn't seem to be consistent here, but better to have one. Nick Bray (chromium) 2012/08/13 23:16:26 The cases where we have copyright notices in html Show quoted text On 2012/08/13 22:16:59, Mark Seaborn wrote: > Shouldn't this file have a copyright message? The codebase doesn't seem to be > consistent here, but better to have one. The cases where we have copyright notices in html files are extremely rare. All the same, I added it.
	2 <head>

	3 <title>NaCl Load Test</title>

	4 </head>

	5 <body>

	6 <h2>NaCl Load Test</h2>

	7 </body>

	8 <script>

	9

	10 function report(msg) {

	11 domAutomationController.setAutomationId(0);

	12 // The automation controller seems to choke on Objects, so turn them into

	13 // strings.

	14 domAutomationController.send(JSON.stringify(msg));

	15 }

	16

	17 function create(manifest_url) {

	18 var e = document.createElement("embed");
	Mark Seaborn 2012/08/13 22:16:59 Nit: use single quotes for consistency? Also, ren Nit: use single quotes for consistency? Also, rename 'e' -> 'embed' to avoid cryptic single-char variable names? Nick Bray (chromium) 2012/08/13 23:16:26 Done. Show quoted text On 2012/08/13 22:16:59, Mark Seaborn wrote: > Nit: use single quotes for consistency? Also, rename 'e' -> 'embed' to avoid > cryptic single-char variable names? Done.
	19 e.src = manifest_url;

	20 e.type = "application/x-nacl";

	21

	22 e.addEventListener('load', function(evt) {

	23 report({type: "Shutdown", message: "1 test passed.", passed: true});

	24 }, true);

	25

	26 e.addEventListener('error', function(evt) {

	27 report({type: "Log", message: "Load error: " + e.lastError});

	28 report({type: "Shutdown", message: "1 test failed.", passed: false});

	29 }, true);

	30

	31 document.body.appendChild(e);

	32 }

	33

	34 create("simple.nmf");

	35 </script>

	36 </html>

OLD	NEW

« chrome/test/data/nacl/DEPS ('K') | « chrome/test/data/nacl/OWNERS ('k') | chrome/test/data/nacl/nacl_test_data.gyp » ('j') | chrome/test/data/nacl/nacl_test_data.gyp » ('J')