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

Unified Diff: ppapi/shared_impl/ppb_url_util_shared.cc

Issue 8826011: Remove PP_Module from parameters for PPB_Var.VarFromUtf8. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge 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
« no previous file with comments | « ppapi/shared_impl/ppb_url_util_shared.h ('k') | ppapi/shared_impl/ppb_var_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/shared_impl/ppb_url_util_shared.cc
diff --git a/ppapi/shared_impl/ppb_url_util_shared.cc b/ppapi/shared_impl/ppb_url_util_shared.cc
index bdc4950fef431cfb9a372a1c80744ec3d1006760..f4735303fa298540fc298d483d3bea4d165e2e05 100644
--- a/ppapi/shared_impl/ppb_url_util_shared.cc
+++ b/ppapi/shared_impl/ppb_url_util_shared.cc
@@ -48,8 +48,7 @@ PP_Var PPB_URLUtil_Shared::Canonicalize(PP_Var url,
StringVar* url_string = StringVar::FromPPVar(url);
if (!url_string)
return PP_MakeNull();
- return GenerateURLReturn(url_string->pp_module(),
- GURL(url_string->value()), components);
+ return GenerateURLReturn(GURL(url_string->value()), components);
}
// static
@@ -65,8 +64,7 @@ PP_Var PPB_URLUtil_Shared::ResolveRelativeToURL(
GURL base_gurl(base_url_string->value());
if (!base_gurl.is_valid())
return PP_MakeNull();
- return GenerateURLReturn(base_url_string->pp_module(),
- base_gurl.Resolve(relative_string->value()),
+ return GenerateURLReturn(base_gurl.Resolve(relative_string->value()),
components);
}
@@ -87,14 +85,12 @@ PP_Bool PPB_URLUtil_Shared::IsSameSecurityOrigin(PP_Var url_a, PP_Var url_b) {
// Used for returning the given GURL from a PPAPI function, with an optional
// out param indicating the components.
-PP_Var PPB_URLUtil_Shared::GenerateURLReturn(
- PP_Module module,
- const GURL& url,
- PP_URLComponents_Dev* components) {
+PP_Var PPB_URLUtil_Shared::GenerateURLReturn(const GURL& url,
+ PP_URLComponents_Dev* components) {
if (!url.is_valid())
return PP_MakeNull();
ConvertComponents(url.parsed_for_possibly_invalid_spec(), components);
- return StringVar::StringToPPVar(module, url.possibly_invalid_spec());
+ return StringVar::StringToPPVar(url.possibly_invalid_spec());
}
PP_Var PPB_URLUtil_Shared::ConvertComponentsAndReturnURL(
« no previous file with comments | « ppapi/shared_impl/ppb_url_util_shared.h ('k') | ppapi/shared_impl/ppb_var_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698