<!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>[190527] trunk</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/190527">190527</a></dd>
<dt>Author</dt> <dd>joepeck@webkit.org</dd>
<dt>Date</dt> <dd>2015-10-02 15:24:07 -0700 (Fri, 02 Oct 2015)</dd>
</dl>
<h3>Log Message</h3>
<pre>Unreviewed, rolling out <a href="http://trac.webkit.org/projects/webkit/changeset/190520">r190520</a>, some tests assert / crash.
Source/JavaScriptCore:
* CMakeLists.txt:
* DerivedSources.make:
* JavaScriptCore.xcodeproj/project.pbxproj:
* heap/Heap.cpp:
(JSC::Heap::willStartCollection): Deleted.
(JSC::Heap::didFinishCollection): Deleted.
* heap/Heap.h:
(JSC::Heap::addObserver): Deleted.
(JSC::Heap::removeObserver): Deleted.
* heap/HeapObserver.h: Removed.
* heap/MarkedSpace.h:
* inspector/InspectorEnvironment.h:
* inspector/JSGlobalObjectInspectorController.cpp:
(Inspector::JSGlobalObjectInspectorController::JSGlobalObjectInspectorController):
(Inspector::JSGlobalObjectInspectorController::vm): Deleted.
* inspector/JSGlobalObjectInspectorController.h:
* inspector/agents/InspectorHeapAgent.cpp: Removed.
* inspector/agents/InspectorHeapAgent.h: Removed.
* inspector/protocol/Heap.json: Removed.
Source/WebCore:
* ForwardingHeaders/heap/HeapObserver.h: Removed.
* ForwardingHeaders/inspector/agents/InspectorHeapAgent.h: Removed.
* WebCore.vcxproj/WebCore.vcxproj:
* WebCore.vcxproj/WebCore.vcxproj.filters:
* inspector/InspectorController.cpp:
(WebCore::InspectorController::InspectorController): Deleted.
(WebCore::InspectorController::vm): Deleted.
* inspector/InspectorController.h:
* inspector/WorkerInspectorController.cpp:
(WebCore::WorkerInspectorController::vm): Deleted.
* inspector/WorkerInspectorController.h:
Source/WebInspectorUI:
* Localizations/en.lproj/localizedStrings.js:
* UserInterface/Base/Main.js:
(WebInspector.loaded): Deleted.
* UserInterface/Base/Utilities.js:
(value): Deleted.
* UserInterface/Controllers/HeapManager.js: Removed.
* UserInterface/Controllers/TimelineManager.js:
(WebInspector.TimelineManager): Deleted.
(WebInspector.TimelineManager.prototype._garbageCollected): Deleted.
* UserInterface/Images/TimelineRecordGarbageCollection.svg: Removed.
* UserInterface/Main.html:
* UserInterface/Models/GarbageCollection.js: Removed.
* UserInterface/Models/ScriptTimelineRecord.js:
(WebInspector.ScriptTimelineRecord.EventType.displayName):
(WebInspector.ScriptTimelineRecord.prototype.isGarbageCollection): Deleted.
* UserInterface/Protocol/HeapObserver.js: Removed.
* UserInterface/Test.html:
* UserInterface/Test/Test.js:
(WebInspector.loaded): Deleted.
* UserInterface/Views/ScriptTimelineDataGridNode.js:
(WebInspector.ScriptTimelineDataGridNode.prototype.get data):
(WebInspector.ScriptTimelineDataGridNode.prototype.createCellContent): Deleted.
(WebInspector.ScriptTimelineDataGridNode): Deleted.
* UserInterface/Views/ScriptTimelineOverviewGraph.js:
(WebInspector.ScriptTimelineOverviewGraph.prototype.updateLayout):
* UserInterface/Views/ScriptTimelineView.js:
(WebInspector.ScriptTimelineView.prototype._processPendingRecords):
* UserInterface/Views/SourceCodeTimelineTreeElement.js:
(WebInspector.SourceCodeTimelineTreeElement):
* UserInterface/Views/TimelineIcons.css:
(.garbage-collection-profile-record .icon): Deleted.
* UserInterface/Views/TimelineRecordBar.css:
(.timeline-record-bar.timeline-record-type-script.script-timeline-record-garbage-collected > .segment): Deleted.
* UserInterface/Views/TimelineRecordBar.js:
(WebInspector.TimelineRecordBar.createCombinedBars):
* UserInterface/Views/TimelineRecordTreeElement.js:
(WebInspector.TimelineRecordTreeElement):
LayoutTests:
* inspector/heap/garbageCollected-expected.txt: Removed.
* inspector/heap/garbageCollected.html: Removed.
* inspector/heap/gc-expected.txt: Removed.
* inspector/heap/gc.html: Removed.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>
<li><a href="#trunkSourceJavaScriptCoreCMakeListstxt">trunk/Source/JavaScriptCore/CMakeLists.txt</a></li>
<li><a href="#trunkSourceJavaScriptCoreChangeLog">trunk/Source/JavaScriptCore/ChangeLog</a></li>
<li><a href="#trunkSourceJavaScriptCoreDerivedSourcesmake">trunk/Source/JavaScriptCore/DerivedSources.make</a></li>
<li><a href="#trunkSourceJavaScriptCoreJavaScriptCorexcodeprojprojectpbxproj">trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapHeapcpp">trunk/Source/JavaScriptCore/heap/Heap.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapHeaph">trunk/Source/JavaScriptCore/heap/Heap.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreheapMarkedSpaceh">trunk/Source/JavaScriptCore/heap/MarkedSpace.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreinspectorInspectorEnvironmenth">trunk/Source/JavaScriptCore/inspector/InspectorEnvironment.h</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="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</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="#trunkSourceWebCoreinspectorInspectorControllercpp">trunk/Source/WebCore/inspector/InspectorController.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorControllerh">trunk/Source/WebCore/inspector/InspectorController.h</a></li>
<li><a href="#trunkSourceWebCoreinspectorWorkerInspectorControllercpp">trunk/Source/WebCore/inspector/WorkerInspectorController.cpp</a></li>
<li><a href="#trunkSourceWebCoreinspectorWorkerInspectorControllerh">trunk/Source/WebCore/inspector/WorkerInspectorController.h</a></li>
<li><a href="#trunkSourceWebInspectorUIChangeLog">trunk/Source/WebInspectorUI/ChangeLog</a></li>
<li><a href="#trunkSourceWebInspectorUILocalizationsenlprojlocalizedStringsjs">trunk/Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceBaseMainjs">trunk/Source/WebInspectorUI/UserInterface/Base/Main.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceBaseUtilitiesjs">trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceControllersTimelineManagerjs">trunk/Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceMainhtml">trunk/Source/WebInspectorUI/UserInterface/Main.html</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceModelsScriptTimelineRecordjs">trunk/Source/WebInspectorUI/UserInterface/Models/ScriptTimelineRecord.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceTestTestjs">trunk/Source/WebInspectorUI/UserInterface/Test/Test.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceTesthtml">trunk/Source/WebInspectorUI/UserInterface/Test.html</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsScriptTimelineDataGridNodejs">trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineDataGridNode.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsScriptTimelineOverviewGraphjs">trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineOverviewGraph.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsScriptTimelineViewjs">trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineView.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsSourceCodeTimelineTreeElementjs">trunk/Source/WebInspectorUI/UserInterface/Views/SourceCodeTimelineTreeElement.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsTimelineIconscss">trunk/Source/WebInspectorUI/UserInterface/Views/TimelineIcons.css</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsTimelineRecordBarcss">trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordBar.css</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsTimelineRecordBarjs">trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordBar.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsTimelineRecordTreeElementjs">trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordTreeElement.js</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li>trunk/LayoutTests/inspector/heap/</li>
<li><a href="#trunkSourceJavaScriptCoreheapHeapObserverh">trunk/Source/JavaScriptCore/heap/HeapObserver.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreinspectoragentsInspectorHeapAgentcpp">trunk/Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.cpp</a></li>
<li><a href="#trunkSourceJavaScriptCoreinspectoragentsInspectorHeapAgenth">trunk/Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.h</a></li>
<li><a href="#trunkSourceJavaScriptCoreinspectorprotocolHeapjson">trunk/Source/JavaScriptCore/inspector/protocol/Heap.json</a></li>
<li><a href="#trunkSourceWebCoreForwardingHeadersheapHeapObserverh">trunk/Source/WebCore/ForwardingHeaders/heap/HeapObserver.h</a></li>
<li><a href="#trunkSourceWebCoreForwardingHeadersinspectoragentsInspectorHeapAgenth">trunk/Source/WebCore/ForwardingHeaders/inspector/agents/InspectorHeapAgent.h</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceControllersHeapManagerjs">trunk/Source/WebInspectorUI/UserInterface/Controllers/HeapManager.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceImagesTimelineRecordGarbageCollectionsvg">trunk/Source/WebInspectorUI/UserInterface/Images/TimelineRecordGarbageCollection.svg</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceModelsGarbageCollectionjs">trunk/Source/WebInspectorUI/UserInterface/Models/GarbageCollection.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceProtocolHeapObserverjs">trunk/Source/WebInspectorUI/UserInterface/Protocol/HeapObserver.js</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/ChangeLog 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/LayoutTests/ChangeLog 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,5 +1,14 @@
</span><span class="cx"> 2015-10-02 Joseph Pecoraro <pecoraro@apple.com>
</span><span class="cx">
</span><ins>+ Unreviewed, rolling out r190520, some tests assert / crash.
+
+ * inspector/heap/garbageCollected-expected.txt: Removed.
+ * inspector/heap/garbageCollected.html: Removed.
+ * inspector/heap/gc-expected.txt: Removed.
+ * inspector/heap/gc.html: Removed.
+
+2015-10-02 Joseph Pecoraro <pecoraro@apple.com>
+
</ins><span class="cx"> Web Inspector: Include Garbage Collection Event in Timeline
</span><span class="cx"> https://bugs.webkit.org/show_bug.cgi?id=142510
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreCMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/CMakeLists.txt (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/CMakeLists.txt 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/CMakeLists.txt 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -347,7 +347,6 @@
</span><span class="cx"> inspector/agents/InspectorAgent.cpp
</span><span class="cx"> inspector/agents/InspectorConsoleAgent.cpp
</span><span class="cx"> inspector/agents/InspectorDebuggerAgent.cpp
</span><del>- inspector/agents/InspectorHeapAgent.cpp
</del><span class="cx"> inspector/agents/InspectorRuntimeAgent.cpp
</span><span class="cx"> inspector/agents/JSGlobalObjectConsoleAgent.cpp
</span><span class="cx"> inspector/agents/JSGlobalObjectDebuggerAgent.cpp
</span><span class="lines">@@ -1027,7 +1026,6 @@
</span><span class="cx"> ${JAVASCRIPTCORE_DIR}/inspector/protocol/Database.json
</span><span class="cx"> ${JAVASCRIPTCORE_DIR}/inspector/protocol/Debugger.json
</span><span class="cx"> ${JAVASCRIPTCORE_DIR}/inspector/protocol/GenericTypes.json
</span><del>- ${JAVASCRIPTCORE_DIR}/inspector/protocol/Heap.json
</del><span class="cx"> ${JAVASCRIPTCORE_DIR}/inspector/protocol/Inspector.json
</span><span class="cx"> ${JAVASCRIPTCORE_DIR}/inspector/protocol/LayerTree.json
</span><span class="cx"> ${JAVASCRIPTCORE_DIR}/inspector/protocol/Network.json
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/ChangeLog (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/ChangeLog 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/ChangeLog 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,3 +1,27 @@
</span><ins>+2015-10-02 Joseph Pecoraro <pecoraro@apple.com>
+
+ Unreviewed, rolling out r190520, some tests assert / crash.
+
+ * CMakeLists.txt:
+ * DerivedSources.make:
+ * JavaScriptCore.xcodeproj/project.pbxproj:
+ * heap/Heap.cpp:
+ (JSC::Heap::willStartCollection): Deleted.
+ (JSC::Heap::didFinishCollection): Deleted.
+ * heap/Heap.h:
+ (JSC::Heap::addObserver): Deleted.
+ (JSC::Heap::removeObserver): Deleted.
+ * heap/HeapObserver.h: Removed.
+ * heap/MarkedSpace.h:
+ * inspector/InspectorEnvironment.h:
+ * inspector/JSGlobalObjectInspectorController.cpp:
+ (Inspector::JSGlobalObjectInspectorController::JSGlobalObjectInspectorController):
+ (Inspector::JSGlobalObjectInspectorController::vm): Deleted.
+ * inspector/JSGlobalObjectInspectorController.h:
+ * inspector/agents/InspectorHeapAgent.cpp: Removed.
+ * inspector/agents/InspectorHeapAgent.h: Removed.
+ * inspector/protocol/Heap.json: Removed.
+
</ins><span class="cx"> 2015-10-01 Geoffrey Garen <ggaren@apple.com>
</span><span class="cx">
</span><span class="cx"> Unreviewed, rolling back in r190450
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreDerivedSourcesmake"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/DerivedSources.make (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/DerivedSources.make 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/DerivedSources.make 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -152,7 +152,6 @@
</span><span class="cx"> $(JavaScriptCore)/inspector/protocol/Database.json \
</span><span class="cx"> $(JavaScriptCore)/inspector/protocol/Debugger.json \
</span><span class="cx"> $(JavaScriptCore)/inspector/protocol/GenericTypes.json \
</span><del>- $(JavaScriptCore)/inspector/protocol/Heap.json \
</del><span class="cx"> $(JavaScriptCore)/inspector/protocol/Inspector.json \
</span><span class="cx"> $(JavaScriptCore)/inspector/protocol/LayerTree.json \
</span><span class="cx"> $(JavaScriptCore)/inspector/protocol/Network.json \
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreJavaScriptCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1290,9 +1290,6 @@
</span><span class="cx"> A532438C18568335002ED692 /* InspectorProtocolObjects.h in Headers */ = {isa = PBXBuildFile; fileRef = A532438618568317002ED692 /* InspectorProtocolObjects.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="cx"> A532439418569709002ED692 /* generate-combined-inspector-json.py in Headers */ = {isa = PBXBuildFile; fileRef = A5324391185696E6002ED692 /* generate-combined-inspector-json.py */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="cx"> A53243981856A489002ED692 /* CombinedDomains.json in Headers */ = {isa = PBXBuildFile; fileRef = A53243951856A475002ED692 /* CombinedDomains.json */; settings = {ATTRIBUTES = (Private, ); }; };
</span><del>- A5339EC61BB399A60054F005 /* InspectorHeapAgent.h in Headers */ = {isa = PBXBuildFile; fileRef = A5339EC51BB399900054F005 /* InspectorHeapAgent.h */; settings = {ATTRIBUTES = (Private, ); }; };
- A5339EC71BB399A90054F005 /* InspectorHeapAgent.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A5339EC41BB399900054F005 /* InspectorHeapAgent.cpp */; };
- A5339EC91BB4B4600054F005 /* HeapObserver.h in Headers */ = {isa = PBXBuildFile; fileRef = A5339EC81BB4B4510054F005 /* HeapObserver.h */; settings = {ATTRIBUTES = (Private, ); }; };
</del><span class="cx"> A53CE08518BC1A5600BEDF76 /* ConsolePrototype.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A53CE08118BC1A5600BEDF76 /* ConsolePrototype.cpp */; };
</span><span class="cx"> A53CE08618BC1A5600BEDF76 /* ConsolePrototype.h in Headers */ = {isa = PBXBuildFile; fileRef = A53CE08218BC1A5600BEDF76 /* ConsolePrototype.h */; };
</span><span class="cx"> A53CE08718BC1A5600BEDF76 /* JSConsole.cpp in Sources */ = {isa = PBXBuildFile; fileRef = A53CE08318BC1A5600BEDF76 /* JSConsole.cpp */; };
</span><span class="lines">@@ -3191,9 +3188,6 @@
</span><span class="cx"> A5324391185696E6002ED692 /* generate-combined-inspector-json.py */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.script.python; path = "generate-combined-inspector-json.py"; sourceTree = "<group>"; };
</span><span class="cx"> A53243951856A475002ED692 /* CombinedDomains.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = CombinedDomains.json; sourceTree = "<group>"; };
</span><span class="cx"> A53243961856A475002ED692 /* InspectorBackendCommands.js */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.javascript; path = InspectorBackendCommands.js; sourceTree = "<group>"; };
</span><del>- A5339EC41BB399900054F005 /* InspectorHeapAgent.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = InspectorHeapAgent.cpp; sourceTree = "<group>"; };
- A5339EC51BB399900054F005 /* InspectorHeapAgent.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = InspectorHeapAgent.h; sourceTree = "<group>"; };
- A5339EC81BB4B4510054F005 /* HeapObserver.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = HeapObserver.h; sourceTree = "<group>"; };
</del><span class="cx"> A53CE08118BC1A5600BEDF76 /* ConsolePrototype.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = ConsolePrototype.cpp; sourceTree = "<group>"; };
</span><span class="cx"> A53CE08218BC1A5600BEDF76 /* ConsolePrototype.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = ConsolePrototype.h; sourceTree = "<group>"; };
</span><span class="cx"> A53CE08318BC1A5600BEDF76 /* JSConsole.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSConsole.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -4282,7 +4276,6 @@
</span><span class="cx"> 0F32BD0F1BB34F190093A57F /* HeapHelperPool.h */,
</span><span class="cx"> C2DA778218E259990066FCB6 /* HeapInlines.h */,
</span><span class="cx"> 2AD8932917E3868F00668276 /* HeapIterationScope.h */,
</span><del>- A5339EC81BB4B4510054F005 /* HeapObserver.h */,
</del><span class="cx"> 2A6F462517E959CE00C45C98 /* HeapOperation.h */,
</span><span class="cx"> 14F97446138C853E00DA1C67 /* HeapRootVisitor.h */,
</span><span class="cx"> C24D31E0161CD695002AA4DB /* HeapStatistics.cpp */,
</span><span class="lines">@@ -5749,8 +5742,6 @@
</span><span class="cx"> A5FD0080189B191A00633231 /* InspectorConsoleAgent.h */,
</span><span class="cx"> A57D23E31890CEBF0031C7FA /* InspectorDebuggerAgent.cpp */,
</span><span class="cx"> A57D23E41890CEBF0031C7FA /* InspectorDebuggerAgent.h */,
</span><del>- A5339EC41BB399900054F005 /* InspectorHeapAgent.cpp */,
- A5339EC51BB399900054F005 /* InspectorHeapAgent.h */,
</del><span class="cx"> A50E4B5D18809DD50068A46D /* InspectorRuntimeAgent.cpp */,
</span><span class="cx"> A50E4B5E18809DD50068A46D /* InspectorRuntimeAgent.h */,
</span><span class="cx"> A5FD0083189B1B7E00633231 /* JSGlobalObjectConsoleAgent.cpp */,
</span><span class="lines">@@ -6429,7 +6420,6 @@
</span><span class="cx"> 14BA7A9813AADFF8005B7C2C /* Heap.h in Headers */,
</span><span class="cx"> C2DA778318E259990066FCB6 /* HeapInlines.h in Headers */,
</span><span class="cx"> 2AD8932B17E3868F00668276 /* HeapIterationScope.h in Headers */,
</span><del>- A5339EC91BB4B4600054F005 /* HeapObserver.h in Headers */,
</del><span class="cx"> 2A6F462617E959CE00C45C98 /* HeapOperation.h in Headers */,
</span><span class="cx"> 14F97447138C853E00DA1C67 /* HeapRootVisitor.h in Headers */,
</span><span class="cx"> C24D31E3161CD695002AA4DB /* HeapStatistics.h in Headers */,
</span><span class="lines">@@ -6472,7 +6462,6 @@
</span><span class="cx"> A5945595182479EB00CC3843 /* InspectorFrontendChannel.h in Headers */,
</span><span class="cx"> A532438A18568335002ED692 /* InspectorFrontendDispatchers.h in Headers */,
</span><span class="cx"> 99F1A7011B98FBEC00463B26 /* InspectorFrontendRouter.h in Headers */,
</span><del>- A5339EC61BB399A60054F005 /* InspectorHeapAgent.h in Headers */,
</del><span class="cx"> E35E03601B7AB43E0073AD2A /* InspectorInstrumentationObject.h in Headers */,
</span><span class="cx"> E33B3E261B7ABD750048DB2E /* InspectorInstrumentationObject.lut.h in Headers */,
</span><span class="cx"> A532438C18568335002ED692 /* InspectorProtocolObjects.h in Headers */,
</span><span class="lines">@@ -7784,7 +7773,6 @@
</span><span class="cx"> A57D23E51890CEBF0031C7FA /* InspectorDebuggerAgent.cpp in Sources */,
</span><span class="cx"> A532438918568335002ED692 /* InspectorFrontendDispatchers.cpp in Sources */,
</span><span class="cx"> 99F1A6FE1B8E6D9400463B26 /* InspectorFrontendRouter.cpp in Sources */,
</span><del>- A5339EC71BB399A90054F005 /* InspectorHeapAgent.cpp in Sources */,
</del><span class="cx"> E35E035F1B7AB43E0073AD2A /* InspectorInstrumentationObject.cpp in Sources */,
</span><span class="cx"> A532438B18568335002ED692 /* InspectorProtocolObjects.cpp in Sources */,
</span><span class="cx"> A50E4B6118809DD50068A46D /* InspectorRuntimeAgent.cpp in Sources */,
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapHeapcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/Heap.cpp (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/Heap.cpp 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/heap/Heap.cpp 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -38,10 +38,10 @@
</span><span class="cx"> #include "HeapVerifier.h"
</span><span class="cx"> #include "IncrementalSweeper.h"
</span><span class="cx"> #include "Interpreter.h"
</span><del>-#include "JSCInlines.h"
</del><span class="cx"> #include "JSGlobalObject.h"
</span><span class="cx"> #include "JSLock.h"
</span><span class="cx"> #include "JSONObject.h"
</span><ins>+#include "JSCInlines.h"
</ins><span class="cx"> #include "JSVirtualMachineInternal.h"
</span><span class="cx"> #include "RecursiveAllocationScope.h"
</span><span class="cx"> #include "RegExpCache.h"
</span><span class="lines">@@ -51,12 +51,13 @@
</span><span class="cx"> #include "VM.h"
</span><span class="cx"> #include "WeakSetInlines.h"
</span><span class="cx"> #include <algorithm>
</span><ins>+#include <wtf/RAMSize.h>
</ins><span class="cx"> #include <wtf/CurrentTime.h>
</span><span class="cx"> #include <wtf/ParallelVectorIterator.h>
</span><span class="cx"> #include <wtf/ProcessID.h>
</span><del>-#include <wtf/RAMSize.h>
</del><span class="cx">
</span><span class="cx"> using namespace std;
</span><ins>+using namespace JSC;
</ins><span class="cx">
</span><span class="cx"> namespace JSC {
</span><span class="cx">
</span><span class="lines">@@ -1167,9 +1168,6 @@
</span><span class="cx">
</span><span class="cx"> if (m_edenActivityCallback)
</span><span class="cx"> m_edenActivityCallback->willCollect();
</span><del>-
- for (auto* observer : m_observers)
- observer->willGarbageCollect();
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void Heap::flushOldStructureIDTables()
</span><span class="lines">@@ -1318,7 +1316,6 @@
</span><span class="cx"> {
</span><span class="cx"> GCPHASE(FinishingCollection);
</span><span class="cx"> double gcEndTime = WTF::monotonicallyIncreasingTime();
</span><del>- HeapOperation operation = m_operationInProgress;
</del><span class="cx"> if (m_operationInProgress == FullCollection)
</span><span class="cx"> m_lastFullGCLength = gcEndTime - gcStartTime;
</span><span class="cx"> else
</span><span class="lines">@@ -1342,9 +1339,6 @@
</span><span class="cx"> RELEASE_ASSERT(m_operationInProgress == EdenCollection || m_operationInProgress == FullCollection);
</span><span class="cx"> m_operationInProgress = NoOperation;
</span><span class="cx"> JAVASCRIPTCORE_GC_END();
</span><del>-
- for (auto* observer : m_observers)
- observer->didGarbageCollect(operation);
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void Heap::resumeCompilerThreads()
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapHeaph"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/Heap.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/Heap.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/heap/Heap.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -28,7 +28,6 @@
</span><span class="cx"> #include "GCIncomingRefCountedSet.h"
</span><span class="cx"> #include "HandleSet.h"
</span><span class="cx"> #include "HandleStack.h"
</span><del>-#include "HeapObserver.h"
</del><span class="cx"> #include "HeapOperation.h"
</span><span class="cx"> #include "JITStubRoutineSet.h"
</span><span class="cx"> #include "ListableHandler.h"
</span><span class="lines">@@ -50,24 +49,28 @@
</span><span class="cx">
</span><span class="cx"> namespace JSC {
</span><span class="cx">
</span><del>-class CodeBlock;
</del><span class="cx"> class CopiedSpace;
</span><del>-class EdenGCActivityCallback;
</del><ins>+class CodeBlock;
</ins><span class="cx"> class ExecutableBase;
</span><ins>+class EdenGCActivityCallback;
</ins><span class="cx"> class FullGCActivityCallback;
</span><span class="cx"> class GCActivityCallback;
</span><span class="cx"> class GCAwareJITStubRoutine;
</span><ins>+class GlobalCodeBlock;
</ins><span class="cx"> class Heap;
</span><span class="cx"> class HeapRootVisitor;
</span><span class="cx"> class HeapVerifier;
</span><span class="cx"> class IncrementalSweeper;
</span><span class="cx"> class JITStubRoutine;
</span><span class="cx"> class JSCell;
</span><ins>+class VM;
</ins><span class="cx"> class JSStack;
</span><span class="cx"> class JSValue;
</span><ins>+class LiveObjectIterator;
</ins><span class="cx"> class LLIntOffsetsExtractor;
</span><span class="cx"> class MarkedArgumentBuffer;
</span><del>-class VM;
</del><ins>+class WeakGCHandlePool;
+class SlotVisitor;
</ins><span class="cx">
</span><span class="cx"> namespace DFG {
</span><span class="cx"> class Worklist;
</span><span class="lines">@@ -130,9 +133,6 @@
</span><span class="cx"> JS_EXPORT_PRIVATE IncrementalSweeper* sweeper();
</span><span class="cx"> JS_EXPORT_PRIVATE void setIncrementalSweeper(std::unique_ptr<IncrementalSweeper>);
</span><span class="cx">
</span><del>- void addObserver(HeapObserver* observer) { m_observers.append(observer); }
- void removeObserver(HeapObserver* observer) { m_observers.removeFirst(observer); }
-
</del><span class="cx"> // true if collection is in progress
</span><span class="cx"> bool isCollecting();
</span><span class="cx"> HeapOperation operationInProgress() { return m_operationInProgress; }
</span><span class="lines">@@ -411,9 +411,7 @@
</span><span class="cx"> RefPtr<GCActivityCallback> m_edenActivityCallback;
</span><span class="cx"> std::unique_ptr<IncrementalSweeper> m_sweeper;
</span><span class="cx"> Vector<MarkedBlock*> m_blockSnapshot;
</span><del>-
- Vector<HeapObserver*> m_observers;
-
</del><ins>+
</ins><span class="cx"> unsigned m_deferralDepth;
</span><span class="cx"> Vector<DFG::Worklist*> m_suspendedCompilerWorklists;
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapHeapObserverh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/heap/HeapObserver.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/HeapObserver.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/heap/HeapObserver.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,42 +0,0 @@
</span><del>-/*
- * Copyright (C) 2015 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. 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 INC. 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 HeapObserver_h
-#define HeapObserver_h
-
-#include "HeapOperation.h"
-
-namespace JSC {
-
-class HeapObserver {
-public:
- virtual ~HeapObserver() { }
- virtual void willGarbageCollect() = 0;
- virtual void didGarbageCollect(HeapOperation) = 0;
-};
-
-} // namespace JSC
-
-#endif // HeapObserver_h
</del></span></pre></div>
<a id="trunkSourceJavaScriptCoreheapMarkedSpaceh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/heap/MarkedSpace.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/heap/MarkedSpace.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/heap/MarkedSpace.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -39,6 +39,7 @@
</span><span class="cx"> class Heap;
</span><span class="cx"> class HeapIterationScope;
</span><span class="cx"> class JSCell;
</span><ins>+class LiveObjectIterator;
</ins><span class="cx"> class LLIntOffsetsExtractor;
</span><span class="cx"> class WeakGCHandle;
</span><span class="cx"> class SlotVisitor;
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorInspectorEnvironmenth"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/inspector/InspectorEnvironment.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/InspectorEnvironment.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/inspector/InspectorEnvironment.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -35,7 +35,6 @@
</span><span class="cx"> namespace JSC {
</span><span class="cx"> class Exception;
</span><span class="cx"> class SourceCode;
</span><del>-class VM;
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> namespace Inspector {
</span><span class="lines">@@ -54,7 +53,6 @@
</span><span class="cx"> virtual void didCallInjectedScriptFunction(JSC::ExecState*) = 0;
</span><span class="cx"> virtual void frontendInitialized() = 0;
</span><span class="cx"> virtual Ref<WTF::Stopwatch> executionStopwatch() = 0;
</span><del>- virtual JSC::VM& vm() = 0;
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> } // namespace Inspector
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorJSGlobalObjectInspectorControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.cpp (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.cpp 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.cpp 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -36,7 +36,6 @@
</span><span class="cx"> #include "InspectorBackendDispatcher.h"
</span><span class="cx"> #include "InspectorFrontendChannel.h"
</span><span class="cx"> #include "InspectorFrontendRouter.h"
</span><del>-#include "InspectorHeapAgent.h"
</del><span class="cx"> #include "JSGlobalObject.h"
</span><span class="cx"> #include "JSGlobalObjectConsoleAgent.h"
</span><span class="cx"> #include "JSGlobalObjectConsoleClient.h"
</span><span class="lines">@@ -63,11 +62,13 @@
</span><span class="cx"> namespace Inspector {
</span><span class="cx">
</span><span class="cx"> JSGlobalObjectInspectorController::JSGlobalObjectInspectorController(JSGlobalObject& globalObject)
</span><del>- : m_globalObject(globalObject)
- , m_injectedScriptManager(std::make_unique<InjectedScriptManager>(*this, InjectedScriptHost::create()))
</del><ins>+ : m_injectedScriptManager(std::make_unique<InjectedScriptManager>(*this, InjectedScriptHost::create()))
</ins><span class="cx"> , m_executionStopwatch(Stopwatch::create())
</span><span class="cx"> , m_frontendRouter(FrontendRouter::create())
</span><span class="cx"> , m_backendDispatcher(BackendDispatcher::create(m_frontendRouter.copyRef()))
</span><ins>+#if ENABLE(REMOTE_INSPECTOR)
+ , m_globalObject(globalObject)
+#endif
</ins><span class="cx"> {
</span><span class="cx"> AgentContext baseContext = {
</span><span class="cx"> *this,
</span><span class="lines">@@ -85,11 +86,9 @@
</span><span class="cx"> auto runtimeAgent = std::make_unique<JSGlobalObjectRuntimeAgent>(context);
</span><span class="cx"> auto consoleAgent = std::make_unique<JSGlobalObjectConsoleAgent>(context);
</span><span class="cx"> auto debuggerAgent = std::make_unique<JSGlobalObjectDebuggerAgent>(context, consoleAgent.get());
</span><del>- auto heapAgent = std::make_unique<InspectorHeapAgent>(context);
</del><span class="cx">
</span><span class="cx"> m_inspectorAgent = inspectorAgent.get();
</span><span class="cx"> m_debuggerAgent = debuggerAgent.get();
</span><del>- m_heapAgent = heapAgent.get();
</del><span class="cx"> m_consoleAgent = consoleAgent.get();
</span><span class="cx"> m_consoleClient = std::make_unique<JSGlobalObjectConsoleClient>(m_consoleAgent);
</span><span class="cx">
</span><span class="lines">@@ -99,7 +98,6 @@
</span><span class="cx"> m_agents.append(WTF::move(runtimeAgent));
</span><span class="cx"> m_agents.append(WTF::move(consoleAgent));
</span><span class="cx"> m_agents.append(WTF::move(debuggerAgent));
</span><del>- m_agents.append(WTF::move(heapAgent));
</del><span class="cx">
</span><span class="cx"> m_executionStopwatch->start();
</span><span class="cx"> }
</span><span class="lines">@@ -285,11 +283,6 @@
</span><span class="cx"> return m_executionStopwatch.copyRef();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-VM& JSGlobalObjectInspectorController::vm()
-{
- return m_globalObject.vm();
-}
-
</del><span class="cx"> #if ENABLE(INSPECTOR_ALTERNATE_DISPATCHERS)
</span><span class="cx"> void JSGlobalObjectInspectorController::appendExtraAgent(std::unique_ptr<InspectorAgentBase> agent)
</span><span class="cx"> {
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorJSGlobalObjectInspectorControllerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/inspector/JSGlobalObjectInspectorController.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -41,6 +41,7 @@
</span><span class="cx"> class Stopwatch;
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+
</ins><span class="cx"> namespace JSC {
</span><span class="cx"> class ConsoleClient;
</span><span class="cx"> class Exception;
</span><span class="lines">@@ -57,7 +58,6 @@
</span><span class="cx"> class InspectorAgent;
</span><span class="cx"> class InspectorConsoleAgent;
</span><span class="cx"> class InspectorDebuggerAgent;
</span><del>-class InspectorHeapAgent;
</del><span class="cx"> class JSGlobalObjectConsoleClient;
</span><span class="cx"> class ScriptCallStack;
</span><span class="cx">
</span><span class="lines">@@ -97,7 +97,6 @@
</span><span class="cx"> virtual void didCallInjectedScriptFunction(JSC::ExecState*) override { }
</span><span class="cx"> virtual void frontendInitialized() override;
</span><span class="cx"> virtual Ref<WTF::Stopwatch> executionStopwatch() override;
</span><del>- virtual JSC::VM& vm() override;
</del><span class="cx">
</span><span class="cx"> #if ENABLE(INSPECTOR_ALTERNATE_DISPATCHERS)
</span><span class="cx"> virtual AugmentableInspectorControllerClient* augmentableInspectorControllerClient() const override { return m_augmentingClient; }
</span><span class="lines">@@ -111,7 +110,6 @@
</span><span class="cx"> private:
</span><span class="cx"> void appendAPIBacktrace(ScriptCallStack* callStack);
</span><span class="cx">
</span><del>- JSC::JSGlobalObject& m_globalObject;
</del><span class="cx"> std::unique_ptr<InjectedScriptManager> m_injectedScriptManager;
</span><span class="cx"> std::unique_ptr<JSGlobalObjectConsoleClient> m_consoleClient;
</span><span class="cx"> Ref<WTF::Stopwatch> m_executionStopwatch;
</span><span class="lines">@@ -120,7 +118,6 @@
</span><span class="cx"> InspectorAgent* m_inspectorAgent { nullptr };
</span><span class="cx"> InspectorConsoleAgent* m_consoleAgent { nullptr };
</span><span class="cx"> InspectorDebuggerAgent* m_debuggerAgent { nullptr };
</span><del>- InspectorHeapAgent* m_heapAgent { nullptr };
</del><span class="cx">
</span><span class="cx"> Ref<FrontendRouter> m_frontendRouter;
</span><span class="cx"> Ref<BackendDispatcher> m_backendDispatcher;
</span><span class="lines">@@ -128,6 +125,10 @@
</span><span class="cx"> bool m_includeNativeCallStackWithExceptions { false };
</span><span class="cx"> bool m_isAutomaticInspection { false };
</span><span class="cx">
</span><ins>+#if ENABLE(REMOTE_INSPECTOR)
+ JSC::JSGlobalObject& m_globalObject;
+#endif
+
</ins><span class="cx"> #if ENABLE(INSPECTOR_ALTERNATE_DISPATCHERS)
</span><span class="cx"> AugmentableInspectorControllerClient* m_augmentingClient { nullptr };
</span><span class="cx"> #endif
</span></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectoragentsInspectorHeapAgentcpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.cpp (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.cpp 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.cpp 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,126 +0,0 @@
</span><del>-/*
- * Copyright (C) 2015 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 "InspectorHeapAgent.h"
-
-#include "InspectorEnvironment.h"
-#include "VM.h"
-#include <wtf/Stopwatch.h>
-
-using namespace JSC;
-
-namespace Inspector {
-
-InspectorHeapAgent::InspectorHeapAgent(AgentContext& context)
- : InspectorAgentBase(ASCIILiteral("Heap"))
- , m_frontendDispatcher(std::make_unique<HeapFrontendDispatcher>(context.frontendRouter))
- , m_backendDispatcher(HeapBackendDispatcher::create(context.backendDispatcher, this))
- , m_environment(context.environment)
-{
-}
-
-InspectorHeapAgent::~InspectorHeapAgent()
-{
- ErrorString ignored;
- disable(ignored);
-}
-
-void InspectorHeapAgent::didCreateFrontendAndBackend(FrontendRouter*, BackendDispatcher*)
-{
-}
-
-void InspectorHeapAgent::willDestroyFrontendAndBackend(DisconnectReason)
-{
-}
-
-void InspectorHeapAgent::enable(ErrorString&)
-{
- if (m_enabled)
- return;
-
- m_enabled = true;
-
- m_environment.vm().heap.addObserver(this);
-}
-
-void InspectorHeapAgent::disable(ErrorString&)
-{
- if (!m_enabled)
- return;
-
- m_enabled = false;
-
- m_environment.vm().heap.removeObserver(this);
-}
-
-void InspectorHeapAgent::gc(ErrorString&)
-{
- VM& vm = m_environment.vm();
- JSLockHolder lock(vm);
- sanitizeStackForVM(&vm);
- vm.heap.collectAllGarbage();
-}
-
-static Inspector::Protocol::Heap::GarbageCollection::Type protocolTypeForHeapOperation(HeapOperation operation)
-{
- switch (operation) {
- case FullCollection:
- return Inspector::Protocol::Heap::GarbageCollection::Type::Full;
- case EdenCollection:
- return Inspector::Protocol::Heap::GarbageCollection::Type::Partial;
- default:
- ASSERT_NOT_REACHED();
- return Inspector::Protocol::Heap::GarbageCollection::Type::Full;
- }
-}
-
-void InspectorHeapAgent::willGarbageCollect()
-{
- ASSERT(m_enabled);
- ASSERT(std::isnan(m_gcStartTime));
-
- m_gcStartTime = m_environment.executionStopwatch()->elapsedTime();
-}
-
-void InspectorHeapAgent::didGarbageCollect(HeapOperation operation)
-{
- ASSERT(m_enabled);
- ASSERT(!std::isnan(m_gcStartTime));
-
- // FIXME: Include number of bytes freed by collection.
-
- double endTime = m_environment.executionStopwatch()->elapsedTime();
- auto collection = Inspector::Protocol::Heap::GarbageCollection::create()
- .setType(protocolTypeForHeapOperation(operation))
- .setStartTime(m_gcStartTime)
- .setEndTime(endTime)
- .release();
-
- m_frontendDispatcher->garbageCollected(WTF::move(collection));
- m_gcStartTime = NAN;
-}
-
-} // namespace Inspector
</del></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectoragentsInspectorHeapAgenth"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/inspector/agents/InspectorHeapAgent.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,68 +0,0 @@
</span><del>-/*
- * Copyright (C) 2015 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 InspectorHeapAgent_h
-#define InspectorHeapAgent_h
-
-#include "InspectorBackendDispatchers.h"
-#include "InspectorFrontendDispatchers.h"
-#include "heap/HeapObserver.h"
-#include "inspector/InspectorAgentBase.h"
-#include <wtf/Forward.h>
-#include <wtf/Noncopyable.h>
-
-namespace Inspector {
-
-typedef String ErrorString;
-
-class JS_EXPORT_PRIVATE InspectorHeapAgent final : public InspectorAgentBase, public HeapBackendDispatcherHandler, public JSC::HeapObserver {
- WTF_MAKE_NONCOPYABLE(InspectorHeapAgent);
-public:
- InspectorHeapAgent(AgentContext&);
- virtual ~InspectorHeapAgent();
-
- virtual void didCreateFrontendAndBackend(FrontendRouter*, BackendDispatcher*) override;
- virtual void willDestroyFrontendAndBackend(DisconnectReason) override;
-
- // HeapBackendDispatcherHandler
- virtual void enable(ErrorString&) override;
- virtual void disable(ErrorString&) override;
- virtual void gc(ErrorString&) override;
-
- // HeapObserver
- virtual void willGarbageCollect() override;
- virtual void didGarbageCollect(JSC::HeapOperation) override;
-
-private:
- std::unique_ptr<HeapFrontendDispatcher> m_frontendDispatcher;
- RefPtr<HeapBackendDispatcher> m_backendDispatcher;
- InspectorEnvironment& m_environment;
- bool m_enabled { false };
- double m_gcStartTime { NAN };
-};
-
-} // namespace Inspector
-
-#endif // InspectorHeapAgent_h
</del></span></pre></div>
<a id="trunkSourceJavaScriptCoreinspectorprotocolHeapjson"></a>
<div class="delfile"><h4>Deleted: trunk/Source/JavaScriptCore/inspector/protocol/Heap.json (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/JavaScriptCore/inspector/protocol/Heap.json 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/JavaScriptCore/inspector/protocol/Heap.json 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,39 +0,0 @@
</span><del>-{
- "domain": "Heap",
- "description": "Heap domain exposes JavaScript heap attributes and capabilities.",
- "types": [
- {
- "id": "GarbageCollection",
- "description": "Information about a garbage collection.",
- "type": "object",
- "properties": [
- { "name": "type", "type": "string", "enum": ["full", "partial"], "description": "The type of garbage collection." },
- { "name": "startTime", "type": "number" },
- { "name": "endTime", "type": "number" }
- ]
- }
- ],
- "commands": [
- {
- "name": "enable",
- "description": "Enables Heap domain events."
- },
- {
- "name": "disable",
- "description": "Disables Heap domain events."
- },
- {
- "name": "gc",
- "description": "Trigger a full garbage collection."
- }
- ],
- "events": [
- {
- "name": "garbageCollected",
- "description": "Information about the garbage collection.",
- "parameters": [
- { "name": "collection", "type": "GarbageCollection" }
- ]
- }
- ]
-}
</del></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/ChangeLog 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,5 +1,21 @@
</span><span class="cx"> 2015-10-02 Joseph Pecoraro <pecoraro@apple.com>
</span><span class="cx">
</span><ins>+ Unreviewed, rolling out r190520, some tests assert / crash.
+
+ * ForwardingHeaders/heap/HeapObserver.h: Removed.
+ * ForwardingHeaders/inspector/agents/InspectorHeapAgent.h: Removed.
+ * WebCore.vcxproj/WebCore.vcxproj:
+ * WebCore.vcxproj/WebCore.vcxproj.filters:
+ * inspector/InspectorController.cpp:
+ (WebCore::InspectorController::InspectorController): Deleted.
+ (WebCore::InspectorController::vm): Deleted.
+ * inspector/InspectorController.h:
+ * inspector/WorkerInspectorController.cpp:
+ (WebCore::WorkerInspectorController::vm): Deleted.
+ * inspector/WorkerInspectorController.h:
+
+2015-10-02 Joseph Pecoraro <pecoraro@apple.com>
+
</ins><span class="cx"> Web Inspector: Include Garbage Collection Event in Timeline
</span><span class="cx"> https://bugs.webkit.org/show_bug.cgi?id=142510
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreForwardingHeadersheapHeapObserverh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebCore/ForwardingHeaders/heap/HeapObserver.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ForwardingHeaders/heap/HeapObserver.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/ForwardingHeaders/heap/HeapObserver.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,4 +0,0 @@
</span><del>-#ifndef WebCore_FWD_HeapObserver_h
-#define WebCore_FWD_HeapObserver_h
-#include <JavaScriptCore/HeapObserver.h>
-#endif
</del></span></pre></div>
<a id="trunkSourceWebCoreForwardingHeadersinspectoragentsInspectorHeapAgenth"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebCore/ForwardingHeaders/inspector/agents/InspectorHeapAgent.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ForwardingHeaders/inspector/agents/InspectorHeapAgent.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/ForwardingHeaders/inspector/agents/InspectorHeapAgent.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,4 +0,0 @@
</span><del>-#ifndef WebCore_FWD_InspectorHeapAgent_h
-#define WebCore_FWD_InspectorHeapAgent_h
-#include <JavaScriptCore/InspectorHeapAgent.h>
-#endif
</del></span></pre></div>
<a id="trunkSourceWebCoreWebCorevcxprojWebCorevcxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -20355,7 +20355,6 @@
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\bindings\ScriptValue.h" />
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\heap\AllocationSpace.h" />
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\heap\Heap.h" />
</span><del>- <ClInclude Include="..\ForwardingHeaders\heap\HeapObserver.h" />
</del><span class="cx"> <ClInclude Include="..\ForwardingHeaders\heap\SlotVisitor.h" />
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\heap\StrongInlines.h" />
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\heap\Weak.h" />
</span><span class="lines">@@ -20385,7 +20384,6 @@
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\inspector\agents\InspectorAgent.h" />
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\inspector\agents\InspectorConsoleAgent.h" />
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\inspector\agents\InspectorDebuggerAgent.h" />
</span><del>- <ClInclude Include="..\ForwardingHeaders\inspector\agents\InspectorHeapAgent.h" />
</del><span class="cx"> <ClInclude Include="..\ForwardingHeaders\inspector\agents\InspectorRuntimeAgent.h" />
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\masm\X86Assembler.h" />
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\parser\SourceCode.h" />
</span></span></pre></div>
<a id="trunkSourceWebCoreWebCorevcxprojWebCorevcxprojfilters"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -11789,9 +11789,6 @@
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\inspector\agents\InspectorDebuggerAgent.h">
</span><span class="cx"> <Filter>ForwardingHeaders\inspector</Filter>
</span><span class="cx"> </ClInclude>
</span><del>- <ClInclude Include="..\ForwardingHeaders\inspector\agents\InspectorHeapAgent.h">
- <Filter>ForwardingHeaders\inspector</Filter>
- </ClInclude>
</del><span class="cx"> <ClInclude Include="..\ForwardingHeaders\inspector\agents\InspectorRuntimeAgent.h">
</span><span class="cx"> <Filter>ForwardingHeaders\inspector</Filter>
</span><span class="cx"> </ClInclude>
</span><span class="lines">@@ -12072,9 +12069,6 @@
</span><span class="cx"> <ClInclude Include="..\ForwardingHeaders\heap\Heap.h">
</span><span class="cx"> <Filter>ForwardingHeaders\heap</Filter>
</span><span class="cx"> </ClInclude>
</span><del>- <ClInclude Include="..\ForwardingHeaders\heap\HeapObserver.h">
- <Filter>ForwardingHeaders\heap</Filter>
- </ClInclude>
</del><span class="cx"> <ClInclude Include="..\ForwardingHeaders\heap\SlotVisitor.h">
</span><span class="cx"> <Filter>ForwardingHeaders\heap</Filter>
</span><span class="cx"> </ClInclude>
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorController.cpp (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorController.cpp 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/inspector/InspectorController.cpp 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -70,7 +70,6 @@
</span><span class="cx"> #include <inspector/InspectorFrontendDispatchers.h>
</span><span class="cx"> #include <inspector/InspectorFrontendRouter.h>
</span><span class="cx"> #include <inspector/agents/InspectorAgent.h>
</span><del>-#include <inspector/agents/InspectorHeapAgent.h>
</del><span class="cx"> #include <profiler/LegacyProfiler.h>
</span><span class="cx"> #include <runtime/JSLock.h>
</span><span class="cx"> #include <wtf/Stopwatch.h>
</span><span class="lines">@@ -171,7 +170,6 @@
</span><span class="cx"> m_domDebuggerAgent = domDebuggerAgentPtr.get();
</span><span class="cx"> m_agents.append(WTF::move(domDebuggerAgentPtr));
</span><span class="cx">
</span><del>- m_agents.append(std::make_unique<InspectorHeapAgent>(pageContext));
</del><span class="cx"> m_agents.append(std::make_unique<InspectorApplicationCacheAgent>(pageContext, pageAgent));
</span><span class="cx"> m_agents.append(std::make_unique<InspectorWorkerAgent>(pageContext));
</span><span class="cx"> m_agents.append(std::make_unique<InspectorLayerTreeAgent>(pageContext));
</span><span class="lines">@@ -476,11 +474,6 @@
</span><span class="cx"> return m_executionStopwatch.copyRef();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-JSC::VM& InspectorController::vm()
-{
- return JSDOMWindowBase::commonVM();
-}
-
</del><span class="cx"> void InspectorController::didComposite(Frame& frame)
</span><span class="cx"> {
</span><span class="cx"> InspectorInstrumentation::didComposite(frame);
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorControllerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorController.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorController.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/inspector/InspectorController.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -131,7 +131,6 @@
</span><span class="cx"> virtual void didCallInjectedScriptFunction(JSC::ExecState*) override;
</span><span class="cx"> virtual void frontendInitialized() override;
</span><span class="cx"> virtual Ref<WTF::Stopwatch> executionStopwatch() override;
</span><del>- virtual JSC::VM& vm() override;
</del><span class="cx">
</span><span class="cx"> WEBCORE_EXPORT void didComposite(Frame&);
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorWorkerInspectorControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/WorkerInspectorController.cpp (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/WorkerInspectorController.cpp 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/inspector/WorkerInspectorController.cpp 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -36,7 +36,6 @@
</span><span class="cx"> #include "InspectorInstrumentation.h"
</span><span class="cx"> #include "InspectorTimelineAgent.h"
</span><span class="cx"> #include "InstrumentingAgents.h"
</span><del>-#include "JSDOMWindowBase.h"
</del><span class="cx"> #include "JSMainThreadExecState.h"
</span><span class="cx"> #include "WebInjectedScriptHost.h"
</span><span class="cx"> #include "WebInjectedScriptManager.h"
</span><span class="lines">@@ -52,7 +51,6 @@
</span><span class="cx"> #include <inspector/InspectorFrontendRouter.h>
</span><span class="cx"> #include <wtf/Stopwatch.h>
</span><span class="cx">
</span><del>-using namespace JSC;
</del><span class="cx"> using namespace Inspector;
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="lines">@@ -197,9 +195,4 @@
</span><span class="cx"> return m_executionStopwatch.copyRef();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-VM& WorkerInspectorController::vm()
-{
- return JSDOMWindowBase::commonVM();
-}
-
</del><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorWorkerInspectorControllerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/WorkerInspectorController.h (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/WorkerInspectorController.h 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebCore/inspector/WorkerInspectorController.h 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -75,7 +75,6 @@
</span><span class="cx"> virtual void didCallInjectedScriptFunction(JSC::ExecState*) override;
</span><span class="cx"> virtual void frontendInitialized() override { }
</span><span class="cx"> virtual Ref<WTF::Stopwatch> executionStopwatch() override;
</span><del>- virtual JSC::VM& vm() override;
</del><span class="cx">
</span><span class="cx"> private:
</span><span class="cx"> friend class InspectorInstrumentation;
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/ChangeLog (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/ChangeLog 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/ChangeLog 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,3 +1,45 @@
</span><ins>+2015-10-02 Joseph Pecoraro <pecoraro@apple.com>
+
+ Unreviewed, rolling out r190520, some tests assert / crash.
+
+ * Localizations/en.lproj/localizedStrings.js:
+ * UserInterface/Base/Main.js:
+ (WebInspector.loaded): Deleted.
+ * UserInterface/Base/Utilities.js:
+ (value): Deleted.
+ * UserInterface/Controllers/HeapManager.js: Removed.
+ * UserInterface/Controllers/TimelineManager.js:
+ (WebInspector.TimelineManager): Deleted.
+ (WebInspector.TimelineManager.prototype._garbageCollected): Deleted.
+ * UserInterface/Images/TimelineRecordGarbageCollection.svg: Removed.
+ * UserInterface/Main.html:
+ * UserInterface/Models/GarbageCollection.js: Removed.
+ * UserInterface/Models/ScriptTimelineRecord.js:
+ (WebInspector.ScriptTimelineRecord.EventType.displayName):
+ (WebInspector.ScriptTimelineRecord.prototype.isGarbageCollection): Deleted.
+ * UserInterface/Protocol/HeapObserver.js: Removed.
+ * UserInterface/Test.html:
+ * UserInterface/Test/Test.js:
+ (WebInspector.loaded): Deleted.
+ * UserInterface/Views/ScriptTimelineDataGridNode.js:
+ (WebInspector.ScriptTimelineDataGridNode.prototype.get data):
+ (WebInspector.ScriptTimelineDataGridNode.prototype.createCellContent): Deleted.
+ (WebInspector.ScriptTimelineDataGridNode): Deleted.
+ * UserInterface/Views/ScriptTimelineOverviewGraph.js:
+ (WebInspector.ScriptTimelineOverviewGraph.prototype.updateLayout):
+ * UserInterface/Views/ScriptTimelineView.js:
+ (WebInspector.ScriptTimelineView.prototype._processPendingRecords):
+ * UserInterface/Views/SourceCodeTimelineTreeElement.js:
+ (WebInspector.SourceCodeTimelineTreeElement):
+ * UserInterface/Views/TimelineIcons.css:
+ (.garbage-collection-profile-record .icon): Deleted.
+ * UserInterface/Views/TimelineRecordBar.css:
+ (.timeline-record-bar.timeline-record-type-script.script-timeline-record-garbage-collected > .segment): Deleted.
+ * UserInterface/Views/TimelineRecordBar.js:
+ (WebInspector.TimelineRecordBar.createCombinedBars):
+ * UserInterface/Views/TimelineRecordTreeElement.js:
+ (WebInspector.TimelineRecordTreeElement):
+
</ins><span class="cx"> 2015-10-02 Devin Rousso <dcrousso+webkit@gmail.com>
</span><span class="cx">
</span><span class="cx"> Web Inspector: Option-Click on a property name in the Computed view should jump to resource view
</span></span></pre></div>
<a id="trunkSourceWebInspectorUILocalizationsenlprojlocalizedStringsjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -291,11 +291,9 @@
</span><span class="cx"> localizedStrings["Frames"] = "Frames";
</span><span class="cx"> localizedStrings["Frames: %d %d (%s %s)"] = "Frames: %d %d (%s %s)";
</span><span class="cx"> localizedStrings["Frames: None Selected"] = "Frames: None Selected";
</span><del>-localizedStrings["Full Garbage Collection"] = "Full Garbage Collection";
</del><span class="cx"> localizedStrings["Full URL"] = "Full URL";
</span><span class="cx"> localizedStrings["Function"] = "Function";
</span><span class="cx"> localizedStrings["Function Name Variable"] = "Function Name Variable";
</span><del>-localizedStrings["Garbage Collection"] = "Garbage Collection";
</del><span class="cx"> localizedStrings["Getter"] = "Getter";
</span><span class="cx"> localizedStrings["Global Breakpoints"] = "Global Breakpoints";
</span><span class="cx"> localizedStrings["Global Variables"] = "Global Variables";
</span><span class="lines">@@ -435,7 +433,6 @@
</span><span class="cx"> localizedStrings["Paint"] = "Paint";
</span><span class="cx"> localizedStrings["Paints"] = "Paints";
</span><span class="cx"> localizedStrings["Parent"] = "Parent";
</span><del>-localizedStrings["Partial Garbage Collection"] = "Partial Garbage Collection";
</del><span class="cx"> localizedStrings["Path"] = "Path";
</span><span class="cx"> localizedStrings["Pause Playback"] = "Pause Playback";
</span><span class="cx"> localizedStrings["Pause Reason"] = "Pause Reason";
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceBaseMainjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Main.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Base/Main.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Main.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -68,8 +68,6 @@
</span><span class="cx"> InspectorBackend.registerDOMDispatcher(new WebInspector.DOMObserver);
</span><span class="cx"> if (InspectorBackend.registerDebuggerDispatcher)
</span><span class="cx"> InspectorBackend.registerDebuggerDispatcher(new WebInspector.DebuggerObserver);
</span><del>- if (InspectorBackend.registerHeapDispatcher)
- InspectorBackend.registerHeapDispatcher(new WebInspector.HeapObserver);
</del><span class="cx"> if (InspectorBackend.registerDatabaseDispatcher)
</span><span class="cx"> InspectorBackend.registerDatabaseDispatcher(new WebInspector.DatabaseObserver);
</span><span class="cx"> if (InspectorBackend.registerDOMStorageDispatcher)
</span><span class="lines">@@ -110,7 +108,6 @@
</span><span class="cx"> this.issueManager = new WebInspector.IssueManager;
</span><span class="cx"> this.analyzerManager = new WebInspector.AnalyzerManager;
</span><span class="cx"> this.runtimeManager = new WebInspector.RuntimeManager;
</span><del>- this.heapManager = new WebInspector.HeapManager;
</del><span class="cx"> this.applicationCacheManager = new WebInspector.ApplicationCacheManager;
</span><span class="cx"> this.timelineManager = new WebInspector.TimelineManager;
</span><span class="cx"> this.debuggerManager = new WebInspector.DebuggerManager;
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceBaseUtilitiesjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -452,23 +452,6 @@
</span><span class="cx"> }
</span><span class="cx"> });
</span><span class="cx">
</span><del>-Object.defineProperty(Array.prototype, "partition",
-{
- value: function(callback)
- {
- let positive = [];
- let negative = [];
- for (let i = 0; i < this.length; ++i) {
- let value = this[i];
- if (callback(value))
- positive.push(value);
- else
- negative.push(value);
- }
- return [positive, negative];
- }
-});
-
</del><span class="cx"> Object.defineProperty(String.prototype, "trimMiddle",
</span><span class="cx"> {
</span><span class="cx"> value: function(maxLength)
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceControllersHeapManagerjs"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebInspectorUI/UserInterface/Controllers/HeapManager.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Controllers/HeapManager.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/HeapManager.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,49 +0,0 @@
</span><del>-/*
- * Copyright (C) 2015 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. 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 INC. 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.
- */
-
-WebInspector.HeapManager = class HeapManager extends WebInspector.Object
-{
- constructor()
- {
- super();
-
- if (window.HeapAgent)
- HeapAgent.enable();
- }
-
- // Public
-
- garbageCollected(payload)
- {
- // Called from WebInspector.HeapObserver.
-
- let collection = WebInspector.GarbageCollection.fromPayload(payload);
- this.dispatchEventToListeners(WebInspector.HeapManager.Event.GarbageCollected, {collection});
- }
-};
-
-WebInspector.HeapManager.Event = {
- GarbageCollected: "heap-manager-garbage-collected"
-};
</del></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceControllersTimelineManagerjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -33,8 +33,6 @@
</span><span class="cx"> WebInspector.Frame.addEventListener(WebInspector.Frame.Event.MainResourceDidChange, this._mainResourceDidChange, this);
</span><span class="cx"> WebInspector.Frame.addEventListener(WebInspector.Frame.Event.ResourceWasAdded, this._resourceWasAdded, this);
</span><span class="cx">
</span><del>- WebInspector.heapManager.addEventListener(WebInspector.HeapManager.Event.GarbageCollected, this._garbageCollected, this);
-
</del><span class="cx"> this._persistentNetworkTimeline = new WebInspector.NetworkTimeline;
</span><span class="cx">
</span><span class="cx"> this._isCapturing = false;
</span><span class="lines">@@ -595,15 +593,6 @@
</span><span class="cx">
</span><span class="cx"> this._addRecord(record);
</span><span class="cx"> }
</span><del>-
- _garbageCollected(event)
- {
- if (!this._isCapturing)
- return;
-
- let collection = event.data.collection;
- this._addRecord(new WebInspector.ScriptTimelineRecord(WebInspector.ScriptTimelineRecord.EventType.GarbageCollected, collection.startTime, collection.endTime, null, null, collection));
- }
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> WebInspector.TimelineManager.Event = {
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceImagesTimelineRecordGarbageCollectionsvg"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebInspectorUI/UserInterface/Images/TimelineRecordGarbageCollection.svg (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Images/TimelineRecordGarbageCollection.svg 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Images/TimelineRecordGarbageCollection.svg 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,8 +0,0 @@
</span><del>-<?xml version="1.0" encoding="utf-8"?>
-<!-- Copyright © 2015 Apple Inc. All rights reserved. -->
-<svg version="1.1" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16">
- <path fill="rgb(228, 198, 172)" d="M 13 1 L 3 1 C 1.898438 1 1 1.898438 1 3 L 1 13 C 1 14.101562 1.898438 15 3 15 L 13 15 C 14.101562 15 15 14.101562 15 13 L 15 3 C 15 1.898438 14.101562 1 13 1 Z"/>
- <path fill="rgb(186, 157, 132)" d="M 13 1 L 3 1 C 1.898438 1 1 1.898438 1 3 L 1 13 C 1 14.101562 1.898438 15 3 15 L 13 15 C 14.101562 15 15 14.101562 15 13 L 15 3 C 15 1.898438 14.101562 1 13 1 M 13 2 C 13.550781 2 14 2.449219 14 3 L 14 13 C 14 13.550781 13.550781 14 13 14 L 3 14 C 2.449219 14 2 13.550781 2 13 L 2 3 C 2 2.449219 2.449219 2 3 2 L 13 2"/>
- <path fill="rgb(186, 157, 132)" d="M 4.773438 11.542969 C 3.855469 10.621094 3.390625 9.347656 3.390625 7.769531 C 3.390625 6.15625 3.863281 4.878906 4.789062 3.964844 C 5.710938 3.054688 7.003906 2.59375 8.625 2.59375 C 9.570312 2.59375 10.519531 2.707031 11.441406 2.929688 C 11.890625 3.035156 12.210938 3.4375 12.210938 3.902344 L 12.210938 5.195312 C 12.210938 5.519531 12.050781 5.824219 11.789062 6.011719 C 11.613281 6.132812 11.410156 6.195312 11.207031 6.195312 C 11.097656 6.195312 10.984375 6.179688 10.878906 6.140625 C 10.015625 5.835938 9.308594 5.683594 8.769531 5.683594 C 8.058594 5.683594 7.753906 5.941406 7.589844 6.132812 C 7.296875 6.476562 7.148438 7.019531 7.148438 7.746094 C 7.148438 8.445312 7.308594 8.972656 7.621094 9.320312 C 7.808594 9.523438 8.148438 9.796875 8.921875 9.796875 C 9.542969 9.796875 10.1875 9.632812 10.839844 9.308594 C 10.980469 9.242188 11.132812 9.207031 11.285156 9.207031 C 11.46875 9.207031 11.648438 9.257812 1
1.8125 9.359375 C 12.105469 9.539062 12.285156 9.859375 12.285156 10.207031 L 12.285156 11.425781 C 12.285156 11.820312 12.050781 12.179688 11.691406 12.339844 C 10.792969 12.738281 9.730469 12.941406 8.527344 12.941406 C 6.957031 12.941406 5.691406 12.472656 4.773438 11.542969"/>
- <path fill="white" d="M 11.285156 11.425781 C 10.515625 11.769531 9.597656 11.941406 8.527344 11.941406 C 7.226562 11.941406 6.210938 11.574219 5.484375 10.839844 C 4.757812 10.105469 4.394531 9.082031 4.394531 7.769531 C 4.394531 6.429688 4.757812 5.398438 5.492188 4.675781 C 6.222656 3.957031 7.269531 3.59375 8.625 3.59375 C 9.492188 3.59375 10.351562 3.695312 11.210938 3.902344 L 11.210938 5.195312 C 10.238281 4.855469 9.425781 4.683594 8.769531 4.683594 C 7.933594 4.683594 7.285156 4.949219 6.832031 5.484375 C 6.375 6.015625 6.148438 6.769531 6.148438 7.746094 C 6.148438 8.707031 6.394531 9.453125 6.882812 9.992188 C 7.371094 10.527344 8.050781 10.796875 8.921875 10.796875 C 9.703125 10.796875 10.488281 10.601562 11.285156 10.207031 Z"/>
-</svg>
</del></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceMainhtml"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Main.html (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Main.html 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Main.html 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -226,7 +226,6 @@
</span><span class="cx"> <script src="Protocol/DOMStorageObserver.js"></script>
</span><span class="cx"> <script src="Protocol/DatabaseObserver.js"></script>
</span><span class="cx"> <script src="Protocol/DebuggerObserver.js"></script>
</span><del>- <script src="Protocol/HeapObserver.js"></script>
</del><span class="cx"> <script src="Protocol/InspectorBackend.js"></script>
</span><span class="cx"> <script src="Protocol/InspectorFrontendAPI.js"></script>
</span><span class="cx"> <script src="Protocol/InspectorObserver.js"></script>
</span><span class="lines">@@ -280,7 +279,6 @@
</span><span class="cx"> <script src="Models/ExecutionContext.js"></script>
</span><span class="cx"> <script src="Models/ExecutionContextList.js"></script>
</span><span class="cx"> <script src="Models/Frame.js"></script>
</span><del>- <script src="Models/GarbageCollection.js"></script>
</del><span class="cx"> <script src="Models/Geometry.js"></script>
</span><span class="cx"> <script src="Models/Gradient.js"></script>
</span><span class="cx"> <script src="Models/IndexedDatabase.js"></script>
</span><span class="lines">@@ -601,7 +599,6 @@
</span><span class="cx"> <script src="Controllers/FormatterContentBuilder.js"></script>
</span><span class="cx"> <script src="Controllers/FormatterSourceMap.js"></script>
</span><span class="cx"> <script src="Controllers/FrameResourceManager.js"></script>
</span><del>- <script src="Controllers/HeapManager.js"></script>
</del><span class="cx"> <script src="Controllers/IssueManager.js"></script>
</span><span class="cx"> <script src="Controllers/JavaScriptLogViewController.js"></script>
</span><span class="cx"> <script src="Controllers/JavaScriptRuntimeCompletionProvider.js"></script>
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceModelsGarbageCollectionjs"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebInspectorUI/UserInterface/Models/GarbageCollection.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Models/GarbageCollection.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/GarbageCollection.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,76 +0,0 @@
</span><del>-/*
- * Copyright (C) 2015 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. 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 INC. 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.
- */
-
-WebInspector.GarbageCollection = class GarbageCollection extends WebInspector.Object
-{
- constructor(type, startTime, endTime)
- {
- super();
-
- console.assert(endTime > startTime);
-
- this._type = type;
- this._startTime = startTime;
- this._endTime = endTime;
- }
-
- // Static
-
- static fromPayload(payload)
- {
- let type = WebInspector.GarbageCollection.Type.Full;
- if (payload.type === HeapAgent.GarbageCollectionType.Partial)
- type = WebInspector.GarbageCollection.Type.Partial;
-
- return new WebInspector.GarbageCollection(type, payload.startTime, payload.endTime);
- }
-
- // Public
-
- get type()
- {
- return this._type;
- }
-
- get startTime()
- {
- return this._startTime;
- }
-
- get endTime()
- {
- return this._endTime;
- }
-
- get duration()
- {
- return this._endTime - this._startTime;
- }
-};
-
-WebInspector.GarbageCollection.Type = {
- Partial: Symbol("Partial"),
- Full: Symbol("Full")
-};
</del></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceModelsScriptTimelineRecordjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Models/ScriptTimelineRecord.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Models/ScriptTimelineRecord.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/ScriptTimelineRecord.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -58,11 +58,6 @@
</span><span class="cx"> return this._profile;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- isGarbageCollection()
- {
- return this._eventType === WebInspector.ScriptTimelineRecord.EventType.GarbageCollected;
- }
-
</del><span class="cx"> saveIdentityToCookie(cookie)
</span><span class="cx"> {
</span><span class="cx"> super.saveIdentityToCookie(cookie);
</span><span class="lines">@@ -163,11 +158,10 @@
</span><span class="cx"> AnimationFrameFired: "script-timeline-record-animation-frame-fired",
</span><span class="cx"> AnimationFrameRequested: "script-timeline-record-animation-frame-requested",
</span><span class="cx"> AnimationFrameCanceled: "script-timeline-record-animation-frame-canceled",
</span><del>- ConsoleProfileRecorded: "script-timeline-record-console-profile-recorded",
- GarbageCollected: "script-timeline-record-garbage-collected"
</del><ins>+ ConsoleProfileRecorded: "script-timeline-record-console-profile-recorded"
</ins><span class="cx"> };
</span><span class="cx">
</span><del>-WebInspector.ScriptTimelineRecord.EventType.displayName = function(eventType, details, includeDetailsInformationInMainTitle)
</del><ins>+WebInspector.ScriptTimelineRecord.EventType.displayName = function(eventType, details, includeTimerIdentifierInMainTitle)
</ins><span class="cx"> {
</span><span class="cx"> if (details && !WebInspector.ScriptTimelineRecord._eventDisplayNames) {
</span><span class="cx"> // These display names are not localized because they closely represent
</span><span class="lines">@@ -341,27 +335,16 @@
</span><span class="cx"> if (details && (details instanceof String || typeof details === "string"))
</span><span class="cx"> return WebInspector.UIString("“%s” Profile Recorded").format(details);
</span><span class="cx"> return WebInspector.UIString("Console Profile Recorded");
</span><del>- case WebInspector.ScriptTimelineRecord.EventType.GarbageCollected:
- console.assert(details);
- if (details && (details instanceof WebInspector.GarbageCollection) && includeDetailsInformationInMainTitle) {
- switch (details.type) {
- case WebInspector.GarbageCollection.Type.Partial:
- return WebInspector.UIString("Partial Garbage Collection");
- case WebInspector.GarbageCollection.Type.Full:
- return WebInspector.UIString("Full Garbage Collection");
- }
- }
- return WebInspector.UIString("Garbage Collection");
</del><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.TimerFired:
</span><del>- if (details && includeDetailsInformationInMainTitle)
</del><ins>+ if (details && includeTimerIdentifierInMainTitle)
</ins><span class="cx"> return WebInspector.UIString("Timer %s Fired").format(details);
</span><span class="cx"> return WebInspector.UIString("Timer Fired");
</span><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.TimerInstalled:
</span><del>- if (details && includeDetailsInformationInMainTitle)
</del><ins>+ if (details && includeTimerIdentifierInMainTitle)
</ins><span class="cx"> return WebInspector.UIString("Timer %s Installed").format(details);
</span><span class="cx"> return WebInspector.UIString("Timer Installed");
</span><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.TimerRemoved:
</span><del>- if (details && includeDetailsInformationInMainTitle)
</del><ins>+ if (details && includeTimerIdentifierInMainTitle)
</ins><span class="cx"> return WebInspector.UIString("Timer %s Removed").format(details);
</span><span class="cx"> return WebInspector.UIString("Timer Removed");
</span><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.AnimationFrameFired:
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceProtocolHeapObserverjs"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebInspectorUI/UserInterface/Protocol/HeapObserver.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Protocol/HeapObserver.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Protocol/HeapObserver.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -1,34 +0,0 @@
</span><del>-/*
- * Copyright (C) 2015 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. 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 INC. 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.
- */
-
-WebInspector.HeapObserver = class HeapObserver
-{
- // Events defined by the "Heap" domain.
-
- garbageCollected(collection)
- {
- WebInspector.heapManager.garbageCollected(collection);
- }
-};
</del></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceTestTestjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Test/Test.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Test/Test.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Test/Test.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -41,7 +41,6 @@
</span><span class="cx"> InspectorBackend.registerDOMDispatcher(new WebInspector.DOMObserver);
</span><span class="cx"> InspectorBackend.registerNetworkDispatcher(new WebInspector.NetworkObserver);
</span><span class="cx"> InspectorBackend.registerDebuggerDispatcher(new WebInspector.DebuggerObserver);
</span><del>- InspectorBackend.registerHeapDispatcher(new WebInspector.HeapObserver);
</del><span class="cx"> InspectorBackend.registerTimelineDispatcher(new WebInspector.TimelineObserver);
</span><span class="cx"> InspectorBackend.registerCSSDispatcher(new WebInspector.CSSObserver);
</span><span class="cx"> InspectorBackend.registerRuntimeDispatcher(new WebInspector.RuntimeObserver);
</span><span class="lines">@@ -55,7 +54,6 @@
</span><span class="cx"> this.logManager = new WebInspector.LogManager;
</span><span class="cx"> this.issueManager = new WebInspector.IssueManager;
</span><span class="cx"> this.runtimeManager = new WebInspector.RuntimeManager;
</span><del>- this.heapManager = new WebInspector.HeapManager;
</del><span class="cx"> this.timelineManager = new WebInspector.TimelineManager;
</span><span class="cx"> this.debuggerManager = new WebInspector.DebuggerManager;
</span><span class="cx"> this.probeManager = new WebInspector.ProbeManager;
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceTesthtml"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Test.html (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Test.html 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Test.html 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -62,7 +62,6 @@
</span><span class="cx"> <script src="Protocol/DOMObserver.js"></script>
</span><span class="cx"> <script src="Protocol/DOMStorageObserver.js"></script>
</span><span class="cx"> <script src="Protocol/DebuggerObserver.js"></script>
</span><del>- <script src="Protocol/HeapObserver.js"></script>
</del><span class="cx"> <script src="Protocol/NetworkObserver.js"></script>
</span><span class="cx"> <script src="Protocol/PageObserver.js"></script>
</span><span class="cx"> <script src="Protocol/RemoteObject.js"></script>
</span><span class="lines">@@ -98,7 +97,6 @@
</span><span class="cx"> <script src="Models/ExecutionContext.js"></script>
</span><span class="cx"> <script src="Models/ExecutionContextList.js"></script>
</span><span class="cx"> <script src="Models/Frame.js"></script>
</span><del>- <script src="Models/GarbageCollection.js"></script>
</del><span class="cx"> <script src="Models/Geometry.js"></script>
</span><span class="cx"> <script src="Models/IndexedDatabase.js"></script>
</span><span class="cx"> <script src="Models/IndexedDatabaseObjectStore.js"></script>
</span><span class="lines">@@ -142,7 +140,6 @@
</span><span class="cx"> <script src="Controllers/DOMTreeManager.js"></script>
</span><span class="cx"> <script src="Controllers/DebuggerManager.js"></script>
</span><span class="cx"> <script src="Controllers/FrameResourceManager.js"></script>
</span><del>- <script src="Controllers/HeapManager.js"></script>
</del><span class="cx"> <script src="Controllers/IssueManager.js"></script>
</span><span class="cx"> <script src="Controllers/LogManager.js"></script>
</span><span class="cx"> <script src="Controllers/ProbeManager.js"></script>
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsScriptTimelineDataGridNodejs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineDataGridNode.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineDataGridNode.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineDataGridNode.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -77,15 +77,8 @@
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span><del>- return {
- eventType: this._record.eventType,
- startTime,
- selfTime: duration,
- totalTime: duration,
- averageTime: duration,
- callCount: NaN,
- location: callFrameOrSourceCodeLocation
- };
</del><ins>+ return {eventType: this._record.eventType, startTime, selfTime: duration, totalTime: duration,
+ averageTime: duration, callCount: 1, location: callFrameOrSourceCodeLocation};
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> updateRangeTimes(startTime, endTime)
</span><span class="lines">@@ -131,9 +124,6 @@
</span><span class="cx"> case "totalTime":
</span><span class="cx"> case "averageTime":
</span><span class="cx"> return isNaN(value) ? emptyValuePlaceholderString : Number.secondsToString(value, true);
</span><del>-
- case "callCount":
- return isNaN(value) ? emptyValuePlaceholderString : value;
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> return super.createCellContent(columnIdentifier, cell);
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsScriptTimelineOverviewGraphjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineOverviewGraph.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineOverviewGraph.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineOverviewGraph.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -73,11 +73,7 @@
</span><span class="cx"> ++recordBarIndex;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- // Create bars for non-GC records and GC records.
- let [gcRecords, nonGCRecords] = this._scriptTimeline.records.partition((x) => x.isGarbageCollection());
- let boundCreateBar = createBar.bind(this);
- WebInspector.TimelineRecordBar.createCombinedBars(nonGCRecords, secondsPerPixel, this, boundCreateBar);
- WebInspector.TimelineRecordBar.createCombinedBars(gcRecords, secondsPerPixel, this, boundCreateBar);
</del><ins>+ WebInspector.TimelineRecordBar.createCombinedBars(this._scriptTimeline.records, secondsPerPixel, this, createBar.bind(this));
</ins><span class="cx">
</span><span class="cx"> // Remove the remaining unused TimelineRecordBars.
</span><span class="cx"> for (; recordBarIndex < this._timelineRecordBars.length; ++recordBarIndex) {
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsScriptTimelineViewjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineView.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineView.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ScriptTimelineView.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -224,10 +224,6 @@
</span><span class="cx"> if (!this._pendingRecords.length)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- var zeroTime = this.zeroTime;
- var startTime = this.startTime;
- var endTime = this.endTime;
-
</del><span class="cx"> for (var scriptTimelineRecord of this._pendingRecords) {
</span><span class="cx"> var rootNodes = [];
</span><span class="cx"> if (scriptTimelineRecord.profile) {
</span><span class="lines">@@ -235,11 +231,15 @@
</span><span class="cx"> rootNodes = scriptTimelineRecord.profile.topDownRootNodes;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- var treeElement = new WebInspector.TimelineRecordTreeElement(scriptTimelineRecord, WebInspector.SourceCodeLocation.NameStyle.Short, true);
</del><ins>+ var zeroTime = this.zeroTime;
+ var treeElement = new WebInspector.TimelineRecordTreeElement(scriptTimelineRecord, WebInspector.SourceCodeLocation.NameStyle.Short, rootNodes.length);
</ins><span class="cx"> var dataGridNode = new WebInspector.ScriptTimelineDataGridNode(scriptTimelineRecord, zeroTime);
</span><span class="cx">
</span><span class="cx"> this._dataGrid.addRowInSortOrder(treeElement, dataGridNode);
</span><span class="cx">
</span><ins>+ var startTime = this.startTime;
+ var endTime = this.endTime;
+
</ins><span class="cx"> for (var profileNode of rootNodes) {
</span><span class="cx"> var profileNodeTreeElement = new WebInspector.ProfileNodeTreeElement(profileNode, this);
</span><span class="cx"> var profileNodeDataGridNode = new WebInspector.ProfileNodeDataGridNode(profileNode, zeroTime, startTime, endTime);
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsSourceCodeTimelineTreeElementjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SourceCodeTimelineTreeElement.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/SourceCodeTimelineTreeElement.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SourceCodeTimelineTreeElement.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -25,13 +25,13 @@
</span><span class="cx">
</span><span class="cx"> WebInspector.SourceCodeTimelineTreeElement = class SourceCodeTimelineTreeElement extends WebInspector.TimelineRecordTreeElement
</span><span class="cx"> {
</span><del>- constructor(sourceCodeTimeline, subtitleNameStyle, includeDetailsInformationInMainTitle)
</del><ins>+ constructor(sourceCodeTimeline, subtitleNameStyle, includeTimerIdentifierInMainTitle)
</ins><span class="cx"> {
</span><span class="cx"> console.assert(sourceCodeTimeline);
</span><span class="cx">
</span><span class="cx"> subtitleNameStyle = subtitleNameStyle || WebInspector.SourceCodeLocation.NameStyle.None;
</span><span class="cx">
</span><del>- super(sourceCodeTimeline.records[0], subtitleNameStyle, includeDetailsInformationInMainTitle, sourceCodeTimeline.sourceCodeLocation, sourceCodeTimeline);
</del><ins>+ super(sourceCodeTimeline.records[0], subtitleNameStyle, includeTimerIdentifierInMainTitle, sourceCodeTimeline.sourceCodeLocation, sourceCodeTimeline);
</ins><span class="cx">
</span><span class="cx"> this._sourceCodeTimeline = sourceCodeTimeline;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsTimelineIconscss"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineIcons.css (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineIcons.css 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineIcons.css 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -103,10 +103,6 @@
</span><span class="cx"> content: url(../Images/TimelineRecordConsoleProfile.svg);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-.garbage-collection-profile-record .icon {
- content: url(../Images/TimelineRecordGarbageCollection.svg);
-}
-
</del><span class="cx"> .timer-record .icon {
</span><span class="cx"> content: url(../Images/TimelineRecordTimer.svg);
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsTimelineRecordBarcss"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordBar.css (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordBar.css 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordBar.css 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -98,8 +98,3 @@
</span><span class="cx"> background-color: hsl(269, 65%, 74%);
</span><span class="cx"> border-color: hsl(273, 33%, 58%);
</span><span class="cx"> }
</span><del>-
-.timeline-record-bar.timeline-record-type-script.script-timeline-record-garbage-collected > .segment {
- background-color: hsl(23, 69%, 73%);
- border-color: hsl(11, 54%, 62%);
-}
</del></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsTimelineRecordBarjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordBar.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordBar.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordBar.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -134,23 +134,21 @@
</span><span class="cx"> var startTimeProperty = usesActiveStartTime ? "activeStartTime" : "startTime";
</span><span class="cx">
</span><span class="cx"> for (var record of visibleRecords) {
</span><del>- var startTime = record[startTimeProperty];
-
</del><span class="cx"> // Check if the previous record is far enough away to create the active bar. We also create it now if the current record has no active state time.
</span><del>- if (!isNaN(activeStartTime) && (activeStartTime + Math.max(activeEndTime - activeStartTime, minimumDuration) + minimumMargin <= startTime
- || (isNaN(startTime) && !isNaN(activeEndTime)))) {
</del><ins>+ if (!isNaN(activeStartTime) && (activeStartTime + Math.max(activeEndTime - activeStartTime, minimumDuration) + minimumMargin <= record[startTimeProperty]
+ || (isNaN(record[startTimeProperty]) && !isNaN(activeEndTime)))) {
</ins><span class="cx"> createBarCallback(activeRecords, WebInspector.TimelineRecordBar.RenderMode.ActiveOnly);
</span><span class="cx"> activeRecords = [];
</span><span class="cx"> activeStartTime = NaN;
</span><span class="cx"> activeEndTime = NaN;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- if (isNaN(startTime))
</del><ins>+ if (isNaN(record[startTimeProperty]))
</ins><span class="cx"> continue;
</span><span class="cx">
</span><span class="cx"> // If this is a new bar, peg the start time.
</span><span class="cx"> if (isNaN(activeStartTime))
</span><del>- activeStartTime = startTime;
</del><ins>+ activeStartTime = record[startTimeProperty];
</ins><span class="cx">
</span><span class="cx"> // Update the end time to be the maximum we encounter. activeEndTime might be NaN, so "|| 0" to prevent Math.max from returning NaN.
</span><span class="cx"> if (!isNaN(record.endTime))
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsTimelineRecordTreeElementjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordTreeElement.js (190526 => 190527)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordTreeElement.js 2015-10-02 22:15:22 UTC (rev 190526)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordTreeElement.js 2015-10-02 22:24:07 UTC (rev 190527)
</span><span class="lines">@@ -25,7 +25,7 @@
</span><span class="cx">
</span><span class="cx"> WebInspector.TimelineRecordTreeElement = class TimelineRecordTreeElement extends WebInspector.GeneralTreeElement
</span><span class="cx"> {
</span><del>- constructor(timelineRecord, subtitleNameStyle, includeDetailsInformationInMainTitle, sourceCodeLocation, representedObject)
</del><ins>+ constructor(timelineRecord, subtitleNameStyle, includeTimerIdentifierInMainTitle, sourceCodeLocation, representedObject)
</ins><span class="cx"> {
</span><span class="cx"> console.assert(timelineRecord);
</span><span class="cx">
</span><span class="lines">@@ -72,7 +72,7 @@
</span><span class="cx"> break;
</span><span class="cx">
</span><span class="cx"> case WebInspector.TimelineRecord.Type.Script:
</span><del>- title = WebInspector.ScriptTimelineRecord.EventType.displayName(timelineRecord.eventType, timelineRecord.details, includeDetailsInformationInMainTitle);
</del><ins>+ title = WebInspector.ScriptTimelineRecord.EventType.displayName(timelineRecord.eventType, timelineRecord.details, includeTimerIdentifierInMainTitle);
</ins><span class="cx">
</span><span class="cx"> switch (timelineRecord.eventType) {
</span><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.ScriptEvaluated:
</span><span class="lines">@@ -87,9 +87,6 @@
</span><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.ConsoleProfileRecorded:
</span><span class="cx"> iconStyleClass = WebInspector.TimelineRecordTreeElement.ConsoleProfileIconStyleClass;
</span><span class="cx"> break;
</span><del>- case WebInspector.ScriptTimelineRecord.EventType.GarbageCollected:
- iconStyleClass = WebInspector.TimelineRecordTreeElement.GarbageCollectionIconStyleClass;
- break;
</del><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.TimerFired:
</span><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.TimerInstalled:
</span><span class="cx"> case WebInspector.ScriptTimelineRecord.EventType.TimerRemoved:
</span><span class="lines">@@ -171,4 +168,3 @@
</span><span class="cx"> WebInspector.TimelineRecordTreeElement.AnimationRecordIconStyleClass = "animation-record";
</span><span class="cx"> WebInspector.TimelineRecordTreeElement.ProbeRecordIconStyleClass = "probe-record";
</span><span class="cx"> WebInspector.TimelineRecordTreeElement.ConsoleProfileIconStyleClass = "console-profile-record";
</span><del>-WebInspector.TimelineRecordTreeElement.GarbageCollectionIconStyleClass = "garbage-collection-profile-record";
</del></span></pre>
</div>
</div>
</body>
</html>