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

Side by Side Diff: Source/core/inspector/InspectorConsoleAgent.cpp

Issue 466753002: DevTools: Do not push to frontend messages from worker while it is alive (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Remove unneeded changes Created 6 years, 4 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2011 Google Inc. All rights reserved. 2 * Copyright (C) 2011 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 26 matching lines...) Expand all
37 #include "core/inspector/InspectorConsoleMessage.h" 37 #include "core/inspector/InspectorConsoleMessage.h"
38 #include "core/inspector/InspectorState.h" 38 #include "core/inspector/InspectorState.h"
39 #include "core/inspector/InspectorTimelineAgent.h" 39 #include "core/inspector/InspectorTimelineAgent.h"
40 #include "core/inspector/InspectorTracingAgent.h" 40 #include "core/inspector/InspectorTracingAgent.h"
41 #include "core/inspector/InstrumentingAgents.h" 41 #include "core/inspector/InstrumentingAgents.h"
42 #include "core/inspector/ScriptArguments.h" 42 #include "core/inspector/ScriptArguments.h"
43 #include "core/inspector/ScriptCallFrame.h" 43 #include "core/inspector/ScriptCallFrame.h"
44 #include "core/inspector/ScriptCallStack.h" 44 #include "core/inspector/ScriptCallStack.h"
45 #include "core/loader/DocumentLoader.h" 45 #include "core/loader/DocumentLoader.h"
46 #include "core/page/Page.h" 46 #include "core/page/Page.h"
47 #include "core/workers/WorkerGlobalScopeProxy.h"
47 #include "platform/network/ResourceError.h" 48 #include "platform/network/ResourceError.h"
48 #include "platform/network/ResourceResponse.h" 49 #include "platform/network/ResourceResponse.h"
49 #include "wtf/CurrentTime.h" 50 #include "wtf/CurrentTime.h"
50 #include "wtf/OwnPtr.h" 51 #include "wtf/OwnPtr.h"
51 #include "wtf/PassOwnPtr.h" 52 #include "wtf/PassOwnPtr.h"
52 #include "wtf/text/StringBuilder.h" 53 #include "wtf/text/StringBuilder.h"
53 #include "wtf/text/WTFString.h" 54 #include "wtf/text/WTFString.h"
54 55
55 namespace blink { 56 namespace blink {
56 57
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
170 void InspectorConsoleAgent::addMessageToConsole(ConsoleMessage* consoleMessage) 171 void InspectorConsoleAgent::addMessageToConsole(ConsoleMessage* consoleMessage)
171 { 172 {
172 if (consoleMessage->callStack()) { 173 if (consoleMessage->callStack()) {
173 addConsoleMessage(adoptPtr(new InspectorConsoleMessage(consoleMessage->s ource(), LogMessageType, consoleMessage->level(), consoleMessage->message(), con soleMessage->callStack(), consoleMessage->requestIdentifier()))); 174 addConsoleMessage(adoptPtr(new InspectorConsoleMessage(consoleMessage->s ource(), LogMessageType, consoleMessage->level(), consoleMessage->message(), con soleMessage->callStack(), consoleMessage->requestIdentifier())));
174 } else { 175 } else {
175 bool canGenerateCallStack = !isWorkerAgent() && m_frontend; 176 bool canGenerateCallStack = !isWorkerAgent() && m_frontend;
176 addConsoleMessage(adoptPtr(new InspectorConsoleMessage(canGenerateCallSt ack, consoleMessage->source(), LogMessageType, consoleMessage->level(), consoleM essage->message(), consoleMessage->url(), consoleMessage->lineNumber(), consoleM essage->columnNumber(), consoleMessage->scriptState(), consoleMessage->requestId entifier()))); 177 addConsoleMessage(adoptPtr(new InspectorConsoleMessage(canGenerateCallSt ack, consoleMessage->source(), LogMessageType, consoleMessage->level(), consoleM essage->message(), consoleMessage->url(), consoleMessage->lineNumber(), consoleM essage->columnNumber(), consoleMessage->scriptState(), consoleMessage->requestId entifier())));
177 } 178 }
178 } 179 }
179 180
181 void InspectorConsoleAgent::addMessageToConsoleFromWorker(WorkerGlobalScopeProxy * proxy, ConsoleMessage* consoleMessage)
182 {
183 InspectorConsoleMessage* message = new InspectorConsoleMessage(false, consol eMessage->source(), LogMessageType, consoleMessage->level(), consoleMessage->mes sage(), consoleMessage->url(), consoleMessage->lineNumber(), consoleMessage->col umnNumber(), consoleMessage->scriptState(), consoleMessage->requestIdentifier()) ;
184 message->setWorkerGlobalScopeProxy(proxy);
185 addConsoleMessage(adoptPtr(message));
186 }
187
188 void InspectorConsoleAgent::workerGlobalScopeTerminated(WorkerGlobalScopeProxy* proxy)
189 {
190 for (size_t i = 0; i < m_consoleMessages.size(); i++) {
191 if (m_consoleMessages[i]->getWorkerGlobalScopeProxy() == proxy)
192 m_consoleMessages[i]->setWorkerGlobalScopeProxy(nullptr);
193 }
194 }
195
180 void InspectorConsoleAgent::addConsoleAPIMessageToConsole(MessageType type, Mess ageLevel level, const String& message, ScriptState* scriptState, PassRefPtrWillB eRawPtr<ScriptArguments> arguments, unsigned long requestIdentifier) 196 void InspectorConsoleAgent::addConsoleAPIMessageToConsole(MessageType type, Mess ageLevel level, const String& message, ScriptState* scriptState, PassRefPtrWillB eRawPtr<ScriptArguments> arguments, unsigned long requestIdentifier)
181 { 197 {
182 if (type == ClearMessageType) { 198 if (type == ClearMessageType) {
183 ErrorString error; 199 ErrorString error;
184 clearMessages(&error); 200 clearMessages(&error);
185 } 201 }
186 202
187 addConsoleMessage(adoptPtr(new InspectorConsoleMessage(!isWorkerAgent(), Con soleAPIMessageSource, type, level, message, arguments, scriptState, requestIdent ifier))); 203 addConsoleMessage(adoptPtr(new InspectorConsoleMessage(!isWorkerAgent(), Con soleAPIMessageSource, type, level, message, arguments, scriptState, requestIdent ifier)));
188 } 204 }
189 205
(...skipping 154 matching lines...) Expand 10 before | Expand all | Expand 10 after
344 private: 360 private:
345 int m_heapObjectId; 361 int m_heapObjectId;
346 }; 362 };
347 363
348 void InspectorConsoleAgent::addInspectedHeapObject(ErrorString*, int inspectedHe apObjectId) 364 void InspectorConsoleAgent::addInspectedHeapObject(ErrorString*, int inspectedHe apObjectId)
349 { 365 {
350 m_injectedScriptManager->injectedScriptHost()->addInspectedObject(adoptPtr(n ew InspectableHeapObject(inspectedHeapObjectId))); 366 m_injectedScriptManager->injectedScriptHost()->addInspectedObject(adoptPtr(n ew InspectableHeapObject(inspectedHeapObjectId)));
351 } 367 }
352 368
353 } // namespace blink 369 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698