Instead of ArrayBuffer, pass blob with printerProvider.onPrintRequested

extensions/test/data/api_test/printer_provider/request_print/test.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+function readBlob(blob, callback) {
+  var reader = new FileReader();
+  reader.onerror = function() { callback(null); };
+  reader.onloadend = function() {
+    callback(reader.result);
+  }
+  reader.readAsText(blob)
+}
+
 chrome.test.sendMessage('loaded', function(test) {
   chrome.test.runTests([function printTest() {
     if (test == 'NO_LISTENER') {
       chrome.test.sendMessage('ready');
       chrome.test.succeed();
       return;
     }
 
     chrome.printerProvider.onPrintRequested.addListener(function(job,
                                                                  callback) {
@@ skipping 11 matching lines @@
         return;
       }
 
       if (test == 'INVALID_VALUE') {
         chrome.test.assertThrows(
             callback,
             ['XXX'],
             'Invalid value for argument 1. ' +
             'Value must be one of: ' +
             '[OK, FAILED, INVALID_TICKET, INVALID_DATA].');
+         chrome.test.succeed();
+      } else if (test == 'OK') {

[not at google - send to devlin, 2015/03/04 21:54:50]

Looks odd that the rest of the test uses the pattern

if (test == whatever) {
  something();
  succeed();
  return;
}
if (test == anotherthing) {
  ...
}

but here you're doing

if (test == whatever) {
  something();
  succeed();
} else if (test == anotherthing) {
  ..
}

I think the whole test should use if-else, or better, switch - up to you. Either
way consistent.

[tbarzic, 2015/03/04 23:24:34]

Done.

+        readBlob(job.document, function(content) {
+          callback(!!content ? 'OK' : 'INVALID_DATA');
+
+          if (content)
+            chrome.test.assertEq('bytes', content);
+
+          chrome.test.assertThrows(
+              callback,
+              [test],
+              'Event callback must not be called more than once.');
+          chrome.test.succeed();
+        });
       } else {
-        chrome.test.assertTrue(test == 'OK' || test == 'FAILED' ||
-            test == 'INVALID_TICKET' || test == 'INVALID_DATA');
+        chrome.test.assertTrue(test == 'FAILED' || test == 'INVALID_TICKET' ||
+                               test == 'INVALID_DATA');

[not at google - send to devlin, 2015/03/04 21:54:50]

If you used a switch you could make this look a bit nicer as well:

case 'FAILED':
case 'INVALID_TICKET':
case 'INVALID_DATA':
  ...
  break;
default:
  chrome.test.fail('Invalid input');

[tbarzic, 2015/03/04 23:24:34]

Done.

         callback(test);
+        chrome.test.assertThrows(
+            callback,
+            [test],
+            'Event callback must not be called more than once.');
+        chrome.test.succeed();
       }
 
-      chrome.test.assertThrows(
-          callback,
-          [test],
-          'Event callback must not be called more than once.');
-
-      chrome.test.succeed();
     });
 
     chrome.test.sendMessage('ready');
   }]);
 });