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

Side by Side Diff: third_party/WebKit/Source/core/layout/HitTestResult.cpp

Issue 2389073002: Fix link's hover state if the link under scrollbar (Closed)
Patch Set: fix for bokan's comments Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2008, 2011 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2008, 2011 Apple Inc. All rights reserved.
3 * Copyright (C) 2012 Nokia Corporation and/or its subsidiary(-ies) 3 * Copyright (C) 2012 Nokia Corporation and/or its subsidiary(-ies)
4 * 4 *
5 * This library is free software; you can redistribute it and/or 5 * This library is free software; you can redistribute it and/or
6 * modify it under the terms of the GNU Library General Public 6 * modify it under the terms of the GNU Library General Public
7 * License as published by the Free Software Foundation; either 7 * License as published by the Free Software Foundation; either
8 * version 2 of the License, or (at your option) any later version. 8 * version 2 of the License, or (at your option) any later version.
9 * 9 *
10 * This library is distributed in the hope that it will be useful, 10 * This library is distributed in the hope that it will be useful,
(...skipping 198 matching lines...) Expand 10 before | Expand all | Expand 10 after
209 if (HTMLAreaElement* area = imageAreaForImage()) { 209 if (HTMLAreaElement* area = imageAreaForImage()) {
210 m_innerNode = area; 210 m_innerNode = area;
211 m_innerPossiblyPseudoNode = area; 211 m_innerPossiblyPseudoNode = area;
212 } 212 }
213 } 213 }
214 214
215 void HitTestResult::setURLElement(Element* n) { 215 void HitTestResult::setURLElement(Element* n) {
216 m_innerURLElement = n; 216 m_innerURLElement = n;
217 } 217 }
218 218
219 Scrollbar* HitTestResult::scrollbar() const {
220 return (m_scrollbar.get() && m_scrollbar.get()->enabled()) ? m_scrollbar.get()
bokan 2016/11/02 21:36:34 So I actually found out this already exists in Scr
221 : nullptr;
222 }
223
219 void HitTestResult::setScrollbar(Scrollbar* s) { 224 void HitTestResult::setScrollbar(Scrollbar* s) {
220 m_scrollbar = s; 225 m_scrollbar = s;
221 } 226 }
222 227
223 LocalFrame* HitTestResult::innerNodeFrame() const { 228 LocalFrame* HitTestResult::innerNodeFrame() const {
224 if (m_innerNode) 229 if (m_innerNode)
225 return m_innerNode->document().frame(); 230 return m_innerNode->document().frame();
226 return nullptr; 231 return nullptr;
227 } 232 }
228 233
(...skipping 271 matching lines...) Expand 10 before | Expand all | Expand 10 after
500 else if (isHTMLMapElement(m_innerNode)) 505 else if (isHTMLMapElement(m_innerNode))
501 imageMapImageElement = toHTMLMapElement(m_innerNode)->imageElement(); 506 imageMapImageElement = toHTMLMapElement(m_innerNode)->imageElement();
502 507
503 if (!imageMapImageElement) 508 if (!imageMapImageElement)
504 return m_innerNode.get(); 509 return m_innerNode.get();
505 510
506 return imageMapImageElement; 511 return imageMapImageElement;
507 } 512 }
508 513
509 } // namespace blink 514 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/layout/HitTestResult.h ('k') | third_party/WebKit/Source/core/page/ChromeClient.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698