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

Unified Diff: Source/core/inspector/InspectorCanvasAgent.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 side-by-side diff with in-line comments
Download patch
Index: Source/core/inspector/InspectorCanvasAgent.cpp
diff --git a/Source/core/inspector/InspectorCanvasAgent.cpp b/Source/core/inspector/InspectorCanvasAgent.cpp
index cacd249c29c08c4d6e43b02df41aeef97fdff710..7b34b4ae1352227cf32ec84e4e159ed7e57eb6a2 100644
--- a/Source/core/inspector/InspectorCanvasAgent.cpp
+++ b/Source/core/inspector/InspectorCanvasAgent.cpp
@@ -141,7 +141,7 @@
void InspectorCanvasAgent::captureFrame(ErrorString* errorString, const FrameId* frameId, TraceLogId* traceLogId)
{
- LocalFrame* frame = frameId ? m_pageAgent->assertFrame(errorString, *frameId) : m_pageAgent->inspectedFrame();
+ LocalFrame* frame = frameId ? m_pageAgent->assertFrame(errorString, *frameId) : m_pageAgent->mainFrame();
if (!frame)
return;
InjectedScriptCanvasModule module = injectedScriptCanvasModule(errorString, ScriptState::forMainWorld(frame));
@@ -151,7 +151,7 @@
void InspectorCanvasAgent::startCapturing(ErrorString* errorString, const FrameId* frameId, TraceLogId* traceLogId)
{
- LocalFrame* frame = frameId ? m_pageAgent->assertFrame(errorString, *frameId) : m_pageAgent->inspectedFrame();
+ LocalFrame* frame = frameId ? m_pageAgent->assertFrame(errorString, *frameId) : m_pageAgent->mainFrame();
if (!frame)
return;
InjectedScriptCanvasModule module = injectedScriptCanvasModule(errorString, ScriptState::forMainWorld(frame));
@@ -268,8 +268,8 @@
{
class NodeVisitor final : public WrappedNodeVisitor {
public:
- NodeVisitor(LocalFrame* frame, FramesWithUninstrumentedCanvases& result)
- : m_frame(frame)
+ NodeVisitor(Page* page, FramesWithUninstrumentedCanvases& result)
+ : m_page(page)
, m_framesWithUninstrumentedCanvases(result)
{
}
@@ -281,7 +281,7 @@
return;
LocalFrame* frame = node->document().frame();
- if (frame->localFrameRoot() != m_frame)
+ if (frame->page() != m_page)
return;
if (toHTMLCanvasElement(node)->renderingContext())
@@ -289,9 +289,9 @@
}
private:
- LocalFrame* m_frame;
+ Page* m_page;
FramesWithUninstrumentedCanvases& m_framesWithUninstrumentedCanvases;
- } nodeVisitor(m_pageAgent->inspectedFrame(), m_framesWithUninstrumentedCanvases);
+ } nodeVisitor(m_pageAgent->page(), m_framesWithUninstrumentedCanvases);
m_framesWithUninstrumentedCanvases.clear();
ScriptProfiler::visitNodeWrappers(&nodeVisitor);
@@ -318,7 +318,7 @@
if (!m_enabled)
return;
Frame* frame = loader->frame();
- if (frame == m_pageAgent->inspectedFrame()) {
+ if (frame == m_pageAgent->mainFrame()) {
for (auto& frame : m_framesWithUninstrumentedCanvases)
frame.value = false;
m_frontend->traceLogsRemoved(0, 0);
« no previous file with comments | « Source/core/inspector/InspectorApplicationCacheAgent.cpp ('k') | Source/core/inspector/InspectorController.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698