Title: [141884] trunk/Source/WebCore
Revision
141884
Author
[email protected]
Date
2013-02-05 04:34:36 -0800 (Tue, 05 Feb 2013)

Log Message

Web Inspector: [Canvas] do not store a dropped trace log in backend
https://bugs.webkit.org/show_bug.cgi?id=108600

Reviewed by Pavel Feldman.

Clear memory immediately on dropping current trace log instead of waiting for the next capture command.

* inspector/InjectedScriptCanvasModuleSource.js:
(.):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (141883 => 141884)


--- trunk/Source/WebCore/ChangeLog	2013-02-05 12:04:39 UTC (rev 141883)
+++ trunk/Source/WebCore/ChangeLog	2013-02-05 12:34:36 UTC (rev 141884)
@@ -1,5 +1,17 @@
 2013-02-05  Andrey Adaikin  <[email protected]>
 
+        Web Inspector: [Canvas] do not store a dropped trace log in backend
+        https://bugs.webkit.org/show_bug.cgi?id=108600
+
+        Reviewed by Pavel Feldman.
+
+        Clear memory immediately on dropping current trace log instead of waiting for the next capture command.
+
+        * inspector/InjectedScriptCanvasModuleSource.js:
+        (.):
+
+2013-02-05  Andrey Adaikin  <[email protected]>
+
         Web Inspector: [Canvas] UI is not restored properly when deleting a live recording profile
         https://bugs.webkit.org/show_bug.cgi?id=108602
 

Modified: trunk/Source/WebCore/inspector/InjectedScriptCanvasModuleSource.js (141883 => 141884)


--- trunk/Source/WebCore/inspector/InjectedScriptCanvasModuleSource.js	2013-02-05 12:04:39 UTC (rev 141883)
+++ trunk/Source/WebCore/inspector/InjectedScriptCanvasModuleSource.js	2013-02-05 12:34:36 UTC (rev 141884)
@@ -2900,6 +2900,16 @@
         this._stopCapturingOnFrameEnd = false;
     },
 
+    /**
+     * @param {!TraceLog} traceLog
+     */
+    dropTraceLog: function(traceLog)
+    {
+        this.stopCapturing(traceLog);
+        if (this._lastTraceLog === traceLog)
+            this._lastTraceLog = null;
+    },
+
     captureFrame: function()
     {
         this._lastTraceLog = new TraceLog();
@@ -3040,7 +3050,9 @@
      */
     dropTraceLog: function(id)
     {
-        this.stopCapturing(id);
+        var traceLog = this._traceLogs[id];
+        if (traceLog)
+            this._manager.dropTraceLog(traceLog);
         delete this._traceLogs[id];
         delete this._traceLogPlayers[id];
     },
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to