<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[173251] trunk/Source</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/173251">173251</a></dd>
<dt>Author</dt> <dd>burg@cs.washington.edu</dd>
<dt>Date</dt> <dd>2014-09-03 22:53:43 -0700 (Wed, 03 Sep 2014)</dd>
</dl>
<h3>Log Message</h3>
<pre>Web Inspector: fix prefixes for subclasses of JSC::ConsoleClient
https://bugs.webkit.org/show_bug.cgi?id=136476
Reviewed by Timothy Hatcher.
Source/JavaScriptCore:
* CMakeLists.txt:
* JavaScriptCore.xcodeproj/project.pbxproj:
* inspector/JSGlobalObjectConsoleClient.cpp: Renamed from Source/JavaScriptCore/inspector/JSConsoleClient.cpp.
* inspector/JSGlobalObjectConsoleClient.h: Renamed from Source/JavaScriptCore/inspector/JSConsoleClient.h.
* inspector/JSGlobalObjectInspectorController.cpp:
(Inspector::JSGlobalObjectInspectorController::JSGlobalObjectInspectorController):
(Inspector::JSGlobalObjectInspectorController::reportAPIException):
* inspector/JSGlobalObjectInspectorController.h:
Source/WebCore:
No new tests. No behavior changed.
* CMakeLists.txt:
* WebCore.exp.in:
* WebCore.order:
* WebCore.vcxproj/WebCore.vcxproj:
* WebCore.vcxproj/WebCore.vcxproj.filters:
* WebCore.xcodeproj/project.pbxproj:
* bindings/js/JSCustomXPathNSResolver.cpp:
(WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):
* bindings/js/ScriptCachedFrameData.cpp:
(WebCore::ScriptCachedFrameData::restore):
* bindings/js/ScriptController.cpp:
(WebCore::ScriptController::clearWindowShell):
(WebCore::ScriptController::initScript):
* css/CSSParser.cpp:
(WebCore::CSSParser::logError):
* dom/Document.cpp:
(WebCore::Document::addConsoleMessage):
(WebCore::Document::addMessage):
* inspector/PageDebuggerAgent.cpp:
(WebCore::PageDebuggerAgent::muteConsole):
(WebCore::PageDebuggerAgent::unmuteConsole):
(WebCore::PageDebuggerAgent::breakpointActionLog):
* inspector/PageRuntimeAgent.cpp:
(WebCore::PageRuntimeAgent::muteConsole):
(WebCore::PageRuntimeAgent::unmuteConsole):
* page/DOMWindow.cpp:
(WebCore::DOMWindow::console):
(WebCore::DOMWindow::dispatchMessageEventWithOriginCheck):
(WebCore::DOMWindow::close):
(WebCore::DOMWindow::printErrorMessage):
(WebCore::DOMWindow::pageConsole): Deleted.
* page/DOMWindow.h:
* page/Page.cpp:
(WebCore::Page::Page):
* page/Page.h:
* page/PageConsoleClient.cpp: Renamed from Source/WebCore/page/PageConsole.cpp.
* page/PageConsoleClient.h: Renamed from Source/WebCore/page/PageConsole.h.
* testing/Internals.cpp:
(WebCore::Internals::consoleProfiles):
* testing/Internals.h:
* xml/XSLStyleSheetLibxslt.cpp:
(WebCore::XSLStyleSheet::parseString):
* xml/XSLTProcessorLibxslt.cpp:
(WebCore::XSLTProcessor::parseErrorFunc):
(WebCore::docLoaderFunc):
Source/WebKit/mac:
* Misc/WebCoreStatistics.mm:
(+[WebCoreStatistics shouldPrintExceptions]):
(+[WebCoreStatistics setShouldPrintExceptions:]):
Source/WebKit/win:
* WebCoreStatistics.cpp:
(WebCoreStatistics::shouldPrintExceptions):
(WebCoreStatistics::setShouldPrintExceptions):</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceJavaScriptCoreCMakeListstxt">trunk/Source/JavaScriptCore/CMakeLists.txt</a></li>
<li><a href="#trunkSourceJavaScriptCoreChangeLog">trunk/Source/JavaScriptCore/ChangeLog</a></li>
<li><a href="#trunkSourceJavaScriptCoreJavaScriptCorexcodeprojprojectpbxproj">trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceJavaScriptCoreinspectorJSGlobalObjectInspectorControllercpp">trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreinspectorJSGlobalObjectInspectorControllerh">trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.h</a></li>
<li><a href="#trunkSourceWebCoreCMakeListstxt">trunk/Source/WebCore/CMakeLists.txt</a></li>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreWebCoreexpin">trunk/Source/WebCore/WebCore.exp.in</a></li>
<li><a href="#trunkSourceWebCoreWebCoreorder">trunk/Source/WebCore/WebCore.order</a></li>
<li><a href="#trunkSourceWebCoreWebCorevcxprojWebCorevcxproj">trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj</a></li>
<li><a href="#trunkSourceWebCoreWebCorevcxprojWebCorevcxprojfilters">trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters</a></li>
<li><a href="#trunkSourceWebCoreWebCorexcodeprojprojectpbxproj">trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceWebCorebindingsjsJSCustomXPathNSResolvercpp">trunk/Source/WebCore/bindings/js/JSCustomXPathNSResolver.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsjsScriptCachedFrameDatacpp">trunk/Source/WebCore/bindings/js/ScriptCachedFrameData.cpp</a></li>
<li><a href="#trunkSourceWebCorebindingsjsScriptControllercpp">trunk/Source/WebCore/bindings/js/ScriptController.cpp</a></li>
<li><a href="#trunkSourceWebCorecssCSSParsercpp">trunk/Source/WebCore/css/CSSParser.cpp</a></li>
<li><a href="#trunkSourceWebCoredomDocumentcpp">trunk/Source/WebCore/dom/Document.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorPageDebuggerAgentcpp">trunk/Source/WebCore/inspector/PageDebuggerAgent.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorPageRuntimeAgentcpp">trunk/Source/WebCore/inspector/PageRuntimeAgent.cpp</a></li>
<li><a href="#trunkSourceWebCorepageDOMWindowcpp">trunk/Source/WebCore/page/DOMWindow.cpp</a></li>
<li><a href="#trunkSourceWebCorepageDOMWindowh">trunk/Source/WebCore/page/DOMWindow.h</a></li>
<li><a href="#trunkSourceWebCorepagePagecpp">trunk/Source/WebCore/page/Page.cpp</a></li>
<li><a href="#trunkSourceWebCorepagePageh">trunk/Source/WebCore/page/Page.h</a></li>
<li><a href="#trunkSourceWebCoretestingInternalsh">trunk/Source/WebCore/testing/Internals.h</a></li>
<li><a href="#trunkSourceWebCorexmlXSLStyleSheetLibxsltcpp">trunk/Source/WebCore/xml/XSLStyleSheetLibxslt.cpp</a></li>
<li><a href="#trunkSourceWebCorexmlXSLTProcessorLibxsltcpp">trunk/Source/WebCore/xml/XSLTProcessorLibxslt.cpp</a></li>
<li><a href="#trunkSourceWebKitmacChangeLog">trunk/Source/WebKit/mac/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitmacMiscWebCoreStatisticsmm">trunk/Source/WebKit/mac/Misc/WebCoreStatistics.mm</a></li>
<li><a href="#trunkSourceWebKitwinChangeLog">trunk/Source/WebKit/win/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitwinWebCoreStatisticscpp">trunk/Source/WebKit/win/WebCoreStatistics.cpp</a></li>
</ul>
<h3>Added Paths</h3>
<ul>
<li><a href="#trunkSourceJavaScriptCoreinspectorJSGlobalObjectConsoleClientcpp">trunk/Source/JavaScriptCore/inspector/JSGlobalObjectConsoleClient.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreinspectorJSGlobalObjectConsoleClienth">trunk/Source/JavaScriptCore/inspector/JSGlobalObjectConsoleClient.h</a></li>
<li><a href="#trunkSourceWebCorepagePageConsoleClientcpp">trunk/Source/WebCore/page/PageConsoleClient.cpp</a></li>
<li><a href="#trunkSourceWebCorepagePageConsoleClienth">trunk/Source/WebCore/page/PageConsoleClient.h</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li><a href="#trunkSourceJavaScriptCoreinspectorJSConsoleClientcpp">trunk/Source/JavaScriptCore/inspector/JSConsoleClient.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreinspectorJSConsoleClienth">trunk/Source/JavaScriptCore/inspector/JSConsoleClient.h</a></li>
<li><a href="#trunkSourceWebCorepagePageConsolecpp">trunk/Source/WebCore/page/PageConsole.cpp</a></li>
<li><a href="#trunkSourceWebCorepagePageConsoleh">trunk/Source/WebCore/page/PageConsole.h</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceJavaScriptCoreCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/CMakeLists.txt (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/CMakeLists.txt 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/JavaScriptCore/CMakeLists.txt 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -283,9 +283,9 @@
</span><span class="cx"> inspector/InspectorAgentRegistry.cpp
</span><span class="cx"> inspector/InspectorBackendDispatcher.cpp
</span><span class="cx"> inspector/InspectorValues.cpp
</span><del>- inspector/JSConsoleClient.cpp
</del><span class="cx"> inspector/JSInjectedScriptHost.cpp
</span><span class="cx"> inspector/JSInjectedScriptHostPrototype.cpp
</span><ins>+ inspector/JSGlobalObjectConsoleClient.cpp
</ins><span class="cx"> inspector/JSGlobalObjectInspectorController.cpp
</span><span class="cx"> inspector/JSGlobalObjectScriptDebugServer.cpp
</span><span class="cx"> inspector/JSJavaScriptCallFrame.cpp
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ChangeLog (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ChangeLog 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/JavaScriptCore/ChangeLog 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1,3 +1,19 @@
</span><ins>+2014-09-02 Brian J. Burg <burg@cs.washington.edu>
+
+ Web Inspector: fix prefixes for subclasses of JSC::ConsoleClient
+ https://bugs.webkit.org/show_bug.cgi?id=136476
+
+ Reviewed by Timothy Hatcher.
+
+ * CMakeLists.txt:
+ * JavaScriptCore.xcodeproj/project.pbxproj:
+ * inspector/JSGlobalObjectConsoleClient.cpp: Renamed from Source/JavaScriptCore/inspector/JSConsoleClient.cpp.
+ * inspector/JSGlobalObjectConsoleClient.h: Renamed from Source/JavaScriptCore/inspector/JSConsoleClient.h.
+ * inspector/JSGlobalObjectInspectorController.cpp:
+ (Inspector::JSGlobalObjectInspectorController::JSGlobalObjectInspectorController):
+ (Inspector::JSGlobalObjectInspectorController::reportAPIException):
+ * inspector/JSGlobalObjectInspectorController.h:
+
</ins><span class="cx"> 2014-09-03 Filip Pizlo <fpizlo@apple.com>
</span><span class="cx">
</span><span class="cx"> Reenable call edge profiling and polymorphic call inlining, now that a bunch of the bugs
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreJavaScriptCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1426,8 +1426,8 @@
</span><span class="cx"> A5BA15ED182340B400A82E69 /* RemoteInspectorXPCConnection.h in Headers */ = {isa = PBXBuildFile; fileRef = A5BA15E6182340B300A82E69 /* RemoteInspectorXPCConnection.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="cx"> A5BA15EE182340B400A82E69 /* RemoteInspectorXPCConnection.mm in Sources */ = {isa = PBXBuildFile; fileRef = A5BA15E7182340B300A82E69 /* RemoteInspectorXPCConnection.mm */; };
</span><span class="cx"> A5BA15F0182345AF00A82E69 /* RemoteInspectorDebuggable.h in Headers */ = {isa = PBXBuildFile; fileRef = A5BA15EF182345AF00A82E69 /* RemoteInspectorDebuggable.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><del>- A5C3A1A518C0490200C9593A /* JSConsoleClient.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A5C3A1A318C0490200C9593A /* JSConsoleClient.cpp */; };
- A5C3A1A618C0490200C9593A /* JSConsoleClient.h in Headers */ = {isa = PBXBuildFile; fileRef = A5C3A1A418C0490200C9593A /* JSConsoleClient.h */; };
</del><ins>+ A5C3A1A518C0490200C9593A /* JSGlobalObjectConsoleClient.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A5C3A1A318C0490200C9593A /* JSGlobalObjectConsoleClient.cpp */; };
+ A5C3A1A618C0490200C9593A /* JSGlobalObjectConsoleClient.h in Headers */ = {isa = PBXBuildFile; fileRef = A5C3A1A418C0490200C9593A /* JSGlobalObjectConsoleClient.h */; };
</ins><span class="cx"> A5CEEE14187F3BAD00E55C99 /* InspectorAgent.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A5CEEE12187F3BAD00E55C99 /* InspectorAgent.cpp */; };
</span><span class="cx"> A5D0A1BB1862301B00C7B496 /* InspectorEnvironment.h in Headers */ = {isa = PBXBuildFile; fileRef = A5D0A1BA1862301B00C7B496 /* InspectorEnvironment.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="cx"> A5D2E665195E174000A518E7 /* JSContextRefInternal.h in Headers */ = {isa = PBXBuildFile; fileRef = A5D2E664195E173800A518E7 /* JSContextRefInternal.h */; };
</span><span class="lines">@@ -3092,8 +3092,8 @@
</span><span class="cx"> A5BA15E6182340B300A82E69 /* RemoteInspectorXPCConnection.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = RemoteInspectorXPCConnection.h; sourceTree = "<group>"; };
</span><span class="cx"> A5BA15E7182340B300A82E69 /* RemoteInspectorXPCConnection.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = RemoteInspectorXPCConnection.mm; sourceTree = "<group>"; };
</span><span class="cx"> A5BA15EF182345AF00A82E69 /* RemoteInspectorDebuggable.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = RemoteInspectorDebuggable.h; sourceTree = "<group>"; };
</span><del>- A5C3A1A318C0490200C9593A /* JSConsoleClient.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSConsoleClient.cpp; sourceTree = "<group>"; };
- A5C3A1A418C0490200C9593A /* JSConsoleClient.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSConsoleClient.h; sourceTree = "<group>"; };
</del><ins>+ A5C3A1A318C0490200C9593A /* JSGlobalObjectConsoleClient.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSGlobalObjectConsoleClient.cpp; sourceTree = "<group>"; };
+ A5C3A1A418C0490200C9593A /* JSGlobalObjectConsoleClient.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSGlobalObjectConsoleClient.h; sourceTree = "<group>"; };
</ins><span class="cx"> A5CEEE12187F3BAD00E55C99 /* InspectorAgent.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = InspectorAgent.cpp; sourceTree = "<group>"; };
</span><span class="cx"> A5CEEE13187F3BAD00E55C99 /* InspectorAgent.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = InspectorAgent.h; sourceTree = "<group>"; };
</span><span class="cx"> A5D0A1BA1862301B00C7B496 /* InspectorEnvironment.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = InspectorEnvironment.h; sourceTree = "<group>"; };
</span><span class="lines">@@ -5388,8 +5388,8 @@
</span><span class="cx"> A593CF811840377100BFCE27 /* InspectorValues.h */,
</span><span class="cx"> A503FA13188E0FAF00110F14 /* JavaScriptCallFrame.cpp */,
</span><span class="cx"> A503FA14188E0FAF00110F14 /* JavaScriptCallFrame.h */,
</span><del>- A5C3A1A318C0490200C9593A /* JSConsoleClient.cpp */,
- A5C3A1A418C0490200C9593A /* JSConsoleClient.h */,
</del><ins>+ A5C3A1A318C0490200C9593A /* JSGlobalObjectConsoleClient.cpp */,
+ A5C3A1A418C0490200C9593A /* JSGlobalObjectConsoleClient.h */,
</ins><span class="cx"> A51007BE187CC3C600B38879 /* JSGlobalObjectInspectorController.cpp */,
</span><span class="cx"> A51007BF187CC3C600B38879 /* JSGlobalObjectInspectorController.h */,
</span><span class="cx"> A503FA27188F105900110F14 /* JSGlobalObjectScriptDebugServer.cpp */,
</span><span class="lines">@@ -5878,7 +5878,7 @@
</span><span class="cx"> 0FFB921816D02EB20055A5DB /* DFGAllocator.h in Headers */,
</span><span class="cx"> A737810C1799EA2E00817533 /* DFGAnalysis.h in Headers */,
</span><span class="cx"> 0F1E3A461534CBAF000F9456 /* DFGArgumentPosition.h in Headers */,
</span><del>- A5C3A1A618C0490200C9593A /* JSConsoleClient.h in Headers */,
</del><ins>+ A5C3A1A618C0490200C9593A /* JSGlobalObjectConsoleClient.h in Headers */,
</ins><span class="cx"> 0F16015E156198C900C2587C /* DFGArgumentsSimplificationPhase.h in Headers */,
</span><span class="cx"> 0F485322187750560083B687 /* DFGArithMode.h in Headers */,
</span><span class="cx"> 0F05C3B41683CF9200BAF45B /* DFGArrayifySlowPathGenerator.h in Headers */,
</span><span class="lines">@@ -7268,7 +7268,7 @@
</span><span class="cx"> A704D90517A0BAA8006BA554 /* DFGInPlaceAbstractState.cpp in Sources */,
</span><span class="cx"> 0F300B7B18AB1B1400A6D72E /* DFGIntegerCheckCombiningPhase.cpp in Sources */,
</span><span class="cx"> 0FC97F3D18202119002C9B26 /* DFGInvalidationPointInjectionPhase.cpp in Sources */,
</span><del>- A5C3A1A518C0490200C9593A /* JSConsoleClient.cpp in Sources */,
</del><ins>+ A5C3A1A518C0490200C9593A /* JSGlobalObjectConsoleClient.cpp in Sources */,
</ins><span class="cx"> 0FEA0A33170D40BF00BB722C /* DFGJITCode.cpp in Sources */,
</span><span class="cx"> 86EC9DCB1328DF82002B2AD7 /* DFGJITCompiler.cpp in Sources */,
</span><span class="cx"> A78A9778179738B8009DF744 /* DFGJITFinalizer.cpp in Sources */,
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorJSConsoleClientcpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/inspector/JSConsoleClient.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/JSConsoleClient.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/JavaScriptCore/inspector/JSConsoleClient.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1,130 +0,0 @@
</span><del>-/*
- * Copyright (C) 2014 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR
- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
- * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
- * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
- * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "JSConsoleClient.h"
-
-#if ENABLE(INSPECTOR)
-
-#include "InspectorConsoleAgent.h"
-#include "ScriptArguments.h"
-#include "ScriptCallStack.h"
-#include "ScriptCallStackFactory.h"
-
-#if USE(CF)
-#include <CoreFoundation/CoreFoundation.h>
-#endif
-
-using namespace JSC;
-
-namespace Inspector {
-
-static bool sLogToSystemConsole = false;
-
-bool JSConsoleClient::logToSystemConsole()
-{
- return sLogToSystemConsole;
-}
-
-void JSConsoleClient::setLogToSystemConsole(bool shouldLog)
-{
- sLogToSystemConsole = shouldLog;
-}
-
-void JSConsoleClient::initializeLogToSystemConsole()
-{
-#if !LOG_DISABLED
- sLogToSystemConsole = true;
-#elif USE(CF)
- Boolean keyExistsAndHasValidFormat = false;
- Boolean preference = CFPreferencesGetAppBooleanValue(CFSTR("JavaScriptCoreOutputConsoleMessagesToSystemConsole"), kCFPreferencesCurrentApplication, &keyExistsAndHasValidFormat);
- if (keyExistsAndHasValidFormat)
- sLogToSystemConsole = preference;
-#endif
-}
-
-JSConsoleClient::JSConsoleClient(InspectorConsoleAgent* consoleAgent)
- : ConsoleClient()
- , m_consoleAgent(consoleAgent)
-{
- static std::once_flag initializeLogging;
- std::call_once(initializeLogging, []{
- JSConsoleClient::initializeLogToSystemConsole();
- });
-}
-
-void JSConsoleClient::messageWithTypeAndLevel(MessageType type, MessageLevel level, JSC::ExecState* exec, PassRefPtr<ScriptArguments> prpArguments)
-{
- RefPtr<ScriptArguments> arguments = prpArguments;
-
- if (JSConsoleClient::logToSystemConsole())
- ConsoleClient::printConsoleMessageWithArguments(MessageSource::ConsoleAPI, type, level, exec, arguments);
-
- String message;
- arguments->getFirstArgumentAsString(message);
- m_consoleAgent->addMessageToConsole(MessageSource::ConsoleAPI, type, level, message, exec, arguments.release());
-}
-
-void JSConsoleClient::count(ExecState* exec, PassRefPtr<ScriptArguments> arguments)
-{
- m_consoleAgent->count(exec, arguments);
-}
-
-void JSConsoleClient::profile(JSC::ExecState*, const String&)
-{
- // FIXME: support |console.profile| for JSContexts. <https://webkit.org/b/136466>
-}
-
-void JSConsoleClient::profileEnd(JSC::ExecState*, const String&)
-{
- // FIXME: support |console.profile| for JSContexts. <https://webkit.org/b/136466>
-}
-
-void JSConsoleClient::time(ExecState*, const String& title)
-{
- m_consoleAgent->startTiming(title);
-}
-
-void JSConsoleClient::timeEnd(ExecState* exec, const String& title)
-{
- RefPtr<ScriptCallStack> callStack(createScriptCallStackForConsole(exec, 1));
- m_consoleAgent->stopTiming(title, callStack.release());
-}
-
-void JSConsoleClient::timeStamp(ExecState*, PassRefPtr<ScriptArguments>)
-{
- // FIXME: JSContext inspection needs a timeline.
- warnUnimplemented(ASCIILiteral("console.timeStamp"));
-}
-
-void JSConsoleClient::warnUnimplemented(const String& method)
-{
- String message = method + " is currently ignored in JavaScript context inspection.";
- m_consoleAgent->addMessageToConsole(MessageSource::ConsoleAPI, MessageType::Log, MessageLevel::Warning, message, nullptr, nullptr);
-}
-
-} // namespace Inspector
-
-#endif
</del></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorJSConsoleClienth"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/inspector/JSConsoleClient.h (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/JSConsoleClient.h 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/JavaScriptCore/inspector/JSConsoleClient.h 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1,63 +0,0 @@
</span><del>-/*
- * Copyright (C) 2014 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR
- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
- * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
- * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
- * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef JSConsoleClient_h
-#define JSConsoleClient_h
-
-#include "ConsoleClient.h"
-
-namespace Inspector {
-
-class InspectorConsoleAgent;
-
-class JSConsoleClient final : public JSC::ConsoleClient {
- WTF_MAKE_FAST_ALLOCATED;
-public:
- explicit JSConsoleClient(InspectorConsoleAgent*);
- virtual ~JSConsoleClient() { }
-
- static bool logToSystemConsole();
- static void setLogToSystemConsole(bool);
- static void initializeLogToSystemConsole();
-
-protected:
- virtual void messageWithTypeAndLevel(MessageType, MessageLevel, JSC::ExecState*, PassRefPtr<ScriptArguments>) override;
- virtual void count(JSC::ExecState*, PassRefPtr<ScriptArguments>) override;
- virtual void profile(JSC::ExecState*, const String& title) override;
- virtual void profileEnd(JSC::ExecState*, const String& title) override;
- virtual void time(JSC::ExecState*, const String& title) override;
- virtual void timeEnd(JSC::ExecState*, const String& title) override;
- virtual void timeStamp(JSC::ExecState*, PassRefPtr<ScriptArguments>) override;
-
-private:
- void warnUnimplemented(const String& method);
- void internalAddMessage(MessageType, MessageLevel, JSC::ExecState*, PassRefPtr<ScriptArguments>);
-
- InspectorConsoleAgent* m_consoleAgent;
-};
-
-}
-
-#endif // !defined(JSConsoleClient_h)
</del></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorJSGlobalObjectConsoleClientcppfromrev173250trunkSourceJavaScriptCoreinspectorJSConsoleClientcpp"></a>
<div class="copfile"><h4>Copied: trunk/Source/JavaScriptCore/inspector/JSGlobalObjectConsoleClient.cpp (from rev 173250, trunk/Source/JavaScriptCore/inspector/JSConsoleClient.cpp) (0 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/JSGlobalObjectConsoleClient.cpp (rev 0)
+++ trunk/Source/JavaScriptCore/inspector/JSGlobalObjectConsoleClient.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -0,0 +1,130 @@
</span><ins>+/*
+ * Copyright (C) 2014 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY
+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR
+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
+ * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "JSGlobalObjectConsoleClient.h"
+
+#if ENABLE(INSPECTOR)
+
+#include "InspectorConsoleAgent.h"
+#include "ScriptArguments.h"
+#include "ScriptCallStack.h"
+#include "ScriptCallStackFactory.h"
+
+#if USE(CF)
+#include <CoreFoundation/CoreFoundation.h>
+#endif
+
+using namespace JSC;
+
+namespace Inspector {
+
+static bool sLogToSystemConsole = false;
+
+bool JSGlobalObjectConsoleClient::logToSystemConsole()
+{
+ return sLogToSystemConsole;
+}
+
+void JSGlobalObjectConsoleClient::setLogToSystemConsole(bool shouldLog)
+{
+ sLogToSystemConsole = shouldLog;
+}
+
+void JSGlobalObjectConsoleClient::initializeLogToSystemConsole()
+{
+#if !LOG_DISABLED
+ sLogToSystemConsole = true;
+#elif USE(CF)
+ Boolean keyExistsAndHasValidFormat = false;
+ Boolean preference = CFPreferencesGetAppBooleanValue(CFSTR("JavaScriptCoreOutputConsoleMessagesToSystemConsole"), kCFPreferencesCurrentApplication, &keyExistsAndHasValidFormat);
+ if (keyExistsAndHasValidFormat)
+ sLogToSystemConsole = preference;
+#endif
+}
+
+JSGlobalObjectConsoleClient::JSGlobalObjectConsoleClient(InspectorConsoleAgent* consoleAgent)
+ : ConsoleClient()
+ , m_consoleAgent(consoleAgent)
+{
+ static std::once_flag initializeLogging;
+ std::call_once(initializeLogging, []{
+ JSGlobalObjectConsoleClient::initializeLogToSystemConsole();
+ });
+}
+
+void JSGlobalObjectConsoleClient::messageWithTypeAndLevel(MessageType type, MessageLevel level, JSC::ExecState* exec, PassRefPtr<ScriptArguments> prpArguments)
+{
+ RefPtr<ScriptArguments> arguments = prpArguments;
+
+ if (JSGlobalObjectConsoleClient::logToSystemConsole())
+ ConsoleClient::printConsoleMessageWithArguments(MessageSource::ConsoleAPI, type, level, exec, arguments);
+
+ String message;
+ arguments->getFirstArgumentAsString(message);
+ m_consoleAgent->addMessageToConsole(MessageSource::ConsoleAPI, type, level, message, exec, arguments.release());
+}
+
+void JSGlobalObjectConsoleClient::count(ExecState* exec, PassRefPtr<ScriptArguments> arguments)
+{
+ m_consoleAgent->count(exec, arguments);
+}
+
+void JSGlobalObjectConsoleClient::profile(JSC::ExecState*, const String&)
+{
+ // FIXME: support |console.profile| for JSContexts. <https://webkit.org/b/136466>
+}
+
+void JSGlobalObjectConsoleClient::profileEnd(JSC::ExecState*, const String&)
+{
+ // FIXME: support |console.profile| for JSContexts. <https://webkit.org/b/136466>
+}
+
+void JSGlobalObjectConsoleClient::time(ExecState*, const String& title)
+{
+ m_consoleAgent->startTiming(title);
+}
+
+void JSGlobalObjectConsoleClient::timeEnd(ExecState* exec, const String& title)
+{
+ RefPtr<ScriptCallStack> callStack(createScriptCallStackForConsole(exec, 1));
+ m_consoleAgent->stopTiming(title, callStack.release());
+}
+
+void JSGlobalObjectConsoleClient::timeStamp(ExecState*, PassRefPtr<ScriptArguments>)
+{
+ // FIXME: JSContext inspection needs a timeline.
+ warnUnimplemented(ASCIILiteral("console.timeStamp"));
+}
+
+void JSGlobalObjectConsoleClient::warnUnimplemented(const String& method)
+{
+ String message = method + " is currently ignored in JavaScript context inspection.";
+ m_consoleAgent->addMessageToConsole(MessageSource::ConsoleAPI, MessageType::Log, MessageLevel::Warning, message, nullptr, nullptr);
+}
+
+} // namespace Inspector
+
+#endif
</ins></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorJSGlobalObjectConsoleClienthfromrev173250trunkSourceJavaScriptCoreinspectorJSConsoleClienth"></a>
<div class="copfile"><h4>Copied: trunk/Source/JavaScriptCore/inspector/JSGlobalObjectConsoleClient.h (from rev 173250, trunk/Source/JavaScriptCore/inspector/JSConsoleClient.h) (0 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/JSGlobalObjectConsoleClient.h (rev 0)
+++ trunk/Source/JavaScriptCore/inspector/JSGlobalObjectConsoleClient.h 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -0,0 +1,63 @@
</span><ins>+/*
+ * Copyright (C) 2014 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY
+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR
+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
+ * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef JSGlobalObjectConsoleClient_h
+#define JSGlobalObjectConsoleClient_h
+
+#include "ConsoleClient.h"
+
+namespace Inspector {
+
+class InspectorConsoleAgent;
+
+class JSGlobalObjectConsoleClient final : public JSC::ConsoleClient {
+ WTF_MAKE_FAST_ALLOCATED;
+public:
+ explicit JSGlobalObjectConsoleClient(InspectorConsoleAgent*);
+ virtual ~JSGlobalObjectConsoleClient() { }
+
+ static bool logToSystemConsole();
+ static void setLogToSystemConsole(bool);
+ static void initializeLogToSystemConsole();
+
+protected:
+ virtual void messageWithTypeAndLevel(MessageType, MessageLevel, JSC::ExecState*, PassRefPtr<ScriptArguments>) override;
+ virtual void count(JSC::ExecState*, PassRefPtr<ScriptArguments>) override;
+ virtual void profile(JSC::ExecState*, const String& title) override;
+ virtual void profileEnd(JSC::ExecState*, const String& title) override;
+ virtual void time(JSC::ExecState*, const String& title) override;
+ virtual void timeEnd(JSC::ExecState*, const String& title) override;
+ virtual void timeStamp(JSC::ExecState*, PassRefPtr<ScriptArguments>) override;
+
+private:
+ void warnUnimplemented(const String& method);
+ void internalAddMessage(MessageType, MessageLevel, JSC::ExecState*, PassRefPtr<ScriptArguments>);
+
+ InspectorConsoleAgent* m_consoleAgent;
+};
+
+}
+
+#endif // !defined(JSGlobalObjectConsoleClient_h)
</ins></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorJSGlobalObjectInspectorControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -35,9 +35,9 @@
</span><span class="cx"> #include "InspectorAgent.h"
</span><span class="cx"> #include "InspectorBackendDispatcher.h"
</span><span class="cx"> #include "InspectorFrontendChannel.h"
</span><del>-#include "JSConsoleClient.h"
</del><span class="cx"> #include "JSGlobalObject.h"
</span><span class="cx"> #include "JSGlobalObjectConsoleAgent.h"
</span><ins>+#include "JSGlobalObjectConsoleClient.h"
</ins><span class="cx"> #include "JSGlobalObjectDebuggerAgent.h"
</span><span class="cx"> #include "JSGlobalObjectRuntimeAgent.h"
</span><span class="cx"> #include "ScriptArguments.h"
</span><span class="lines">@@ -62,7 +62,7 @@
</span><span class="cx"> auto debuggerAgent = std::make_unique<JSGlobalObjectDebuggerAgent>(m_injectedScriptManager.get(), m_globalObject, consoleAgent.get());
</span><span class="cx">
</span><span class="cx"> m_consoleAgent = consoleAgent.get();
</span><del>- m_consoleClient = std::make_unique<JSConsoleClient>(m_consoleAgent);
</del><ins>+ m_consoleClient = std::make_unique<JSGlobalObjectConsoleClient>(m_consoleAgent);
</ins><span class="cx">
</span><span class="cx"> runtimeAgent->setScriptDebugServer(&debuggerAgent->scriptDebugServer());
</span><span class="cx">
</span><span class="lines">@@ -156,7 +156,7 @@
</span><span class="cx"> String errorMessage = exception.toString(exec)->value(exec);
</span><span class="cx"> exec->clearException();
</span><span class="cx">
</span><del>- if (JSConsoleClient::logToSystemConsole()) {
</del><ins>+ if (JSGlobalObjectConsoleClient::logToSystemConsole()) {
</ins><span class="cx"> if (callStack->size()) {
</span><span class="cx"> const ScriptCallFrame& callFrame = callStack->at(0);
</span><span class="cx"> ConsoleClient::printConsoleMessage(MessageSource::JS, MessageType::Log, MessageLevel::Error, errorMessage, callFrame.sourceURL(), callFrame.lineNumber(), callFrame.columnNumber());
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorJSGlobalObjectInspectorControllerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.h (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.h 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.h 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -48,7 +48,7 @@
</span><span class="cx"> class InspectorBackendDispatcher;
</span><span class="cx"> class InspectorConsoleAgent;
</span><span class="cx"> class InspectorFrontendChannel;
</span><del>-class JSConsoleClient;
</del><ins>+class JSGlobalObjectConsoleClient;
</ins><span class="cx"> class ScriptCallStack;
</span><span class="cx">
</span><span class="cx"> class JSGlobalObjectInspectorController final : public InspectorEnvironment {
</span><span class="lines">@@ -83,7 +83,7 @@
</span><span class="cx">
</span><span class="cx"> JSC::JSGlobalObject& m_globalObject;
</span><span class="cx"> std::unique_ptr<InjectedScriptManager> m_injectedScriptManager;
</span><del>- std::unique_ptr<JSConsoleClient> m_consoleClient;
</del><ins>+ std::unique_ptr<JSGlobalObjectConsoleClient> m_consoleClient;
</ins><span class="cx"> InspectorConsoleAgent* m_consoleAgent;
</span><span class="cx"> InspectorAgentRegistry m_agents;
</span><span class="cx"> InspectorFrontendChannel* m_inspectorFrontendChannel;
</span></span></pre></div>
<a id="trunkSourceWebCoreCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/CMakeLists.txt (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/CMakeLists.txt 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/CMakeLists.txt 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1938,7 +1938,7 @@
</span><span class="cx"> page/OriginAccessEntry.cpp
</span><span class="cx"> page/Page.cpp
</span><span class="cx"> page/PageActivityAssertionToken.cpp
</span><del>- page/PageConsole.cpp
</del><ins>+ page/PageConsoleClient.cpp
</ins><span class="cx"> page/PageGroup.cpp
</span><span class="cx"> page/PageGroupLoadDeferrer.cpp
</span><span class="cx"> page/PageSerializer.cpp
</span></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/ChangeLog 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1,3 +1,58 @@
</span><ins>+2014-09-02 Brian J. Burg <burg@cs.washington.edu>
+
+ Web Inspector: fix prefixes for subclasses of JSC::ConsoleClient
+ https://bugs.webkit.org/show_bug.cgi?id=136476
+
+ Reviewed by Timothy Hatcher.
+
+ No new tests. No behavior changed.
+
+ * CMakeLists.txt:
+ * WebCore.exp.in:
+ * WebCore.order:
+ * WebCore.vcxproj/WebCore.vcxproj:
+ * WebCore.vcxproj/WebCore.vcxproj.filters:
+ * WebCore.xcodeproj/project.pbxproj:
+ * bindings/js/JSCustomXPathNSResolver.cpp:
+ (WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):
+ * bindings/js/ScriptCachedFrameData.cpp:
+ (WebCore::ScriptCachedFrameData::restore):
+ * bindings/js/ScriptController.cpp:
+ (WebCore::ScriptController::clearWindowShell):
+ (WebCore::ScriptController::initScript):
+ * css/CSSParser.cpp:
+ (WebCore::CSSParser::logError):
+ * dom/Document.cpp:
+ (WebCore::Document::addConsoleMessage):
+ (WebCore::Document::addMessage):
+ * inspector/PageDebuggerAgent.cpp:
+ (WebCore::PageDebuggerAgent::muteConsole):
+ (WebCore::PageDebuggerAgent::unmuteConsole):
+ (WebCore::PageDebuggerAgent::breakpointActionLog):
+ * inspector/PageRuntimeAgent.cpp:
+ (WebCore::PageRuntimeAgent::muteConsole):
+ (WebCore::PageRuntimeAgent::unmuteConsole):
+ * page/DOMWindow.cpp:
+ (WebCore::DOMWindow::console):
+ (WebCore::DOMWindow::dispatchMessageEventWithOriginCheck):
+ (WebCore::DOMWindow::close):
+ (WebCore::DOMWindow::printErrorMessage):
+ (WebCore::DOMWindow::pageConsole): Deleted.
+ * page/DOMWindow.h:
+ * page/Page.cpp:
+ (WebCore::Page::Page):
+ * page/Page.h:
+ * page/PageConsoleClient.cpp: Renamed from Source/WebCore/page/PageConsole.cpp.
+ * page/PageConsoleClient.h: Renamed from Source/WebCore/page/PageConsole.h.
+ * testing/Internals.cpp:
+ (WebCore::Internals::consoleProfiles):
+ * testing/Internals.h:
+ * xml/XSLStyleSheetLibxslt.cpp:
+ (WebCore::XSLStyleSheet::parseString):
+ * xml/XSLTProcessorLibxslt.cpp:
+ (WebCore::XSLTProcessor::parseErrorFunc):
+ (WebCore::docLoaderFunc):
+
</ins><span class="cx"> 2014-09-03 Maciej Stachowiak <mjs@apple.com>
</span><span class="cx">
</span><span class="cx"> Fix a few leftovers from removing MIME-related WKSI usage
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCoreexpin"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.exp.in (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.exp.in 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/WebCore.exp.in 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -187,8 +187,6 @@
</span><span class="cx"> __ZN7WebCore11MemoryCache19getOriginsWithCacheERN3WTF7HashSetINS1_6RefPtrINS_14SecurityOriginEEENS_18SecurityOriginHashENS1_10HashTraitsIS5_EEEE
</span><span class="cx"> __ZN7WebCore11MemoryCache20removeImageFromCacheERKNS_3URLERKN3WTF6StringE
</span><span class="cx"> __ZN7WebCore11MemoryCache25removeResourcesWithOriginEPNS_14SecurityOriginE
</span><del>-__ZN7WebCore11PageConsole21shouldPrintExceptionsEv
-__ZN7WebCore11PageConsole24setShouldPrintExceptionsEb
</del><span class="cx"> __ZN7WebCore11SQLResultOkE
</span><span class="cx"> __ZN7WebCore11URLWithDataEP6NSDataP5NSURL
</span><span class="cx"> __ZN7WebCore11getURLBytesEPK7__CFURLRN3WTF6VectorIcLm512ENS3_15CrashOnOverflowEEE
</span><span class="lines">@@ -763,6 +761,8 @@
</span><span class="cx"> __ZN7WebCore17KeyframeValueList6insertEN3WTF10PassOwnPtrIKNS_14AnimationValueEEE
</span><span class="cx"> __ZN7WebCore17MouseRelatedEvent7offsetXEv
</span><span class="cx"> __ZN7WebCore17MouseRelatedEvent7offsetYEv
</span><ins>+__ZN7WebCore17PageConsoleClient21shouldPrintExceptionsEv
+__ZN7WebCore17PageConsoleClient24setShouldPrintExceptionsEb
</ins><span class="cx"> __ZN7WebCore17PlatformCAFilters17setFiltersOnLayerEP7CALayerRKNS_16FilterOperationsE
</span><span class="cx"> __ZN7WebCore17PlatformCAFilters23filterValueForOperationEPKNS_15FilterOperationEi
</span><span class="cx"> __ZN7WebCore17PlatformCAFilters25setBlendingFiltersOnLayerEP7CALayerNS_9BlendModeE
</span><span class="lines">@@ -2047,7 +2047,6 @@
</span><span class="cx"> __ZNK7WebCore9RenderBox12clientHeightEv
</span><span class="cx"> __ZNK7WebCore9RenderBox20flippedClientBoxRectEv
</span><span class="cx"> __ZNK7WebCore9TreeScope14getElementByIdERKN3WTF6StringE
</span><del>-__ZTVN7WebCore11PageConsoleE
</del><span class="cx"> __ZTVN7WebCore12BlobRegistryE
</span><span class="cx"> __ZTVN7WebCore12ChromeClientE
</span><span class="cx"> __ZTVN7WebCore14LoaderStrategyE
</span><span class="lines">@@ -2058,6 +2057,7 @@
</span><span class="cx"> __ZTVN7WebCore16DatabaseStrategyE
</span><span class="cx"> __ZTVN7WebCore16IconDatabaseBaseE
</span><span class="cx"> __ZTVN7WebCore17FrameLoaderClientE
</span><ins>+__ZTVN7WebCore17PageConsoleClientE
</ins><span class="cx"> __ZTVN7WebCore19BlurFilterOperationE
</span><span class="cx"> __ZTVN7WebCore21BlobDataFileReferenceE
</span><span class="cx"> __ZTVN7WebCore22DefaultFilterOperationE
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCoreorder"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.order (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.order 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/WebCore.order 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -770,7 +770,7 @@
</span><span class="cx"> __ZN7WebCore8Settings32defaultDOMTimerAlignmentIntervalEv
</span><span class="cx"> __ZN7WebCore6RegionC1Ev
</span><span class="cx"> __ZN7WebCore13PageThrottlerC1EPNS_4PageE
</span><del>-__ZN7WebCore11PageConsoleC1EPNS_4PageE
</del><ins>+__ZN7WebCore17PageConsoleClientC1EPNS_4PageE
</ins><span class="cx"> __ZN7WebCore20networkStateNotifierEv
</span><span class="cx"> __ZN7WebCore20NetworkStateNotifierC1Ev
</span><span class="cx"> __ZN7WebCore20NetworkStateNotifierC2Ev
</span><span class="lines">@@ -5268,7 +5268,7 @@
</span><span class="cx"> __ZN7WebCore9CSSParser15currentLocationEv
</span><span class="cx"> __ZN7WebCore9CSSParser11syntaxErrorERKNS0_8LocationENS0_15SyntaxErrorTypeE
</span><span class="cx"> __ZN7WebCore9CSSParser8logErrorERKN3WTF6StringEi
</span><del>-__ZN7WebCore11PageConsole10addMessageENS_13MessageSourceENS_12MessageLevelERKN3WTF6StringES6_jjNS3_10PassRefPtrINS_15ScriptCallStackEEEPN3JSC9ExecStateEm
</del><ins>+__ZN7WebCore17PageConsoleClient10addMessageENS_13MessageSourceENS_12MessageLevelERKN3WTF6StringES6_jjNS3_10PassRefPtrINS_15ScriptCallStackEEEPN3JSC9ExecStateEm
</ins><span class="cx"> __ZN7WebCore21InspectorConsoleAgent19addMessageToConsoleENS_13MessageSourceENS_11MessageTypeENS_12MessageLevelERKN3WTF6StringES7_jjPN3JSC9ExecStateEm
</span><span class="cx"> __ZN7WebCore9CSSParser13parseSVGValueENS_13CSSPropertyIDEb
</span><span class="cx"> __ZN7WebCore9CSSParser29parseDeprecatedLinearGradientEPNS_18CSSParserValueListERN3WTF6RefPtrINS_8CSSValueEEENS_17CSSGradientRepeatE
</span><span class="lines">@@ -6344,9 +6344,9 @@
</span><span class="cx"> __ZN7WebCore18InjectedScriptHost17InspectableObjectD0Ev
</span><span class="cx"> __ZN7WebCore21BackForwardController5closeEv
</span><span class="cx"> __ZN7WebCore13PageThrottler9clearPageEv
</span><del>-__ZN7WebCore11PageConsoleD0Ev
</del><span class="cx"> __ZN7WebCore13PageThrottlerD1Ev
</span><span class="cx"> __ZN7WebCore13PageThrottlerD2Ev
</span><ins>+__ZN7WebCore17PageConsoleClientD0Ev
</ins><span class="cx"> __ZN7WebCore15FeatureObserverD1Ev
</span><span class="cx"> __ZN7WebCore5FrameD1Ev
</span><span class="cx"> __ZN7WebCore5FrameD2Ev
</span><span class="lines">@@ -11726,7 +11726,7 @@
</span><span class="cx"> __ZN7WebCore15ScriptArguments24getFirstArgumentAsStringERN3WTF6StringEb
</span><span class="cx"> __ZNK7WebCore11ScriptValue8toStringEPN3JSC9ExecStateE
</span><span class="cx"> __ZN7WebCore21InspectorConsoleAgent19addMessageToConsoleENS_13MessageSourceENS_11MessageTypeENS_12MessageLevelERKN3WTF6StringEPN3JSC9ExecStateENS4_10PassRefPtrINS_15ScriptArgumentsEEEm
</span><del>-__ZN7WebCore11PageConsole21shouldPrintExceptionsEv
</del><ins>+__ZN7WebCore17PageConsoleClient21shouldPrintExceptionsEv
</ins><span class="cx"> __ZN7WebCore15ScriptArgumentsD1Ev
</span><span class="cx"> __ZN7WebCore15ScriptArgumentsD2Ev
</span><span class="cx"> __ZN7WebCore11ScriptValueD1Ev
</span><span class="lines">@@ -14781,7 +14781,7 @@
</span><span class="cx"> __ZN7WebCore17deviceScaleFactorEPNS_5FrameE
</span><span class="cx"> __ZN7WebCore25printErrorMessageForFrameEPNS_5FrameERKN3WTF6StringE
</span><span class="cx"> __ZN7WebCore9DOMWindow17printErrorMessageERKN3WTF6StringE
</span><del>-__ZN7WebCore11PageConsole10addMessageENS_13MessageSourceENS_12MessageLevelERKN3WTF6StringEmPNS_8DocumentE
</del><ins>+__ZN7WebCore17PageConsoleClient10addMessageENS_13MessageSourceENS_12MessageLevelERKN3WTF6StringEmPNS_8DocumentE
</ins><span class="cx"> __ZN7WebCore14SubframeLoader21resourceWillUsePluginERKN3WTF6StringES4_b
</span><span class="cx"> __ZN7WebCore17HTMLObjectElement21renderFallbackContentEv
</span><span class="cx"> __ZN7WebCore9FrameView20postLayoutTimerFiredEPNS_5TimerIS0_EE
</span><span class="lines">@@ -18731,7 +18731,7 @@
</span><span class="cx"> __ZNK7WebCore9InputType16isMouseFocusableEv
</span><span class="cx"> __ZNK7WebCore16HTMLInputElement31isTextFormControlMouseFocusableEv
</span><span class="cx"> __ZNK7WebCore16HTMLInputElement17canStartSelectionEv
</span><del>-__ZN7WebCore11PageConsole10addMessageENS_13MessageSourceENS_12MessageLevelERKN3WTF6StringENS3_10PassRefPtrINS_15ScriptCallStackEEE
</del><ins>+__ZN7WebCore17PageConsoleClient10addMessageENS_13MessageSourceENS_12MessageLevelERKN3WTF6StringENS3_10PassRefPtrINS_15ScriptCallStackEEE
</ins><span class="cx"> __ZN7WebCoreL17filterConstructorERKNS_13QualifiedNameEPNS_8DocumentEb
</span><span class="cx"> __ZN7WebCore16SVGFilterElement6createERKNS_13QualifiedNameEPNS_8DocumentE
</span><span class="cx"> __ZN7WebCore16SVGFilterElementC2ERKNS_13QualifiedNameEPNS_8DocumentE
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCorevcxprojWebCorevcxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -7177,7 +7177,7 @@
</span><span class="cx"> <ClCompile Include="..\page\OriginAccessEntry.cpp" />
</span><span class="cx"> <ClCompile Include="..\page\Page.cpp" />
</span><span class="cx"> <ClCompile Include="..\page\PageActivityAssertionToken.cpp" />
</span><del>- <ClCompile Include="..\page\PageConsole.cpp" />
</del><ins>+ <ClCompile Include="..\page\PageConsoleClient.cpp" />
</ins><span class="cx"> <ClCompile Include="..\page\PageGroup.cpp" />
</span><span class="cx"> <ClCompile Include="..\page\PageGroupLoadDeferrer.cpp" />
</span><span class="cx"> <ClCompile Include="..\page\PageSerializer.cpp" />
</span><span class="lines">@@ -19158,7 +19158,7 @@
</span><span class="cx"> <ClInclude Include="..\page\Navigator.h" />
</span><span class="cx"> <ClInclude Include="..\page\NavigatorBase.h" />
</span><span class="cx"> <ClInclude Include="..\page\OriginAccessEntry.h" />
</span><del>- <ClInclude Include="..\page\PageConsole.h" />
</del><ins>+ <ClInclude Include="..\page\PageConsoleClient.h" />
</ins><span class="cx"> <ClInclude Include="..\page\PageGroup.h" />
</span><span class="cx"> <ClInclude Include="..\page\PageGroupLoadDeferrer.h" />
</span><span class="cx"> <ClInclude Include="..\page\PageSerializer.h" />
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCorevcxprojWebCorevcxprojfilters"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -4983,7 +4983,7 @@
</span><span class="cx"> <ClCompile Include="..\page\PageActivityAssertionToken.cpp">
</span><span class="cx"> <Filter>page</Filter>
</span><span class="cx"> </ClCompile>
</span><del>- <ClCompile Include="..\page\PageConsole.cpp">
</del><ins>+ <ClCompile Include="..\page\PageConsoleClient.cpp">
</ins><span class="cx"> <Filter>page</Filter>
</span><span class="cx"> </ClCompile>
</span><span class="cx"> <ClCompile Include="..\page\PageGroup.cpp">
</span><span class="lines">@@ -12931,7 +12931,7 @@
</span><span class="cx"> <ClInclude Include="..\page\OriginAccessEntry.h">
</span><span class="cx"> <Filter>page</Filter>
</span><span class="cx"> </ClInclude>
</span><del>- <ClInclude Include="..\page\PageConsole.h">
</del><ins>+ <ClInclude Include="..\page\PageConsoleClient.h">
</ins><span class="cx"> <Filter>page</Filter>
</span><span class="cx"> </ClInclude>
</span><span class="cx"> <ClInclude Include="..\page\PageGroup.h">
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -5811,8 +5811,8 @@
</span><span class="cx"> D7613A501474F13F00DB8606 /* WebKitNamedFlow.cpp in Sources */ = {isa = PBXBuildFile; fileRef = D7613A4E1474F13F00DB8606 /* WebKitNamedFlow.cpp */; };
</span><span class="cx"> D7613AC414753E5600DB8606 /* JSWebKitNamedFlow.cpp in Sources */ = {isa = PBXBuildFile; fileRef = D7613AC214753E5600DB8606 /* JSWebKitNamedFlow.cpp */; };
</span><span class="cx"> D8B6152F1032495100C8554A /* Cookie.h in Headers */ = {isa = PBXBuildFile; fileRef = D8B6152E1032495100C8554A /* Cookie.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><del>- DAED203016F2442B0070EC0F /* PageConsole.cpp in Sources */ = {isa = PBXBuildFile; fileRef = DAACB3D916F2416400666135 /* PageConsole.cpp */; };
- DAED203116F244480070EC0F /* PageConsole.h in Headers */ = {isa = PBXBuildFile; fileRef = DAACB3DA16F2416400666135 /* PageConsole.h */; settings = {ATTRIBUTES = (Private, ); }; };
</del><ins>+ DAED203016F2442B0070EC0F /* PageConsoleClient.cpp in Sources */ = {isa = PBXBuildFile; fileRef = DAACB3D916F2416400666135 /* PageConsoleClient.cpp */; };
+ DAED203116F244480070EC0F /* PageConsoleClient.h in Headers */ = {isa = PBXBuildFile; fileRef = DAACB3DA16F2416400666135 /* PageConsoleClient.h */; settings = {ATTRIBUTES = (Private, ); }; };
</ins><span class="cx"> DB23C2CB0A508D29002489EB /* IndentOutdentCommand.cpp in Sources */ = {isa = PBXBuildFile; fileRef = DB23C2C90A508D29002489EB /* IndentOutdentCommand.cpp */; };
</span><span class="cx"> DB23C2CC0A508D29002489EB /* IndentOutdentCommand.h in Headers */ = {isa = PBXBuildFile; fileRef = DB23C2CA0A508D29002489EB /* IndentOutdentCommand.h */; };
</span><span class="cx"> DD05FE0D0B8BA3C6009ACDFE /* WebCoreObjCExtras.h in Headers */ = {isa = PBXBuildFile; fileRef = DD05FE0B0B8BA3C6009ACDFE /* WebCoreObjCExtras.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="lines">@@ -13277,8 +13277,8 @@
</span><span class="cx"> D7613AC214753E5600DB8606 /* JSWebKitNamedFlow.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSWebKitNamedFlow.cpp; sourceTree = "<group>"; };
</span><span class="cx"> D7613AC314753E5600DB8606 /* JSWebKitNamedFlow.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSWebKitNamedFlow.h; sourceTree = "<group>"; };
</span><span class="cx"> D8B6152E1032495100C8554A /* Cookie.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = Cookie.h; sourceTree = "<group>"; };
</span><del>- DAACB3D916F2416400666135 /* PageConsole.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = PageConsole.cpp; sourceTree = "<group>"; };
- DAACB3DA16F2416400666135 /* PageConsole.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PageConsole.h; sourceTree = "<group>"; };
</del><ins>+ DAACB3D916F2416400666135 /* PageConsoleClient.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = PageConsoleClient.cpp; sourceTree = "<group>"; };
+ DAACB3DA16F2416400666135 /* PageConsoleClient.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PageConsoleClient.h; sourceTree = "<group>"; };
</ins><span class="cx"> DB23C2C90A508D29002489EB /* IndentOutdentCommand.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = IndentOutdentCommand.cpp; sourceTree = "<group>"; };
</span><span class="cx"> DB23C2CA0A508D29002489EB /* IndentOutdentCommand.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = IndentOutdentCommand.h; sourceTree = "<group>"; };
</span><span class="cx"> DBFCB0EBFF5CD77EBEB35395 /* RenderMathMLSpace.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = RenderMathMLSpace.cpp; path = rendering/mathml/RenderMathMLSpace.cpp; sourceTree = SOURCE_ROOT; };
</span><span class="lines">@@ -16556,8 +16556,8 @@
</span><span class="cx"> 65A21467097A329100B9050A /* Page.h */,
</span><span class="cx"> CD08285A1757250800EC5FB7 /* PageActivityAssertionToken.cpp */,
</span><span class="cx"> CD08285B1757250800EC5FB7 /* PageActivityAssertionToken.h */,
</span><del>- DAACB3D916F2416400666135 /* PageConsole.cpp */,
- DAACB3DA16F2416400666135 /* PageConsole.h */,
</del><ins>+ DAACB3D916F2416400666135 /* PageConsoleClient.cpp */,
+ DAACB3DA16F2416400666135 /* PageConsoleClient.h */,
</ins><span class="cx"> A5A2AF091829734300DE1729 /* PageDebuggable.cpp */,
</span><span class="cx"> A5A2AF0A1829734300DE1729 /* PageDebuggable.h */,
</span><span class="cx"> 9302B0BC0D79F82900C7EE83 /* PageGroup.cpp */,
</span><span class="lines">@@ -25384,7 +25384,7 @@
</span><span class="cx"> 3774ABA50FA21EB400AD7DE9 /* OverlapTestRequestClient.h in Headers */,
</span><span class="cx"> 65A21468097A329100B9050A /* Page.h in Headers */,
</span><span class="cx"> 1477E7770BF4134A00152872 /* PageCache.h in Headers */,
</span><del>- DAED203116F244480070EC0F /* PageConsole.h in Headers */,
</del><ins>+ DAED203116F244480070EC0F /* PageConsoleClient.h in Headers */,
</ins><span class="cx"> F3820893147D35F90010BC06 /* PageConsoleAgent.h in Headers */,
</span><span class="cx"> A5A2AF0C1829734300DE1729 /* PageDebuggable.h in Headers */,
</span><span class="cx"> F34742DD134362F000531BC2 /* PageDebuggerAgent.h in Headers */,
</span><span class="lines">@@ -28899,7 +28899,7 @@
</span><span class="cx"> 65FEA86909833ADE00BED4AB /* Page.cpp in Sources */,
</span><span class="cx"> CD08285C1757250F00EC5FB7 /* PageActivityAssertionToken.cpp in Sources */,
</span><span class="cx"> 1477E7760BF4134A00152872 /* PageCache.cpp in Sources */,
</span><del>- DAED203016F2442B0070EC0F /* PageConsole.cpp in Sources */,
</del><ins>+ DAED203016F2442B0070EC0F /* PageConsoleClient.cpp in Sources */,
</ins><span class="cx"> F3820892147D35F90010BC06 /* PageConsoleAgent.cpp in Sources */,
</span><span class="cx"> A5A2AF0B1829734300DE1729 /* PageDebuggable.cpp in Sources */,
</span><span class="cx"> F34742DC134362F000531BC2 /* PageDebuggerAgent.cpp in Sources */,
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsjsJSCustomXPathNSResolvercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/js/JSCustomXPathNSResolver.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/js/JSCustomXPathNSResolver.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/bindings/js/JSCustomXPathNSResolver.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -32,7 +32,7 @@
</span><span class="cx"> #include "JSDOMWindowCustom.h"
</span><span class="cx"> #include "JSMainThreadExecState.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "SecurityOrigin.h"
</span><span class="cx"> #include <runtime/JSLock.h>
</span><span class="cx"> #include <wtf/Ref.h>
</span><span class="lines">@@ -80,7 +80,7 @@
</span><span class="cx"> callType = m_customResolver->methodTable()->getCallData(m_customResolver.get(), callData);
</span><span class="cx"> if (callType == CallTypeNone) {
</span><span class="cx"> // FIXME: <http://webkit.org/b/114312> JSCustomXPathNSResolver::lookupNamespaceURI Console Message should include Line, Column, and SourceURL
</span><del>- if (PageConsole* console = m_globalObject->impl().pageConsole())
</del><ins>+ if (PageConsoleClient* console = m_globalObject->impl().console())
</ins><span class="cx"> console->addMessage(MessageSource::JS, MessageLevel::Error, ASCIILiteral("XPathNSResolver does not have a lookupNamespaceURI method."));
</span><span class="cx"> return String();
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsjsScriptCachedFrameDatacpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/js/ScriptCachedFrameData.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/js/ScriptCachedFrameData.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/bindings/js/ScriptCachedFrameData.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -36,7 +36,7 @@
</span><span class="cx"> #include "Frame.h"
</span><span class="cx"> #include "GCController.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "PageGroup.h"
</span><span class="cx"> #include <heap/StrongInlines.h>
</span><span class="cx"> #include <runtime/JSLock.h>
</span></span></pre></div>
<a id="trunkSourceWebCorebindingsjsScriptControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/bindings/js/ScriptController.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/bindings/js/ScriptController.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/bindings/js/ScriptController.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -37,7 +37,7 @@
</span><span class="cx"> #include "MainFrame.h"
</span><span class="cx"> #include "NP_jsobject.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "PageGroup.h"
</span><span class="cx"> #include "PluginView.h"
</span><span class="cx"> #include "ScriptSourceCode.h"
</span></span></pre></div>
<a id="trunkSourceWebCorecssCSSParsercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/css/CSSParser.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/css/CSSParser.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/css/CSSParser.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -65,7 +65,7 @@
</span><span class="cx"> #include "MediaList.h"
</span><span class="cx"> #include "MediaQueryExp.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "Pair.h"
</span><span class="cx"> #include "Rect.h"
</span><span class="cx"> #include "RenderTheme.h"
</span><span class="lines">@@ -11689,7 +11689,7 @@
</span><span class="cx"> void CSSParser::logError(const String& message, int lineNumber)
</span><span class="cx"> {
</span><span class="cx"> // FIXME: <http://webkit.org/b/114313> CSS parser console message errors should include column numbers.
</span><del>- PageConsole& console = m_styleSheet->singleOwnerDocument()->page()->console();
</del><ins>+ PageConsoleClient& console = m_styleSheet->singleOwnerDocument()->page()->console();
</ins><span class="cx"> console.addMessage(MessageSource::CSS, MessageLevel::Warning, message, m_styleSheet->baseURL().string(), lineNumber + 1, 0);
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoredomDocumentcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/dom/Document.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/dom/Document.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/dom/Document.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -105,7 +105,7 @@
</span><span class="cx"> #include "NodeIterator.h"
</span><span class="cx"> #include "NodeRareData.h"
</span><span class="cx"> #include "NodeWithIndex.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "PageGroup.h"
</span><span class="cx"> #include "PageTransitionEvent.h"
</span><span class="cx"> #include "PlatformLocale.h"
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorPageDebuggerAgentcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/PageDebuggerAgent.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/PageDebuggerAgent.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/inspector/PageDebuggerAgent.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -38,7 +38,7 @@
</span><span class="cx"> #include "InspectorPageAgent.h"
</span><span class="cx"> #include "InstrumentingAgents.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "PageScriptDebugServer.h"
</span><span class="cx"> #include "ScriptState.h"
</span><span class="cx"> #include <inspector/InjectedScript.h>
</span><span class="lines">@@ -108,12 +108,12 @@
</span><span class="cx">
</span><span class="cx"> void PageDebuggerAgent::muteConsole()
</span><span class="cx"> {
</span><del>- PageConsole::mute();
</del><ins>+ PageConsoleClient::mute();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PageDebuggerAgent::unmuteConsole()
</span><span class="cx"> {
</span><del>- PageConsole::unmute();
</del><ins>+ PageConsoleClient::unmute();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PageDebuggerAgent::breakpointActionLog(JSC::ExecState* exec, const String& message)
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorPageRuntimeAgentcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/PageRuntimeAgent.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/PageRuntimeAgent.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/inspector/PageRuntimeAgent.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -40,7 +40,7 @@
</span><span class="cx"> #include "JSDOMWindowBase.h"
</span><span class="cx"> #include "MainFrame.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "ScriptController.h"
</span><span class="cx"> #include "ScriptState.h"
</span><span class="cx"> #include "SecurityOrigin.h"
</span><span class="lines">@@ -146,12 +146,12 @@
</span><span class="cx">
</span><span class="cx"> void PageRuntimeAgent::muteConsole()
</span><span class="cx"> {
</span><del>- PageConsole::mute();
</del><ins>+ PageConsoleClient::mute();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PageRuntimeAgent::unmuteConsole()
</span><span class="cx"> {
</span><del>- PageConsole::unmute();
</del><ins>+ PageConsoleClient::unmute();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void PageRuntimeAgent::reportExecutionContextCreation()
</span></span></pre></div>
<a id="trunkSourceWebCorepageDOMWindowcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/DOMWindow.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/DOMWindow.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/page/DOMWindow.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -76,7 +76,7 @@
</span><span class="cx"> #include "MessageEvent.h"
</span><span class="cx"> #include "Navigator.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "PageGroup.h"
</span><span class="cx"> #include "PageTransitionEvent.h"
</span><span class="cx"> #include "Performance.h"
</span><span class="lines">@@ -702,11 +702,11 @@
</span><span class="cx"> return m_toolbar.get();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-PageConsole* DOMWindow::pageConsole() const
</del><ins>+PageConsoleClient* DOMWindow::console() const
</ins><span class="cx"> {
</span><span class="cx"> if (!isCurrentlyDisplayedInFrame())
</span><del>- return 0;
- return m_frame->page() ? &m_frame->page()->console() : 0;
</del><ins>+ return nullptr;
+ return m_frame->page() ? &m_frame->page()->console() : nullptr;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> DOMApplicationCache* DOMWindow::applicationCache() const
</span><span class="lines">@@ -937,7 +937,7 @@
</span><span class="cx"> if (!intendedTargetOrigin->isSameSchemeHostPort(document()->securityOrigin())) {
</span><span class="cx"> String message = "Unable to post message to " + intendedTargetOrigin->toString() +
</span><span class="cx"> ". Recipient has origin " + document()->securityOrigin()->toString() + ".\n";
</span><del>- pageConsole()->addMessage(MessageSource::Security, MessageLevel::Error, message, stackTrace);
</del><ins>+ console()->addMessage(MessageSource::Security, MessageLevel::Error, message, stackTrace);
</ins><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="lines">@@ -1036,7 +1036,7 @@
</span><span class="cx"> bool allowScriptsToCloseWindows = m_frame->settings().allowScriptsToCloseWindows();
</span><span class="cx">
</span><span class="cx"> if (!(page->openedByDOM() || page->backForward().count() <= 1 || allowScriptsToCloseWindows)) {
</span><del>- pageConsole()->addMessage(MessageSource::JS, MessageLevel::Warning, ASCIILiteral("Can't close the window since it was not opened by JavaScript"));
</del><ins>+ console()->addMessage(MessageSource::JS, MessageLevel::Warning, ASCIILiteral("Can't close the window since it was not opened by JavaScript"));
</ins><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -2008,8 +2008,8 @@
</span><span class="cx"> if (message.isEmpty())
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- if (PageConsole* console = pageConsole())
- console->addMessage(MessageSource::JS, MessageLevel::Error, message);
</del><ins>+ if (PageConsoleClient* pageConsole = console())
+ pageConsole->addMessage(MessageSource::JS, MessageLevel::Error, message);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> String DOMWindow::crossDomainAccessErrorMessage(const DOMWindow& activeWindow)
</span></span></pre></div>
<a id="trunkSourceWebCorepageDOMWindowh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/DOMWindow.h (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/DOMWindow.h 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/page/DOMWindow.h 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -67,7 +67,7 @@
</span><span class="cx"> class Navigator;
</span><span class="cx"> class Node;
</span><span class="cx"> class Page;
</span><del>- class PageConsole;
</del><ins>+ class PageConsoleClient;
</ins><span class="cx"> class Performance;
</span><span class="cx"> class PostMessageTimer;
</span><span class="cx"> class ScheduledAction;
</span><span class="lines">@@ -235,7 +235,7 @@
</span><span class="cx"> PassRefPtr<WebKitPoint> webkitConvertPointFromPageToNode(Node*, const WebKitPoint*) const;
</span><span class="cx"> PassRefPtr<WebKitPoint> webkitConvertPointFromNodeToPage(Node*, const WebKitPoint*) const;
</span><span class="cx">
</span><del>- PageConsole* pageConsole() const;
</del><ins>+ PageConsoleClient* console() const;
</ins><span class="cx">
</span><span class="cx"> void printErrorMessage(const String&);
</span><span class="cx"> String crossDomainAccessErrorMessage(const DOMWindow& activeWindow);
</span></span></pre></div>
<a id="trunkSourceWebCorepagePagecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/Page.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/Page.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/page/Page.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -57,7 +57,7 @@
</span><span class="cx"> #include "NetworkStateNotifier.h"
</span><span class="cx"> #include "PageActivityAssertionToken.h"
</span><span class="cx"> #include "PageCache.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "PageDebuggable.h"
</span><span class="cx"> #include "PageGroup.h"
</span><span class="cx"> #include "PageThrottler.h"
</span><span class="lines">@@ -192,7 +192,7 @@
</span><span class="cx"> #endif
</span><span class="cx"> , m_alternativeTextClient(pageClients.alternativeTextClient)
</span><span class="cx"> , m_scriptedAnimationsSuspended(false)
</span><del>- , m_console(std::make_unique<PageConsole>(*this))
</del><ins>+ , m_consoleClient(std::make_unique<PageConsoleClient>(*this))
</ins><span class="cx"> #if ENABLE(REMOTE_INSPECTOR)
</span><span class="cx"> , m_inspectorDebuggable(std::make_unique<PageDebuggable>(*this))
</span><span class="cx"> #endif
</span></span></pre></div>
<a id="trunkSourceWebCorepagePageh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/page/Page.h (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/Page.h 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/page/Page.h 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -81,7 +81,7 @@
</span><span class="cx"> class MainFrame;
</span><span class="cx"> class MediaCanStartListener;
</span><span class="cx"> class PageActivityAssertionToken;
</span><del>-class PageConsole;
</del><ins>+class PageConsoleClient;
</ins><span class="cx"> class PageDebuggable;
</span><span class="cx"> class PageGroup;
</span><span class="cx"> class PageThrottler;
</span><span class="lines">@@ -408,7 +408,7 @@
</span><span class="cx"> PageThrottler* pageThrottler() { return m_pageThrottler.get(); }
</span><span class="cx"> WEBCORE_EXPORT void createPageThrottler();
</span><span class="cx">
</span><del>- PageConsole& console() { return *m_console; }
</del><ins>+ PageConsoleClient& console() { return *m_consoleClient; }
</ins><span class="cx">
</span><span class="cx"> void hiddenPageCSSAnimationSuspensionStateChanged();
</span><span class="cx">
</span><span class="lines">@@ -571,7 +571,7 @@
</span><span class="cx">
</span><span class="cx"> bool m_scriptedAnimationsSuspended;
</span><span class="cx"> std::unique_ptr<PageThrottler> m_pageThrottler;
</span><del>- const std::unique_ptr<PageConsole> m_console;
</del><ins>+ const std::unique_ptr<PageConsoleClient> m_consoleClient;
</ins><span class="cx">
</span><span class="cx"> #if ENABLE(REMOTE_INSPECTOR)
</span><span class="cx"> const std::unique_ptr<PageDebuggable> m_inspectorDebuggable;
</span></span></pre></div>
<a id="trunkSourceWebCorepagePageConsolecpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebCore/page/PageConsole.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/PageConsole.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/page/PageConsole.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1,198 +0,0 @@
</span><del>-/*
- * Copyright (C) 2013, 2014 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- *
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- * 3. Neither the name of Apple Inc. ("Apple") nor the names of
- * its contributors may be used to endorse or promote products derived
- * from this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
- * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
- * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
- * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
- * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
- * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
- * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
- * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "PageConsole.h"
-
-#include "Chrome.h"
-#include "ChromeClient.h"
-#include "Document.h"
-#include "Frame.h"
-#include "InspectorConsoleInstrumentation.h"
-#include "InspectorController.h"
-#include "JSMainThreadExecState.h"
-#include "MainFrame.h"
-#include "Page.h"
-#include "ScriptableDocumentParser.h"
-#include "Settings.h"
-#include <bindings/ScriptValue.h>
-#include <inspector/ScriptArguments.h>
-#include <inspector/ScriptCallStack.h>
-#include <inspector/ScriptCallStackFactory.h>
-
-using namespace Inspector;
-
-namespace WebCore {
-
-PageConsole::PageConsole(Page& page)
- : m_page(page)
-{
-}
-
-PageConsole::~PageConsole()
-{
-}
-
-static int muteCount = 0;
-static bool printExceptions = false;
-
-bool PageConsole::shouldPrintExceptions()
-{
- return printExceptions;
-}
-
-void PageConsole::setShouldPrintExceptions(bool print)
-{
- printExceptions = print;
-}
-
-void PageConsole::mute()
-{
- muteCount++;
-}
-
-void PageConsole::unmute()
-{
- ASSERT(muteCount > 0);
- muteCount--;
-}
-
-void PageConsole::addMessage(MessageSource source, MessageLevel level, const String& message, unsigned long requestIdentifier, Document* document)
-{
- String url;
- if (document)
- url = document->url().string();
-
- // FIXME: The below code attempts to determine line numbers for parser generated errors, but this is not the only reason why we can get here.
- // For example, if we are still parsing and get a WebSocket network error, it will be erroneously attributed to a line where parsing was paused.
- // Also, we should determine line numbers for script generated messages (e.g. calling getImageData on a canvas).
- // We probably need to split this function into multiple ones, as appropriate for different call sites. Or maybe decide based on MessageSource.
- // https://bugs.webkit.org/show_bug.cgi?id=125340
- unsigned line = 0;
- unsigned column = 0;
- if (document && document->parsing() && !document->isInDocumentWrite() && document->scriptableDocumentParser()) {
- ScriptableDocumentParser* parser = document->scriptableDocumentParser();
- if (!parser->isWaitingForScripts() && !JSMainThreadExecState::currentState()) {
- TextPosition position = parser->textPosition();
- line = position.m_line.oneBasedInt();
- column = position.m_column.oneBasedInt();
- }
- }
- addMessage(source, level, message, url, line, column, 0, JSMainThreadExecState::currentState(), requestIdentifier);
-}
-
-void PageConsole::addMessage(MessageSource source, MessageLevel level, const String& message, PassRefPtr<ScriptCallStack> callStack)
-{
- addMessage(source, level, message, String(), 0, 0, callStack, 0);
-}
-
-void PageConsole::addMessage(MessageSource source, MessageLevel level, const String& message, const String& url, unsigned lineNumber, unsigned columnNumber, PassRefPtr<ScriptCallStack> callStack, JSC::ExecState* state, unsigned long requestIdentifier)
-{
- if (muteCount && source != MessageSource::ConsoleAPI)
- return;
-
- if (callStack)
- InspectorInstrumentation::addMessageToConsole(&m_page, source, MessageType::Log, level, message, callStack, requestIdentifier);
- else
- InspectorInstrumentation::addMessageToConsole(&m_page, source, MessageType::Log, level, message, url, lineNumber, columnNumber, state, requestIdentifier);
-
- if (source == MessageSource::CSS)
- return;
-
- if (m_page.usesEphemeralSession())
- return;
-
- m_page.chrome().client().addMessageToConsole(source, level, message, lineNumber, columnNumber, url);
-
- if (!m_page.settings().logsPageMessagesToSystemConsoleEnabled() && !shouldPrintExceptions())
- return;
-
- ConsoleClient::printConsoleMessage(MessageSource::ConsoleAPI, MessageType::Log, level, message, url, lineNumber, columnNumber);
-}
-
-
-void PageConsole::messageWithTypeAndLevel(MessageType type, MessageLevel level, JSC::ExecState* exec, PassRefPtr<Inspector::ScriptArguments> prpArguments)
-{
- RefPtr<ScriptArguments> arguments = prpArguments;
-
- String message;
- bool gotMessage = arguments->getFirstArgumentAsString(message);
- InspectorInstrumentation::addMessageToConsole(&m_page, MessageSource::ConsoleAPI, type, level, message, exec, arguments);
-
- if (m_page.usesEphemeralSession())
- return;
-
- if (gotMessage) {
- size_t stackSize = type == MessageType::Trace ? ScriptCallStack::maxCallStackSizeToCapture : 1;
- RefPtr<ScriptCallStack> callStack(createScriptCallStackForConsole(exec, stackSize));
- const ScriptCallFrame& lastCaller = callStack->at(0);
- m_page.chrome().client().addMessageToConsole(MessageSource::ConsoleAPI, type, level, message, lastCaller.lineNumber(), lastCaller.columnNumber(), lastCaller.sourceURL());
- }
-
- if (m_page.settings().logsPageMessagesToSystemConsoleEnabled() || PageConsole::shouldPrintExceptions())
- ConsoleClient::printConsoleMessageWithArguments(MessageSource::ConsoleAPI, type, level, exec, arguments.release());
-}
-
-void PageConsole::count(JSC::ExecState* exec, PassRefPtr<ScriptArguments> arguments)
-{
- InspectorInstrumentation::consoleCount(&m_page, exec, arguments);
-}
-
-void PageConsole::profile(JSC::ExecState* exec, const String& title)
-{
- InspectorInstrumentation::startProfiling(&m_page, exec, title);
-}
-
-void PageConsole::profileEnd(JSC::ExecState* exec, const String& title)
-{
- if (RefPtr<JSC::Profile> profile = InspectorInstrumentation::stopProfiling(&m_page, exec, title))
- m_profiles.append(profile.release());
-}
-
-void PageConsole::time(JSC::ExecState*, const String& title)
-{
- InspectorInstrumentation::startConsoleTiming(&m_page.mainFrame(), title);
-}
-
-void PageConsole::timeEnd(JSC::ExecState* exec, const String& title)
-{
- RefPtr<ScriptCallStack> callStack(createScriptCallStackForConsole(exec, 1));
- InspectorInstrumentation::stopConsoleTiming(&m_page.mainFrame(), title, callStack.release());
-}
-
-void PageConsole::timeStamp(JSC::ExecState*, PassRefPtr<ScriptArguments> arguments)
-{
- InspectorInstrumentation::consoleTimeStamp(&m_page.mainFrame(), arguments);
-}
-
-void PageConsole::clearProfiles()
-{
- m_profiles.clear();
-}
-
-} // namespace WebCore
</del></span></pre></div>
<a id="trunkSourceWebCorepagePageConsoleh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebCore/page/PageConsole.h (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/PageConsole.h 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/page/PageConsole.h 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1,82 +0,0 @@
</span><del>-/*
- * Copyright (C) 2013 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- *
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- * 3. Neither the name of Apple Inc. ("Apple") nor the names of
- * its contributors may be used to endorse or promote products derived
- * from this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
- * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
- * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
- * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
- * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
- * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
- * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
- * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef PageConsole_h
-#define PageConsole_h
-
-#include <inspector/ScriptCallStack.h>
-#include <runtime/ConsoleClient.h>
-#include <wtf/Forward.h>
-
-namespace JSC {
-class ExecState;
-class Profile;
-}
-
-namespace WebCore {
-
-class Document;
-class Page;
-
-typedef Vector<RefPtr<JSC::Profile>> ProfilesArray;
-
-class PageConsole final : public JSC::ConsoleClient {
-public:
- explicit PageConsole(Page&);
- virtual ~PageConsole();
-
- WEBCORE_EXPORT static bool shouldPrintExceptions();
- WEBCORE_EXPORT static void setShouldPrintExceptions(bool);
-
- static void mute();
- static void unmute();
-
- void addMessage(MessageSource, MessageLevel, const String& message, const String& sourceURL, unsigned lineNumber, unsigned columnNumber, PassRefPtr<Inspector::ScriptCallStack> = nullptr, JSC::ExecState* = nullptr, unsigned long requestIdentifier = 0);
- void addMessage(MessageSource, MessageLevel, const String& message, PassRefPtr<Inspector::ScriptCallStack>);
- void addMessage(MessageSource, MessageLevel, const String& message, unsigned long requestIdentifier = 0, Document* = nullptr);
-
- const ProfilesArray& profiles() const { return m_profiles; }
- void clearProfiles();
-
-protected:
- virtual void messageWithTypeAndLevel(MessageType, MessageLevel, JSC::ExecState*, PassRefPtr<Inspector::ScriptArguments>) override;
- virtual void count(JSC::ExecState*, PassRefPtr<Inspector::ScriptArguments>) override;
- virtual void profile(JSC::ExecState*, const String& title) override;
- virtual void profileEnd(JSC::ExecState*, const String& title) override;
- virtual void time(JSC::ExecState*, const String& title) override;
- virtual void timeEnd(JSC::ExecState*, const String& title) override;
- virtual void timeStamp(JSC::ExecState*, PassRefPtr<Inspector::ScriptArguments>) override;
-
-private:
- Page& m_page;
- ProfilesArray m_profiles;
-};
-
-} // namespace WebCore
-
-#endif // PageConsole_h
</del></span></pre></div>
<a id="trunkSourceWebCorepagePageConsoleClientcppfromrev173250trunkSourceWebCorepagePageConsolecpp"></a>
<div class="copfile"><h4>Copied: trunk/Source/WebCore/page/PageConsoleClient.cpp (from rev 173250, trunk/Source/WebCore/page/PageConsole.cpp) (0 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/PageConsoleClient.cpp (rev 0)
+++ trunk/Source/WebCore/page/PageConsoleClient.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -0,0 +1,198 @@
</span><ins>+/*
+ * Copyright (C) 2013, 2014 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ * 3. Neither the name of Apple Inc. ("Apple") nor the names of
+ * its contributors may be used to endorse or promote products derived
+ * from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+ * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
+ * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
+ * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "PageConsoleClient.h"
+
+#include "Chrome.h"
+#include "ChromeClient.h"
+#include "Document.h"
+#include "Frame.h"
+#include "InspectorConsoleInstrumentation.h"
+#include "InspectorController.h"
+#include "JSMainThreadExecState.h"
+#include "MainFrame.h"
+#include "Page.h"
+#include "ScriptableDocumentParser.h"
+#include "Settings.h"
+#include <bindings/ScriptValue.h>
+#include <inspector/ScriptArguments.h>
+#include <inspector/ScriptCallStack.h>
+#include <inspector/ScriptCallStackFactory.h>
+
+using namespace Inspector;
+
+namespace WebCore {
+
+PageConsoleClient::PageConsoleClient(Page& page)
+ : m_page(page)
+{
+}
+
+PageConsoleClient::~PageConsoleClient()
+{
+}
+
+static int muteCount = 0;
+static bool printExceptions = false;
+
+bool PageConsoleClient::shouldPrintExceptions()
+{
+ return printExceptions;
+}
+
+void PageConsoleClient::setShouldPrintExceptions(bool print)
+{
+ printExceptions = print;
+}
+
+void PageConsoleClient::mute()
+{
+ muteCount++;
+}
+
+void PageConsoleClient::unmute()
+{
+ ASSERT(muteCount > 0);
+ muteCount--;
+}
+
+void PageConsoleClient::addMessage(MessageSource source, MessageLevel level, const String& message, unsigned long requestIdentifier, Document* document)
+{
+ String url;
+ if (document)
+ url = document->url().string();
+
+ // FIXME: The below code attempts to determine line numbers for parser generated errors, but this is not the only reason why we can get here.
+ // For example, if we are still parsing and get a WebSocket network error, it will be erroneously attributed to a line where parsing was paused.
+ // Also, we should determine line numbers for script generated messages (e.g. calling getImageData on a canvas).
+ // We probably need to split this function into multiple ones, as appropriate for different call sites. Or maybe decide based on MessageSource.
+ // https://bugs.webkit.org/show_bug.cgi?id=125340
+ unsigned line = 0;
+ unsigned column = 0;
+ if (document && document->parsing() && !document->isInDocumentWrite() && document->scriptableDocumentParser()) {
+ ScriptableDocumentParser* parser = document->scriptableDocumentParser();
+ if (!parser->isWaitingForScripts() && !JSMainThreadExecState::currentState()) {
+ TextPosition position = parser->textPosition();
+ line = position.m_line.oneBasedInt();
+ column = position.m_column.oneBasedInt();
+ }
+ }
+ addMessage(source, level, message, url, line, column, 0, JSMainThreadExecState::currentState(), requestIdentifier);
+}
+
+void PageConsoleClient::addMessage(MessageSource source, MessageLevel level, const String& message, PassRefPtr<ScriptCallStack> callStack)
+{
+ addMessage(source, level, message, String(), 0, 0, callStack, 0);
+}
+
+void PageConsoleClient::addMessage(MessageSource source, MessageLevel level, const String& message, const String& url, unsigned lineNumber, unsigned columnNumber, PassRefPtr<ScriptCallStack> callStack, JSC::ExecState* state, unsigned long requestIdentifier)
+{
+ if (muteCount && source != MessageSource::ConsoleAPI)
+ return;
+
+ if (callStack)
+ InspectorInstrumentation::addMessageToConsole(&m_page, source, MessageType::Log, level, message, callStack, requestIdentifier);
+ else
+ InspectorInstrumentation::addMessageToConsole(&m_page, source, MessageType::Log, level, message, url, lineNumber, columnNumber, state, requestIdentifier);
+
+ if (source == MessageSource::CSS)
+ return;
+
+ if (m_page.usesEphemeralSession())
+ return;
+
+ m_page.chrome().client().addMessageToConsole(source, level, message, lineNumber, columnNumber, url);
+
+ if (!m_page.settings().logsPageMessagesToSystemConsoleEnabled() && !shouldPrintExceptions())
+ return;
+
+ ConsoleClient::printConsoleMessage(MessageSource::ConsoleAPI, MessageType::Log, level, message, url, lineNumber, columnNumber);
+}
+
+
+void PageConsoleClient::messageWithTypeAndLevel(MessageType type, MessageLevel level, JSC::ExecState* exec, PassRefPtr<Inspector::ScriptArguments> prpArguments)
+{
+ RefPtr<ScriptArguments> arguments = prpArguments;
+
+ String message;
+ bool gotMessage = arguments->getFirstArgumentAsString(message);
+ InspectorInstrumentation::addMessageToConsole(&m_page, MessageSource::ConsoleAPI, type, level, message, exec, arguments);
+
+ if (m_page.usesEphemeralSession())
+ return;
+
+ if (gotMessage) {
+ size_t stackSize = type == MessageType::Trace ? ScriptCallStack::maxCallStackSizeToCapture : 1;
+ RefPtr<ScriptCallStack> callStack(createScriptCallStackForConsole(exec, stackSize));
+ const ScriptCallFrame& lastCaller = callStack->at(0);
+ m_page.chrome().client().addMessageToConsole(MessageSource::ConsoleAPI, type, level, message, lastCaller.lineNumber(), lastCaller.columnNumber(), lastCaller.sourceURL());
+ }
+
+ if (m_page.settings().logsPageMessagesToSystemConsoleEnabled() || PageConsoleClient::shouldPrintExceptions())
+ ConsoleClient::printConsoleMessageWithArguments(MessageSource::ConsoleAPI, type, level, exec, arguments.release());
+}
+
+void PageConsoleClient::count(JSC::ExecState* exec, PassRefPtr<ScriptArguments> arguments)
+{
+ InspectorInstrumentation::consoleCount(&m_page, exec, arguments);
+}
+
+void PageConsoleClient::profile(JSC::ExecState* exec, const String& title)
+{
+ InspectorInstrumentation::startProfiling(&m_page, exec, title);
+}
+
+void PageConsoleClient::profileEnd(JSC::ExecState* exec, const String& title)
+{
+ if (RefPtr<JSC::Profile> profile = InspectorInstrumentation::stopProfiling(&m_page, exec, title))
+ m_profiles.append(profile.release());
+}
+
+void PageConsoleClient::time(JSC::ExecState*, const String& title)
+{
+ InspectorInstrumentation::startConsoleTiming(&m_page.mainFrame(), title);
+}
+
+void PageConsoleClient::timeEnd(JSC::ExecState* exec, const String& title)
+{
+ RefPtr<ScriptCallStack> callStack(createScriptCallStackForConsole(exec, 1));
+ InspectorInstrumentation::stopConsoleTiming(&m_page.mainFrame(), title, callStack.release());
+}
+
+void PageConsoleClient::timeStamp(JSC::ExecState*, PassRefPtr<ScriptArguments> arguments)
+{
+ InspectorInstrumentation::consoleTimeStamp(&m_page.mainFrame(), arguments);
+}
+
+void PageConsoleClient::clearProfiles()
+{
+ m_profiles.clear();
+}
+
+} // namespace WebCore
</ins></span></pre></div>
<a id="trunkSourceWebCorepagePageConsoleClienthfromrev173250trunkSourceWebCorepagePageConsoleh"></a>
<div class="copfile"><h4>Copied: trunk/Source/WebCore/page/PageConsoleClient.h (from rev 173250, trunk/Source/WebCore/page/PageConsole.h) (0 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/page/PageConsoleClient.h (rev 0)
+++ trunk/Source/WebCore/page/PageConsoleClient.h 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -0,0 +1,82 @@
</span><ins>+/*
+ * Copyright (C) 2013 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ * 3. Neither the name of Apple Inc. ("Apple") nor the names of
+ * its contributors may be used to endorse or promote products derived
+ * from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
+ * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+ * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
+ * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
+ * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef PageConsoleClient_h
+#define PageConsoleClient_h
+
+#include <inspector/ScriptCallStack.h>
+#include <runtime/ConsoleClient.h>
+#include <wtf/Forward.h>
+
+namespace JSC {
+class ExecState;
+class Profile;
+}
+
+namespace WebCore {
+
+class Document;
+class Page;
+
+typedef Vector<RefPtr<JSC::Profile>> ProfilesArray;
+
+class PageConsoleClient final : public JSC::ConsoleClient {
+public:
+ explicit PageConsoleClient(Page&);
+ virtual ~PageConsoleClient();
+
+ WEBCORE_EXPORT static bool shouldPrintExceptions();
+ WEBCORE_EXPORT static void setShouldPrintExceptions(bool);
+
+ static void mute();
+ static void unmute();
+
+ void addMessage(MessageSource, MessageLevel, const String& message, const String& sourceURL, unsigned lineNumber, unsigned columnNumber, PassRefPtr<Inspector::ScriptCallStack> = nullptr, JSC::ExecState* = nullptr, unsigned long requestIdentifier = 0);
+ void addMessage(MessageSource, MessageLevel, const String& message, PassRefPtr<Inspector::ScriptCallStack>);
+ void addMessage(MessageSource, MessageLevel, const String& message, unsigned long requestIdentifier = 0, Document* = nullptr);
+
+ const ProfilesArray& profiles() const { return m_profiles; }
+ void clearProfiles();
+
+protected:
+ virtual void messageWithTypeAndLevel(MessageType, MessageLevel, JSC::ExecState*, PassRefPtr<Inspector::ScriptArguments>) override;
+ virtual void count(JSC::ExecState*, PassRefPtr<Inspector::ScriptArguments>) override;
+ virtual void profile(JSC::ExecState*, const String& title) override;
+ virtual void profileEnd(JSC::ExecState*, const String& title) override;
+ virtual void time(JSC::ExecState*, const String& title) override;
+ virtual void timeEnd(JSC::ExecState*, const String& title) override;
+ virtual void timeStamp(JSC::ExecState*, PassRefPtr<Inspector::ScriptArguments>) override;
+
+private:
+ Page& m_page;
+ ProfilesArray m_profiles;
+};
+
+} // namespace WebCore
+
+#endif // PageConsoleClient_h
</ins></span></pre></div>
<a id="trunkSourceWebCoretestingInternalsh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/testing/Internals.h (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/testing/Internals.h 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/testing/Internals.h 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -31,7 +31,7 @@
</span><span class="cx"> #include "ContextDestructionObserver.h"
</span><span class="cx"> #include "ExceptionCodePlaceholder.h"
</span><span class="cx"> #include "NodeList.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include <bindings/ScriptValue.h>
</span><span class="cx"> #include <runtime/ArrayBuffer.h>
</span><span class="cx"> #include <runtime/Float32Array.h>
</span></span></pre></div>
<a id="trunkSourceWebCorexmlXSLStyleSheetLibxsltcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/xml/XSLStyleSheetLibxslt.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/xml/XSLStyleSheetLibxslt.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/xml/XSLStyleSheetLibxslt.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -27,7 +27,7 @@
</span><span class="cx"> #include "Document.h"
</span><span class="cx"> #include "Frame.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "TransformSource.h"
</span><span class="cx"> #include "XMLDocumentParser.h"
</span><span class="cx"> #include "XMLDocumentParserScope.h"
</span><span class="lines">@@ -140,7 +140,7 @@
</span><span class="cx"> xmlFreeDoc(m_stylesheetDoc);
</span><span class="cx"> m_stylesheetDocTaken = false;
</span><span class="cx">
</span><del>- PageConsole* console = 0;
</del><ins>+ PageConsoleClient* console = nullptr;
</ins><span class="cx"> Frame* frame = ownerDocument()->frame();
</span><span class="cx"> if (frame && frame->page())
</span><span class="cx"> console = &frame->page()->console();
</span></span></pre></div>
<a id="trunkSourceWebCorexmlXSLTProcessorLibxsltcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/xml/XSLTProcessorLibxslt.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/xml/XSLTProcessorLibxslt.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebCore/xml/XSLTProcessorLibxslt.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -30,7 +30,7 @@
</span><span class="cx"> #include "Document.h"
</span><span class="cx"> #include "Frame.h"
</span><span class="cx"> #include "Page.h"
</span><del>-#include "PageConsole.h"
</del><ins>+#include "PageConsoleClient.h"
</ins><span class="cx"> #include "ResourceError.h"
</span><span class="cx"> #include "ResourceRequest.h"
</span><span class="cx"> #include "ResourceResponse.h"
</span><span class="lines">@@ -79,7 +79,7 @@
</span><span class="cx">
</span><span class="cx"> void XSLTProcessor::parseErrorFunc(void* userData, xmlError* error)
</span><span class="cx"> {
</span><del>- PageConsole* console = static_cast<PageConsole*>(userData);
</del><ins>+ PageConsoleClient* console = static_cast<PageConsoleClient*>(userData);
</ins><span class="cx"> if (!console)
</span><span class="cx"> return;
</span><span class="cx">
</span><span class="lines">@@ -138,7 +138,7 @@
</span><span class="cx"> globalCachedResourceLoader->printAccessDeniedMessage(url);
</span><span class="cx"> }
</span><span class="cx">
</span><del>- PageConsole* console = 0;
</del><ins>+ PageConsoleClient* console = nullptr;
</ins><span class="cx"> Frame* frame = globalProcessor->xslStylesheet()->ownerDocument()->frame();
</span><span class="cx"> if (frame && frame->page())
</span><span class="cx"> console = &frame->page()->console();
</span></span></pre></div>
<a id="trunkSourceWebKitmacChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/ChangeLog (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/ChangeLog 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebKit/mac/ChangeLog 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1,3 +1,14 @@
</span><ins>+2014-09-02 Brian J. Burg <burg@cs.washington.edu>
+
+ Web Inspector: fix prefixes for subclasses of JSC::ConsoleClient
+ https://bugs.webkit.org/show_bug.cgi?id=136476
+
+ Reviewed by Timothy Hatcher.
+
+ * Misc/WebCoreStatistics.mm:
+ (+[WebCoreStatistics shouldPrintExceptions]):
+ (+[WebCoreStatistics setShouldPrintExceptions:]):
+
</ins><span class="cx"> 2014-09-03 Enrica Casucci <enrica@apple.com>
</span><span class="cx">
</span><span class="cx"> Remove PLATFORM(IOS) from WebCore/editing (Part 1).
</span></span></pre></div>
<a id="trunkSourceWebKitmacMiscWebCoreStatisticsmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/Misc/WebCoreStatistics.mm (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/Misc/WebCoreStatistics.mm 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebKit/mac/Misc/WebCoreStatistics.mm 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -41,7 +41,7 @@
</span><span class="cx"> #import <WebCore/IconDatabase.h>
</span><span class="cx"> #import <WebCore/JSDOMWindow.h>
</span><span class="cx"> #import <WebCore/PageCache.h>
</span><del>-#import <WebCore/PageConsole.h>
</del><ins>+#import <WebCore/PageConsoleClient.h>
</ins><span class="cx"> #import <WebCore/PrintContext.h>
</span><span class="cx"> #import <WebCore/RenderTreeAsText.h>
</span><span class="cx"> #import <WebCore/RenderView.h>
</span><span class="lines">@@ -168,13 +168,13 @@
</span><span class="cx"> + (BOOL)shouldPrintExceptions
</span><span class="cx"> {
</span><span class="cx"> JSLockHolder lock(JSDOMWindow::commonVM());
</span><del>- return PageConsole::shouldPrintExceptions();
</del><ins>+ return PageConsoleClient::shouldPrintExceptions();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> + (void)setShouldPrintExceptions:(BOOL)print
</span><span class="cx"> {
</span><span class="cx"> JSLockHolder lock(JSDOMWindow::commonVM());
</span><del>- PageConsole::setShouldPrintExceptions(print);
</del><ins>+ PageConsoleClient::setShouldPrintExceptions(print);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> + (void)emptyCache
</span></span></pre></div>
<a id="trunkSourceWebKitwinChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/win/ChangeLog (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/win/ChangeLog 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebKit/win/ChangeLog 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -1,3 +1,14 @@
</span><ins>+2014-09-02 Brian J. Burg <burg@cs.washington.edu>
+
+ Web Inspector: fix prefixes for subclasses of JSC::ConsoleClient
+ https://bugs.webkit.org/show_bug.cgi?id=136476
+
+ Reviewed by Timothy Hatcher.
+
+ * WebCoreStatistics.cpp:
+ (WebCoreStatistics::shouldPrintExceptions):
+ (WebCoreStatistics::setShouldPrintExceptions):
+
</ins><span class="cx"> 2014-09-02 Maciej Stachowiak <mjs@apple.com>
</span><span class="cx">
</span><span class="cx"> Clean up naming for and slightly refactor legacy video fullscreen support
</span></span></pre></div>
<a id="trunkSourceWebKitwinWebCoreStatisticscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/win/WebCoreStatistics.cpp (173250 => 173251)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/win/WebCoreStatistics.cpp 2014-09-04 04:29:30 UTC (rev 173250)
+++ trunk/Source/WebKit/win/WebCoreStatistics.cpp 2014-09-04 05:53:43 UTC (rev 173251)
</span><span class="lines">@@ -36,7 +36,7 @@
</span><span class="cx"> #include <WebCore/IconDatabase.h>
</span><span class="cx"> #include <WebCore/JSDOMWindow.h>
</span><span class="cx"> #include <WebCore/PageCache.h>
</span><del>-#include <WebCore/PageConsole.h>
</del><ins>+#include <WebCore/PageConsoleClient.h>
</ins><span class="cx"> #include <WebCore/RenderView.h>
</span><span class="cx"> #include <WebCore/SharedBuffer.h>
</span><span class="cx">
</span><span class="lines">@@ -268,14 +268,14 @@
</span><span class="cx"> return E_POINTER;
</span><span class="cx">
</span><span class="cx"> JSLockHolder lock(JSDOMWindow::commonVM());
</span><del>- *shouldPrint = PageConsole::shouldPrintExceptions();
</del><ins>+ *shouldPrint = PageConsoleClient::shouldPrintExceptions();
</ins><span class="cx"> return S_OK;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> HRESULT WebCoreStatistics::setShouldPrintExceptions(BOOL print)
</span><span class="cx"> {
</span><span class="cx"> JSLockHolder lock(JSDOMWindow::commonVM());
</span><del>- PageConsole::setShouldPrintExceptions(print);
</del><ins>+ PageConsoleClient::setShouldPrintExceptions(print);
</ins><span class="cx"> return S_OK;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre>
</div>
</div>
</body>
</html>