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

Unified Diff: ppapi/shared_impl/url_util_impl.h

Issue 6676045: Implement a proxy for URL util. Some of the implementation that doesn't need ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 9 months 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/shared_impl/url_util_impl.h
===================================================================
--- ppapi/shared_impl/url_util_impl.h (revision 0)
+++ ppapi/shared_impl/url_util_impl.h (revision 0)
@@ -0,0 +1,73 @@
+// 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.
+
+#ifndef PPAPI_SHARED_IMPL_URL_UTIL_IMPL_H_
+#define PPAPI_SHARED_IMPL_URL_UTIL_IMPL_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "googleurl/src/url_parse.h"
+#include "ppapi/c/dev/ppb_url_util_dev.h"
+#include "ppapi/c/pp_module.h"
+#include "ppapi/c/pp_var.h"
+
+class GURL;
+
+namespace pp {
+namespace shared_impl {
+
+// Contains the implementation of URLUtil that is shared between the proxy and
yzshen1 2011/03/16 21:19:23 [minor] It seems to me URLUtil is not an existing
+// the renderer.
+class URLUtilImpl {
+ public:
+ // The functions here would normally take the var interface for constructing
+ // return strings. However, at the current time there's some mixup between
+ // using Var and VarDeprecated. To resolve this, we instead pass the pointer
+ // the the string creation function so can be used independently of this.
yzshen1 2011/03/16 21:19:23 the -> to, please.
+ typedef PP_Var (*VarFromUtf8)(PP_Module, const char*, uint32_t);
+
+ // Function that converts the given var to a std::string or NULL if the
+ // var is not a string or is invalid.
+ //
+ // We could use PPB_Var for this, but that interface requires an additional
+ // string conversion. Both the proxy and the host side maintain the strings
+ // in a std::string, and the form we want for passing to GURL is also a
+ // std::string. Parameterizing this separately saves this, and also solves
+ // the same problem that VarFromUtf8 does.
+ typedef const std::string* (*StringFromVar)(PP_Var var);
+
+ // PPB_URLUtil shared functions.
+ static PP_Var Canonicalize(StringFromVar string_from_var,
+ VarFromUtf8 var_from_utf8,
+ PP_Module pp_module,
+ PP_Var url,
+ PP_URLComponents_Dev* components);
+ static PP_Var ResolveRelativeToURL(StringFromVar string_from_var,
+ VarFromUtf8 var_from_utf8,
+ PP_Module pp_module,
+ PP_Var base_url,
+ PP_Var relative_string,
+ PP_URLComponents_Dev* components);
+ static PP_Bool IsSameSecurityOrigin(StringFromVar string_from_var,
+ PP_Var url_a, PP_Var url_b);
+
+ // Converts components from a GoogleUrl parsed to a PPAPI parsed structure.
+ // Output can be NULL to specify "do nothing." This rule is followed by all
+ // the url util functions, so we implement it once here.
+ static void ConvertComponents(const url_parse::Parsed& input,
+ PP_URLComponents_Dev* output);
+
+ // Used for returning the given GURL from a PPAPI function, with an optional
+ // out param indicating the components.
+ static PP_Var GenerateURLReturn(VarFromUtf8 var_from_utf8,
+ PP_Module pp_module,
+ const GURL& url,
+ PP_URLComponents_Dev* components);
+};
+
+} // namespace shared_impl
+} // namespace pp
+
+#endif
Property changes on: ppapi/shared_impl/url_util_impl.h
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698