Expose requireAsync method and make it more robust

extensions/test/data/mojo_private_unittest.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.
 
 'use strict';
 
 let mojoPrivate = require('mojoPrivate').binding;
 let test = require('test').binding;
 let unittestBindings = require('test_environment_specific_bindings');
 
@@ skipping 13 matching lines @@
     }));
     mojoPrivate.define('dependency', test.callbackPass(function() {
       return {result: 12345};
     }));
   },
 
   function testDefineModuleDoesNotExist() {
     mojoPrivate.define('testModule', ['does not exist!'], test.fail);
     test.succeed();
   },
+
+  function testRequireAsync() {
+    console.log('requireAsync');
+    mojoPrivate.requireAsync('mojo/public/js/codec').then(
+        test.callbackPass(function(codec) {
+          test.assertEq('function', typeof codec.Message);
+        }));
+  },
+
+  function testDefineAndRequire() {
+    mojoPrivate.define('testModule', ['dependency'],
+                       test.callbackPass(function(module) {
+      test.assertEq(12345, module.result);
+      mojoPrivate.requireAsync('dependency').then(

[Sam McNally, 2015/01/27 04:06:28]

Does this need to be inside a define()ed module?

[Kevin M, 2015/01/27 18:10:09]

Done.

+          test.succeed,

[Sam McNally, 2015/01/27 04:06:28]

test.callbackPass()

[Kevin M, 2015/01/27 18:10:09]

Done.

+          test.fail);
+    }));
+    mojoPrivate.define('dependency', test.callbackPass(function() {
+      return {result: 12345};
+    }));
+  }
 ], test.runTests, exports);