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

Side by Side Diff: Source/core/dom/MouseRelatedEvent.cpp

Issue 23819007: Have Node::document() return a reference instead of a pointer (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase on master Created 7 years, 3 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
« no previous file with comments | « Source/core/dom/LiveNodeList.cpp ('k') | Source/core/dom/MutationObserverInterestGroup.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2001 Peter Kelly (pmk@post.com) 2 * Copyright (C) 2001 Peter Kelly (pmk@post.com)
3 * Copyright (C) 2001 Tobias Anton (anton@stud.fbi.fh-darmstadt.de) 3 * Copyright (C) 2001 Tobias Anton (anton@stud.fbi.fh-darmstadt.de)
4 * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com) 4 * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
5 * Copyright (C) 2003, 2005, 2006, 2008 Apple Inc. All rights reserved. 5 * Copyright (C) 2003, 2005, 2006, 2008 Apple Inc. All rights reserved.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 { 134 {
135 Node* targetNode = target() ? target()->toNode() : 0; 135 Node* targetNode = target() ? target()->toNode() : 0;
136 if (!targetNode) 136 if (!targetNode)
137 return; 137 return;
138 138
139 // Compute coordinates that are based on the target. 139 // Compute coordinates that are based on the target.
140 m_layerLocation = m_pageLocation; 140 m_layerLocation = m_pageLocation;
141 m_offsetLocation = m_pageLocation; 141 m_offsetLocation = m_pageLocation;
142 142
143 // Must have an updated render tree for this math to work correctly. 143 // Must have an updated render tree for this math to work correctly.
144 targetNode->document()->updateLayoutIgnorePendingStylesheets(); 144 targetNode->document().updateLayoutIgnorePendingStylesheets();
145 145
146 // Adjust offsetLocation to be relative to the target's position. 146 // Adjust offsetLocation to be relative to the target's position.
147 if (RenderObject* r = targetNode->renderer()) { 147 if (RenderObject* r = targetNode->renderer()) {
148 FloatPoint localPos = r->absoluteToLocal(absoluteLocation(), UseTransfor ms); 148 FloatPoint localPos = r->absoluteToLocal(absoluteLocation(), UseTransfor ms);
149 m_offsetLocation = roundedLayoutPoint(localPos); 149 m_offsetLocation = roundedLayoutPoint(localPos);
150 float scaleFactor = 1 / pageZoomFactor(this); 150 float scaleFactor = 1 / pageZoomFactor(this);
151 if (scaleFactor != 1.0f) 151 if (scaleFactor != 1.0f)
152 m_offsetLocation.scale(scaleFactor, scaleFactor); 152 m_offsetLocation.scale(scaleFactor, scaleFactor);
153 } 153 }
154 154
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 } 221 }
222 222
223 int MouseRelatedEvent::y() const 223 int MouseRelatedEvent::y() const
224 { 224 {
225 // FIXME: This is not correct. 225 // FIXME: This is not correct.
226 // See Microsoft documentation and <http://www.quirksmode.org/dom/w3c_events .html>. 226 // See Microsoft documentation and <http://www.quirksmode.org/dom/w3c_events .html>.
227 return m_clientLocation.y(); 227 return m_clientLocation.y();
228 } 228 }
229 229
230 } // namespace WebCore 230 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/core/dom/LiveNodeList.cpp ('k') | Source/core/dom/MutationObserverInterestGroup.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698