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

Unified Diff: android_webview/renderer/aw_render_frame_ext.cc

Issue 2803163002: Move address parser and prefixes to android_webview/. (Closed)
Patch Set: Bring back ContentViewStatics import Created 3 years, 8 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
« no previous file with comments | « android_webview/native/aw_contents_statics.cc ('k') | android_webview/test/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: android_webview/renderer/aw_render_frame_ext.cc
diff --git a/android_webview/renderer/aw_render_frame_ext.cc b/android_webview/renderer/aw_render_frame_ext.cc
index 0b5ba939c0615e7149f294269b497b7a790c17ba..7230acb565cfc24cddfeec9095f300167d46b7ee 100644
--- a/android_webview/renderer/aw_render_frame_ext.cc
+++ b/android_webview/renderer/aw_render_frame_ext.cc
@@ -6,7 +6,6 @@
#include "android_webview/common/render_view_messages.h"
#include "android_webview/renderer/aw_render_frame_ext.h"
#include "base/strings/utf_string_conversions.h"
-#include "content/public/renderer/android_content_detection_prefixes.h"
#include "content/public/renderer/document_state.h"
#include "content/public/renderer/render_frame.h"
#include "content/public/renderer/render_view.h"
@@ -30,6 +29,10 @@ namespace android_webview {
namespace {
+const char kAddressPrefix[] = "geo:0,0?q=";
+const char kEmailPrefix[] = "mailto:";
+const char kPhoneNumberPrefix[] = "tel:";
+
GURL GetAbsoluteUrl(const blink::WebNode& node,
const base::string16& url_fragment) {
return GURL(
@@ -81,13 +84,13 @@ bool RemovePrefixAndAssignIfMatches(const base::StringPiece& prefix,
}
void DistinguishAndAssignSrcLinkType(const GURL& url, AwHitTestData* data) {
- if (RemovePrefixAndAssignIfMatches(content::kAddressPrefix, url,
+ if (RemovePrefixAndAssignIfMatches(kAddressPrefix, url,
&data->extra_data_for_type)) {
data->type = AwHitTestData::GEO_TYPE;
- } else if (RemovePrefixAndAssignIfMatches(content::kPhoneNumberPrefix, url,
+ } else if (RemovePrefixAndAssignIfMatches(kPhoneNumberPrefix, url,
&data->extra_data_for_type)) {
data->type = AwHitTestData::PHONE_TYPE;
- } else if (RemovePrefixAndAssignIfMatches(content::kEmailPrefix, url,
+ } else if (RemovePrefixAndAssignIfMatches(kEmailPrefix, url,
&data->extra_data_for_type)) {
data->type = AwHitTestData::EMAIL_TYPE;
} else {
« no previous file with comments | « android_webview/native/aw_contents_statics.cc ('k') | android_webview/test/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698