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

Side by Side Diff: sky/engine/core/dom/Document.cpp

Issue 949313002: Fix tracing for Document::updateRenderTree (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 10 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
« no previous file with comments | « no previous file | no next file » | 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) 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 952 matching lines...) Expand 10 before | Expand all | Expand 10 after
963 // tear down the render tree or (unfortunately) run script. Kill the whole r enderer if 963 // tear down the render tree or (unfortunately) run script. Kill the whole r enderer if
964 // someone managed to get into here from inside layout or paint. 964 // someone managed to get into here from inside layout or paint.
965 RELEASE_ASSERT(!view()->isInPerformLayout()); 965 RELEASE_ASSERT(!view()->isInPerformLayout());
966 RELEASE_ASSERT(!view()->isPainting()); 966 RELEASE_ASSERT(!view()->isPainting());
967 967
968 // Script can run below in WidgetUpdates, so protect the LocalFrame. 968 // Script can run below in WidgetUpdates, so protect the LocalFrame.
969 // FIXME: Can this still happen? How does script run inside 969 // FIXME: Can this still happen? How does script run inside
970 // UpdateSuspendScope::performDeferredWidgetTreeOperations() ? 970 // UpdateSuspendScope::performDeferredWidgetTreeOperations() ?
971 RefPtr<LocalFrame> protect(m_frame); 971 RefPtr<LocalFrame> protect(m_frame);
972 972
973 TRACE_EVENT_BEGIN0("blink", "Document::updateRenderTree"); 973 TRACE_EVENT0("blink", "Document::updateRenderTree");
974 TRACE_EVENT_SCOPED_SAMPLING_STATE("blink", "UpdateRenderTree");
975 974
976 m_styleRecalcElementCounter = 0; 975 m_styleRecalcElementCounter = 0;
977 976
978 DocumentAnimations::updateOutdatedAnimationPlayersIfNeeded(*this); 977 DocumentAnimations::updateOutdatedAnimationPlayersIfNeeded(*this);
979 evaluateMediaQueryListIfNeeded(); 978 evaluateMediaQueryListIfNeeded();
980 updateDistributionIfNeeded(); 979 updateDistributionIfNeeded();
981 980
982 // FIXME: We should update style on our ancestor chain before proceeding 981 // FIXME: We should update style on our ancestor chain before proceeding
983 // however doing so currently causes several tests to crash, as LocalFrame:: setDocument calls Document::attach 982 // however doing so currently causes several tests to crash, as LocalFrame:: setDocument calls Document::attach
984 // before setting the LocalDOMWindow on the LocalFrame, or the SecurityOrigi n on the document. The attach, in turn 983 // before setting the LocalDOMWindow on the LocalFrame, or the SecurityOrigi n on the document. The attach, in turn
(...skipping 1230 matching lines...) Expand 10 before | Expand all | Expand 10 after
2215 using namespace blink; 2214 using namespace blink;
2216 void showLiveDocumentInstances() 2215 void showLiveDocumentInstances()
2217 { 2216 {
2218 WeakDocumentSet& set = liveDocumentSet(); 2217 WeakDocumentSet& set = liveDocumentSet();
2219 fprintf(stderr, "There are %u documents currently alive:\n", set.size()); 2218 fprintf(stderr, "There are %u documents currently alive:\n", set.size());
2220 for (WeakDocumentSet::const_iterator it = set.begin(); it != set.end(); ++it ) { 2219 for (WeakDocumentSet::const_iterator it = set.begin(); it != set.end(); ++it ) {
2221 fprintf(stderr, "- Document %p URL: %s\n", *it, (*it)->url().string().ut f8().data()); 2220 fprintf(stderr, "- Document %p URL: %s\n", *it, (*it)->url().string().ut f8().data());
2222 } 2221 }
2223 } 2222 }
2224 #endif 2223 #endif
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698