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

Unified Diff: third_party/WebKit/Source/modules/accessibility/AXImageMapLink.cpp

Issue 1912863002: Fix image map focus ring painting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
Index: third_party/WebKit/Source/modules/accessibility/AXImageMapLink.cpp
diff --git a/third_party/WebKit/Source/modules/accessibility/AXImageMapLink.cpp b/third_party/WebKit/Source/modules/accessibility/AXImageMapLink.cpp
index 0a9ceba63e2b79fc3575dcde39273e591aa22f2e..d80fa7923f5c2dc7628104ea19e3cd67fbcc3587 100644
--- a/third_party/WebKit/Source/modules/accessibility/AXImageMapLink.cpp
+++ b/third_party/WebKit/Source/modules/accessibility/AXImageMapLink.cpp
@@ -104,21 +104,9 @@ KURL AXImageMapLink::url() const
LayoutRect AXImageMapLink::elementRect() const
{
- HTMLAreaElement* area = areaElement();
- HTMLMapElement* map = mapElement();
pdr. 2016/04/23 03:32:22 We seem to have lost this code which uses the mapE
Xianzhu 2016/04/25 23:18:15 I gave up changing this in this CL. Filed bug 6065
- if (!area || !map)
- return LayoutRect();
-
- LayoutObject* layoutObject;
- if (m_parent && m_parent->isAXLayoutObject())
pdr. 2016/04/23 03:32:23 We no longer have this accessibility code which se
- layoutObject = toAXLayoutObject(m_parent)->getLayoutObject();
- else
- layoutObject = map->layoutObject();
-
- if (!layoutObject)
- return LayoutRect();
-
- return area->computeRect(layoutObject);
+ if (HTMLAreaElement* area = areaElement())
+ return area->computeAbsoluteRect();
+ return LayoutRect();
}
DEFINE_TRACE(AXImageMapLink)

Powered by Google App Engine
This is Rietveld 408576698