Remove PP_Module from parameters for PPB_Var.VarFromUtf8.

ppapi/native_client/tests/ppapi_test_lib/test_interface.cc

 // Copyright (c) 2011 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.
 
 #include "native_client/tests/ppapi_test_lib/test_interface.h"
 
 #include <string.h>
 #include <map>
 #include <new>
 
@@ skipping 12 matching lines @@
 #include "ppapi/c/ppb_image_data.h"
 #include "ppapi/c/ppb_instance.h"
 #include "ppapi/c/ppb_messaging.h"
 #include "ppapi/c/ppb_var.h"
 #include "ppapi/c/dev/ppb_testing_dev.h"
 
 void PostTestMessage(nacl::string test_name, nacl::string message) {
   nacl::string test_message = test_name;
   test_message += ":";
   test_message += message;
-  PP_Var post_var = PPBVar()->VarFromUtf8(pp_instance(),

[bbudge, 2011/12/07 15:53:28]

This was passing the instance instead of the module?

[dmichael (off chromium), 2011/12/07 18:07:38]

Yep! Fortunately(?) we didn't really use the module for anything, so it was a
harmless bug.

-                                          test_message.c_str(),
+  PP_Var post_var = PPBVar()->VarFromUtf8(test_message.c_str(),
                                           test_message.size());
   PPBMessaging()->PostMessage(pp_instance(), post_var);
   PPBVar()->Release(post_var);
 }
 
 PP_Var PP_MakeString(const char* s) {
-  return PPBVar()->VarFromUtf8(pp_module(), s, strlen(s));
+  return PPBVar()->VarFromUtf8(s, strlen(s));
 }
 
 nacl::string StringifyVar(const PP_Var& var) {
   uint32_t dummy_size;
   switch (var.type) {
     default:
      return "<UNKNOWN>" +  toString(var.type);
     case  PP_VARTYPE_NULL:
       return "<NULL>";
     case  PP_VARTYPE_BOOL:
@@ skipping 61 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 
 namespace {
 
 struct CallbackInfo {
   nacl::string callback_name;
   PP_CompletionCallback user_callback;
 };
 
 void ReportCallbackInvocationToJS(const char* callback_name) {
-  PP_Var callback_var = PPBVar()->VarFromUtf8(pp_module(),
-                                              callback_name,
+  PP_Var callback_var = PPBVar()->VarFromUtf8(callback_name,
                                               strlen(callback_name));
   // Report using postmessage for async tests.
   PPBMessaging()->PostMessage(pp_instance(), callback_var);
   PPBVar()->Release(callback_var);
 }
 
 void CallbackWrapper(void* user_data, int32_t result) {
   CallbackInfo* callback_info = reinterpret_cast<CallbackInfo*>(user_data);
   PP_RunCompletionCallback(&callback_info->user_callback, result);
   ReportCallbackInvocationToJS(callback_info->callback_name.c_str());
@@ skipping 95 matching lines @@
   for (int y = origin.y; y < origin.y + size.height && !found_error; y++) {
     for (int x = origin.x; x < origin.x + size.width && !found_error; x++) {
       uint32_t pixel_color = static_cast<uint32_t*>(bitmap)[stride * y + x];
       found_error = (pixel_color != expected_color);
     }
   }
 
   PPBCore()->ReleaseResource(image);
   return !found_error;
 }