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

Unified Diff: Source/core/paint/DeprecatedPaintLayer.cpp

Issue 1094133004: Image maps should work for images with alt content (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated Created 5 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 | « Source/core/layout/LayoutView.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/paint/DeprecatedPaintLayer.cpp
diff --git a/Source/core/paint/DeprecatedPaintLayer.cpp b/Source/core/paint/DeprecatedPaintLayer.cpp
index e1d9bc4864943ad3ccecd6146362164b0a87b738..79084f08573488803d5507f9feead7c864e1c9ec 100644
--- a/Source/core/paint/DeprecatedPaintLayer.cpp
+++ b/Source/core/paint/DeprecatedPaintLayer.cpp
@@ -2060,14 +2060,14 @@ bool DeprecatedPaintLayer::hitTestContents(HitTestResult& result, const LayoutRe
}
Node* e = enclosingElement();
- if (!result.innerNode())
- result.setInnerNode(e);
-
- // FIXME: missing call to result.setLocalPoint(). What we would really want to do here is to
+ // FIXME: should be a call to result.setNodeAndPosition. What we would really want to do here is to
// return and look for the nearest non-anonymous ancestor, and ignore aunts and uncles on
// our way. It's bad to look for it manually like we do here, and give up on setting a local
// point in the result, because that has bad implications for text selection and
// caretRangeFromPoint(). See crbug.com/461791
+ if (!result.innerNode())
+ result.setInnerNode(e);
+
}
return true;
}
« no previous file with comments | « Source/core/layout/LayoutView.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698