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

Side by Side Diff: Source/core/rendering/InlineBox.h

Issue 16240007: Fix hit testing to be aware of inert inlines. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: testing style Created 7 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2009, 2010, 2011 Apple Inc. All r ights reserved. 2 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2009, 2010, 2011 Apple Inc. All r ights reserved.
3 * 3 *
4 * This library is free software; you can redistribute it and/or 4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU Library General Public 5 * modify it under the terms of the GNU Library General Public
6 * License as published by the Free Software Foundation; either 6 * License as published by the Free Software Foundation; either
7 * version 2 of the License, or (at your option) any later version. 7 * version 2 of the License, or (at your option) any later version.
8 * 8 *
9 * This library is distributed in the hope that it will be useful, 9 * This library is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 259 matching lines...) Expand 10 before | Expand all | Expand 10 after
270 virtual bool canAccommodateEllipsis(bool ltr, int blockEdge, int ellipsisWid th) const; 270 virtual bool canAccommodateEllipsis(bool ltr, int blockEdge, int ellipsisWid th) const;
271 // visibleLeftEdge, visibleRightEdge are in the parent's coordinate system. 271 // visibleLeftEdge, visibleRightEdge are in the parent's coordinate system.
272 virtual float placeEllipsisBox(bool ltr, float visibleLeftEdge, float visibl eRightEdge, float ellipsisWidth, float &truncatedWidth, bool&); 272 virtual float placeEllipsisBox(bool ltr, float visibleLeftEdge, float visibl eRightEdge, float ellipsisWidth, float &truncatedWidth, bool&);
273 273
274 #ifndef NDEBUG 274 #ifndef NDEBUG
275 void setHasBadParent(); 275 void setHasBadParent();
276 #endif 276 #endif
277 277
278 int expansion() const { return m_bitfields.expansion(); } 278 int expansion() const { return m_bitfields.expansion(); }
279 279
280 bool visibleToHitTestRequest(const HitTestRequest& request) const { return r enderer()->style()->visibility() == VISIBLE && (request.ignorePointerEventsNone( ) || renderer()->style()->pointerEvents() != PE_NONE); } 280 bool visibleToHitTestRequest(const HitTestRequest& request) const { return r enderer()->style()->visibility() == VISIBLE && (request.ignorePointerEventsNone( ) || renderer()->style()->pointerEvents() != PE_NONE) && !renderer()->isInert(); }
Hajime Morrita 2013/06/03 03:28:35 Can we factored out this somehow to getrid of dupl
falken 2013/06/03 03:47:55 Yeah it should be possible.
falken 2013/06/04 04:33:26 Done.
281 281
282 EVerticalAlign verticalAlign() const { return renderer()->style(m_bitfields. firstLine())->verticalAlign(); } 282 EVerticalAlign verticalAlign() const { return renderer()->style(m_bitfields. firstLine())->verticalAlign(); }
283 283
284 // Use with caution! The type is not checked! 284 // Use with caution! The type is not checked!
285 RenderBoxModelObject* boxModelObject() const 285 RenderBoxModelObject* boxModelObject() const
286 { 286 {
287 if (!m_renderer->isText()) 287 if (!m_renderer->isText())
288 return toRenderBoxModelObject(m_renderer); 288 return toRenderBoxModelObject(m_renderer);
289 return 0; 289 return 0;
290 } 290 }
(...skipping 146 matching lines...) Expand 10 before | Expand all | Expand 10 after
437 437
438 } // namespace WebCore 438 } // namespace WebCore
439 439
440 #ifndef NDEBUG 440 #ifndef NDEBUG
441 // Outside the WebCore namespace for ease of invocation from gdb. 441 // Outside the WebCore namespace for ease of invocation from gdb.
442 void showTree(const WebCore::InlineBox*); 442 void showTree(const WebCore::InlineBox*);
443 void showLineTree(const WebCore::InlineBox*); 443 void showLineTree(const WebCore::InlineBox*);
444 #endif 444 #endif
445 445
446 #endif // InlineBox_h 446 #endif // InlineBox_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698