Fixed problem in o3dConverter where it was not outputting all of the

vertex data in JSON format. Fixed problems in effect.js where GLSL
code path needed to name the texture coordinate attributes with the
semantic names and not the names like "diffuseUV" because otherwise
the backend did not know how to hook them up. Fixed problem with cube
map faces being flipped and bug in binding cube map textures.

At this point o3d-webgl can load the converted cube and teapot, though
the teapot is currently missing the bump map. Tested O3D plugin with
this sample to ensure no regression.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/1729012

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

1 files changed, 7 insertions, 2 deletions

diff --git a/o3d/samples/o3d-webgl/client.js b/o3d/samples/o3d-webgl/client.js
index 4e13450..fb54b79 100644
--- a/o3d/samples/o3d-webgl/client.js
+++ b/o3d/samples/o3d-webgl/client.js
@@ -752,7 +752,12 @@ o3d.Client.prototype.clearTickCallback = function() {
  */
 o3d.Client.prototype.setErrorCallback =
     function(error_callback) {
-  this.error_callback = error_callback;
+  // Other code expects to not see a null error callback.
+  if (error_callback) {
+    this.error_callback = error_callback;
+  } else {
+    this.error_callback = function(string) {};
+  }
 };
 
 
@@ -764,7 +769,7 @@ o3d.Client.prototype.setErrorCallback =
  * time or if you call ClearErrorCallback.
  */
 o3d.Client.prototype.clearErrorCallback = function() {
-  this.error_callback = null;
+  this.setErrorCallback(null);
 };