Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(46)

Unified Diff: ppapi/tests/test_var.cc

Issue 8826011: Remove PP_Module from parameters for PPB_Var.VarFromUtf8. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix windows build and some nacl tests. Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ppapi/tests/test_var.cc
diff --git a/ppapi/tests/test_var.cc b/ppapi/tests/test_var.cc
index ff899a0fb58a488225f8bbd05f7aef3fe8972107..050fdb97f3eadb7ab5377d0fad81984b7bbb3938 100644
--- a/ppapi/tests/test_var.cc
+++ b/ppapi/tests/test_var.cc
@@ -46,8 +46,7 @@ std::string TestVar::TestBasicString() {
{
const char kStr[] = "Hello";
const uint32_t kStrLen(sizeof(kStr) - 1);
- PP_Var str = var_interface_->VarFromUtf8(pp::Module::Get()->pp_module(),
- kStr, kStrLen);
+ PP_Var str = var_interface_->VarFromUtf8(kStr, kStrLen);
ASSERT_EQ(PP_VARTYPE_STRING, str.type);
// Reading back the string should work.
@@ -122,8 +121,7 @@ std::string TestVar::TestNullInputInUtf8Conversion() {
// 0-length string should not dereference input string, and should produce
// an empty string.
- converted_string = var_interface_->VarFromUtf8(
- pp::Module::Get()->pp_module(), NULL, 0);
+ converted_string = var_interface_->VarFromUtf8(NULL, 0);
if (converted_string.type != PP_VARTYPE_STRING) {
return "Expected 0 length to return empty string.";
}

Powered by Google App Engine
This is Rietveld 408576698