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

Unified Diff: Source/web/WebPageSerializerImpl.cpp

Issue 202893002: Replace elementHasLegalLinkAttribute() helper with virtual function (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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: Source/web/WebPageSerializerImpl.cpp
diff --git a/Source/web/WebPageSerializerImpl.cpp b/Source/web/WebPageSerializerImpl.cpp
index 68b0e9d79e00037a713523e566250e63297458bc..b4787c233ef13703146bd1d3e0d98f1fa66dd469 100644
--- a/Source/web/WebPageSerializerImpl.cpp
+++ b/Source/web/WebPageSerializerImpl.cpp
@@ -78,7 +78,6 @@
#include "config.h"
#include "WebPageSerializerImpl.h"
-#include "DOMUtilitiesPrivate.h"
#include "HTMLNames.h"
#include "WebFrameImpl.h"
#include "core/dom/Document.h"
@@ -317,7 +316,7 @@ void WebPageSerializerImpl::openTagToString(Element* element,
// Check whether we need to replace some resource links
// with local resource paths.
const QualifiedName& attrName = attribute.name();
- if (elementHasLegalLinkAttribute(element, attrName)) {
+ if (element->hasLegalLinkAttribute(attrName)) {
// For links start with "javascript:", we do not change it.
if (attrValue.startsWith("javascript:", false))
result.append(attrValue);

Powered by Google App Engine
This is Rietveld 408576698