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

Side by Side Diff: Source/core/frame/LocalFrame.cpp

Issue 892693006: Revert of DevTools: use per-LocalFrame instrumenting agents instead of per-Page ones. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
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 | Annotate | Revision Log
« no previous file with comments | « Source/core/frame/LocalFrame.h ('k') | Source/core/inspector/InspectorApplicationCacheAgent.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) 1998, 1999 Torben Weis <weis@kde.org> 2 * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
3 * 1999 Lars Knoll <knoll@kde.org> 3 * 1999 Lars Knoll <knoll@kde.org>
4 * 1999 Antti Koivisto <koivisto@kde.org> 4 * 1999 Antti Koivisto <koivisto@kde.org>
5 * 2000 Simon Hausmann <hausmann@kde.org> 5 * 2000 Simon Hausmann <hausmann@kde.org>
6 * 2000 Stefan Schimanski <1Stein@gmx.de> 6 * 2000 Stefan Schimanski <1Stein@gmx.de>
7 * 2001 George Staikos <staikos@kde.org> 7 * 2001 George Staikos <staikos@kde.org>
8 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ights reserved. 8 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ights reserved.
9 * Copyright (C) 2005 Alexey Proskuryakov <ap@nypop.com> 9 * Copyright (C) 2005 Alexey Proskuryakov <ap@nypop.com>
10 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) 10 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 #include "core/frame/FrameConsole.h" 44 #include "core/frame/FrameConsole.h"
45 #include "core/frame/FrameDestructionObserver.h" 45 #include "core/frame/FrameDestructionObserver.h"
46 #include "core/frame/FrameHost.h" 46 #include "core/frame/FrameHost.h"
47 #include "core/frame/FrameView.h" 47 #include "core/frame/FrameView.h"
48 #include "core/frame/LocalDOMWindow.h" 48 #include "core/frame/LocalDOMWindow.h"
49 #include "core/frame/Settings.h" 49 #include "core/frame/Settings.h"
50 #include "core/html/HTMLFrameElementBase.h" 50 #include "core/html/HTMLFrameElementBase.h"
51 #include "core/html/HTMLPlugInElement.h" 51 #include "core/html/HTMLPlugInElement.h"
52 #include "core/inspector/ConsoleMessageStorage.h" 52 #include "core/inspector/ConsoleMessageStorage.h"
53 #include "core/inspector/InspectorInstrumentation.h" 53 #include "core/inspector/InspectorInstrumentation.h"
54 #include "core/inspector/InstrumentingAgents.h"
55 #include "core/layout/HitTestResult.h" 54 #include "core/layout/HitTestResult.h"
56 #include "core/layout/compositing/RenderLayerCompositor.h" 55 #include "core/layout/compositing/RenderLayerCompositor.h"
57 #include "core/loader/FrameLoaderClient.h" 56 #include "core/loader/FrameLoaderClient.h"
58 #include "core/page/Chrome.h" 57 #include "core/page/Chrome.h"
59 #include "core/page/EventHandler.h" 58 #include "core/page/EventHandler.h"
60 #include "core/page/FocusController.h" 59 #include "core/page/FocusController.h"
61 #include "core/page/Page.h" 60 #include "core/page/Page.h"
62 #include "core/page/scrolling/ScrollingCoordinator.h" 61 #include "core/page/scrolling/ScrollingCoordinator.h"
63 #include "core/paint/TransformRecorder.h" 62 #include "core/paint/TransformRecorder.h"
64 #include "core/rendering/RenderLayer.h" 63 #include "core/rendering/RenderLayer.h"
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 // frameDestroyed() notifications aren't needed. 214 // frameDestroyed() notifications aren't needed.
216 setDOMWindow(nullptr); 215 setDOMWindow(nullptr);
217 216
218 for (const auto& frameDestructionObserver : m_destructionObservers) 217 for (const auto& frameDestructionObserver : m_destructionObservers)
219 frameDestructionObserver->frameDestroyed(); 218 frameDestructionObserver->frameDestroyed();
220 #endif 219 #endif
221 } 220 }
222 221
223 void LocalFrame::trace(Visitor* visitor) 222 void LocalFrame::trace(Visitor* visitor)
224 { 223 {
225 visitor->trace(m_instrumentingAgents);
226 #if ENABLE(OILPAN) 224 #if ENABLE(OILPAN)
227 visitor->trace(m_destructionObservers); 225 visitor->trace(m_destructionObservers);
228 visitor->trace(m_loader); 226 visitor->trace(m_loader);
229 visitor->trace(m_navigationScheduler); 227 visitor->trace(m_navigationScheduler);
230 visitor->trace(m_view); 228 visitor->trace(m_view);
231 visitor->trace(m_domWindow); 229 visitor->trace(m_domWindow);
232 visitor->trace(m_pagePopupOwner); 230 visitor->trace(m_pagePopupOwner);
233 visitor->trace(m_script); 231 visitor->trace(m_script);
234 visitor->trace(m_editor); 232 visitor->trace(m_editor);
235 visitor->trace(m_spellChecker); 233 visitor->trace(m_spellChecker);
(...skipping 183 matching lines...) Expand 10 before | Expand all | Expand 10 after
419 417
420 LocalFrame* LocalFrame::localFrameRoot() 418 LocalFrame* LocalFrame::localFrameRoot()
421 { 419 {
422 LocalFrame* curFrame = this; 420 LocalFrame* curFrame = this;
423 while (curFrame && curFrame->tree().parent() && curFrame->tree().parent()->i sLocalFrame()) 421 while (curFrame && curFrame->tree().parent() && curFrame->tree().parent()->i sLocalFrame())
424 curFrame = toLocalFrame(curFrame->tree().parent()); 422 curFrame = toLocalFrame(curFrame->tree().parent());
425 423
426 return curFrame; 424 return curFrame;
427 } 425 }
428 426
429 InstrumentingAgents* LocalFrame::instrumentingAgents()
430 {
431 return m_instrumentingAgents.get();
432 }
433
434 void LocalFrame::setInstrumentingAgents(InstrumentingAgents* instrumentingAgents )
435 {
436 m_instrumentingAgents = instrumentingAgents;
437 }
438
439 bool LocalFrame::inScope(TreeScope* scope) const 427 bool LocalFrame::inScope(TreeScope* scope) const
440 { 428 {
441 ASSERT(scope); 429 ASSERT(scope);
442 Document* doc = document(); 430 Document* doc = document();
443 if (!doc) 431 if (!doc)
444 return false; 432 return false;
445 // FIXME: This check is broken in for OOPI. 433 // FIXME: This check is broken in for OOPI.
446 HTMLFrameOwnerElement* owner = doc->ownerElement(); 434 HTMLFrameOwnerElement* owner = doc->ownerElement();
447 if (!owner) 435 if (!owner)
448 return false; 436 return false;
(...skipping 371 matching lines...) Expand 10 before | Expand all | Expand 10 after
820 , m_script(ScriptController::create(this)) 808 , m_script(ScriptController::create(this))
821 , m_editor(Editor::create(*this)) 809 , m_editor(Editor::create(*this))
822 , m_spellChecker(SpellChecker::create(*this)) 810 , m_spellChecker(SpellChecker::create(*this))
823 , m_selection(FrameSelection::create(this)) 811 , m_selection(FrameSelection::create(this))
824 , m_eventHandler(adoptPtrWillBeNoop(new EventHandler(this))) 812 , m_eventHandler(adoptPtrWillBeNoop(new EventHandler(this)))
825 , m_console(FrameConsole::create(*this)) 813 , m_console(FrameConsole::create(*this))
826 , m_inputMethodController(InputMethodController::create(*this)) 814 , m_inputMethodController(InputMethodController::create(*this))
827 , m_pageZoomFactor(parentPageZoomFactor(this)) 815 , m_pageZoomFactor(parentPageZoomFactor(this))
828 , m_textZoomFactor(parentTextZoomFactor(this)) 816 , m_textZoomFactor(parentTextZoomFactor(this))
829 , m_inViewSourceMode(false) 817 , m_inViewSourceMode(false)
830 , m_instrumentingAgents(host->instrumentingAgents())
831 { 818 {
832 } 819 }
833 820
834 void LocalFrame::detachView() 821 void LocalFrame::detachView()
835 { 822 {
836 // We detach the FrameView's custom scroll bars as early as 823 // We detach the FrameView's custom scroll bars as early as
837 // possible to prevent m_doc->detach() from messing with the view 824 // possible to prevent m_doc->detach() from messing with the view
838 // such that its scroll bars won't be torn down. 825 // such that its scroll bars won't be torn down.
839 // 826 //
840 // FIXME: We should revisit this. 827 // FIXME: We should revisit this.
841 if (m_view) 828 if (m_view)
842 m_view->prepareForDetach(); 829 m_view->prepareForDetach();
843 } 830 }
844 831
845 } // namespace blink 832 } // namespace blink
OLDNEW
« no previous file with comments | « Source/core/frame/LocalFrame.h ('k') | Source/core/inspector/InspectorApplicationCacheAgent.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698