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

Side by Side Diff: third_party/WebKit/Source/core/dom/Document.cpp

Issue 2350433002: Extract more of the mouse logic from EventHandler (Closed)
Patch Set: Remove redundant condition Created 4 years, 2 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org) 5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All r ights reserved. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All r ights reserved.
7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/) 7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/)
8 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved. 8 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
9 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies) 9 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
10 * Copyright (C) Research In Motion Limited 2010-2011. All rights reserved. 10 * Copyright (C) Research In Motion Limited 2010-2011. All rights reserved.
(...skipping 3172 matching lines...) Expand 10 before | Expand all | Expand 10 after
3183 frame = toLocalFrame(frame->tree().parent()); 3183 frame = toLocalFrame(frame->tree().parent());
3184 // Srcdoc documents cannot be top-level documents, by definition, 3184 // Srcdoc documents cannot be top-level documents, by definition,
3185 // because they need to be contained in iframes with the srcdoc. 3185 // because they need to be contained in iframes with the srcdoc.
3186 DCHECK(frame); 3186 DCHECK(frame);
3187 } 3187 }
3188 referrerDocument = frame->document(); 3188 referrerDocument = frame->document();
3189 } 3189 }
3190 return referrerDocument->m_url.strippedForUseAsReferrer(); 3190 return referrerDocument->m_url.strippedForUseAsReferrer();
3191 } 3191 }
3192 3192
3193 MouseEventWithHitTestResults Document::prepareMouseEvent(const HitTestRequest& r equest, const LayoutPoint& documentPoint, const PlatformMouseEvent& event) 3193 MouseEventWithHitTestResults Document::performMouseEventHitTest(const HitTestReq uest& request, const LayoutPoint& documentPoint, const PlatformMouseEvent& event )
3194 { 3194 {
3195 DCHECK(layoutViewItem().isNull() || layoutViewItem().isLayoutView()); 3195 DCHECK(layoutViewItem().isNull() || layoutViewItem().isLayoutView());
3196 3196
3197 // LayoutView::hitTest causes a layout, and we don't want to hit that until the first 3197 // LayoutView::hitTest causes a layout, and we don't want to hit that until the first
3198 // layout because until then, there is nothing shown on the screen - the use r can't 3198 // layout because until then, there is nothing shown on the screen - the use r can't
3199 // have intentionally clicked on something belonging to this page. Furthermo re, 3199 // have intentionally clicked on something belonging to this page. Furthermo re,
3200 // mousemove events before the first layout should not lead to a premature l ayout() 3200 // mousemove events before the first layout should not lead to a premature l ayout()
3201 // happening, which could show a flash of white. 3201 // happening, which could show a flash of white.
3202 // See also the similar code in EventHandler::hitTestResultAtPoint. 3202 // See also the similar code in EventHandler::hitTestResultAtPoint.
3203 if (layoutViewItem().isNull() || !view() || !view()->didFirstLayout()) 3203 if (layoutViewItem().isNull() || !view() || !view()->didFirstLayout())
(...skipping 2881 matching lines...) Expand 10 before | Expand all | Expand 10 after
6085 } 6085 }
6086 6086
6087 void showLiveDocumentInstances() 6087 void showLiveDocumentInstances()
6088 { 6088 {
6089 WeakDocumentSet& set = liveDocumentSet(); 6089 WeakDocumentSet& set = liveDocumentSet();
6090 fprintf(stderr, "There are %u documents currently alive:\n", set.size()); 6090 fprintf(stderr, "There are %u documents currently alive:\n", set.size());
6091 for (Document* document : set) 6091 for (Document* document : set)
6092 fprintf(stderr, "- Document %p URL: %s\n", document, document->url().get String().utf8().data()); 6092 fprintf(stderr, "- Document %p URL: %s\n", document, document->url().get String().utf8().data());
6093 } 6093 }
6094 #endif 6094 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698