[NaCl SDK] Fix running .pexe in Release mode.

(NOTE: this fixes the NaCl SDK waterfall)

A bug in common.js was causing the embed to load .pexe with
application/x-nacl, which produces a bizarre failure message:

NaCl module load failed: manifest: program property 'portable' does not have required key: 'url'.

BUG=none
R=noelallen@chromium.org

Review URL: https://codereview.chromium.org/22777002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216889 0039d316-1c4b-4281-b951-d872f2087c98

native_client_sdk/src/examples/common.js

@@ -6,6 +6,10 @@
 // string.
 var isTest = false;
 
+// Set to true when loading a "Release" NaCl module, false when loading a
+// "Debug" NaCl module.
+var isRelease = false;
+
 // Javascript module pattern:
 //   see http://en.wikipedia.org/wiki/Unobtrusive_JavaScript#Namespaces
 // In essence, we define an anonymous function which is immediately called and
@@ -22,11 +26,10 @@ var common = (function() {
    * Return the mime type for NaCl plugin.
    *
    * @param {string} tool The name of the toolchain, e.g. "glibc", "newlib" etc.
-   * @param {bool} isRelease True if this is a release build.
    * @return {string} The mime-type for the kind of NaCl plugin matching
    * the given toolchain.
    */
-  function mimeTypeForTool(tool, isRelease) {
+  function mimeTypeForTool(tool) {
     // For NaCl modules use application/x-nacl.
     var mimetype = 'application/x-nacl';
     if (isHostToolchain(tool)) {
@@ -293,8 +296,8 @@ var common = (function() {
     // status message indicating that the module is still loading.  Otherwise,
     // do not change the status message.
     updateStatus('Page loaded.');
-    var isRelease = path.toLowerCase().indexOf('release') != -1;
-    if (!browserSupportsNaCl(tool, isRelease)) {
+    isRelease = path.toLowerCase().indexOf('release') != -1;
+    if (!browserSupportsNaCl(tool)) {
       updateStatus(
           'Browser does not support NaCl (' + tool + '), or NaCl is disabled');
     } else if (common.naclModule == null) {