Title: [131469] trunk/Tools
Revision
131469
Author
[email protected]
Date
2012-10-16 10:14:55 -0700 (Tue, 16 Oct 2012)

Log Message

Unreviewed, rolling out r131461.
http://trac.webkit.org/changeset/131461
https://bugs.webkit.org/show_bug.cgi?id=99474

Broke win7 bots (Requested by danakj|gardening on #webkit).

Patch by Sheriff Bot <[email protected]> on 2012-10-16

* DumpRenderTree/chromium/TestEventPrinter.cpp:
* DumpRenderTree/chromium/TestEventPrinter.h:
(TestEventPrinter):
* DumpRenderTree/chromium/TestShell.cpp:
(TestShell::dump):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (131468 => 131469)


--- trunk/Tools/ChangeLog	2012-10-16 17:08:28 UTC (rev 131468)
+++ trunk/Tools/ChangeLog	2012-10-16 17:14:55 UTC (rev 131469)
@@ -1,3 +1,17 @@
+2012-10-16  Sheriff Bot  <[email protected]>
+
+        Unreviewed, rolling out r131461.
+        http://trac.webkit.org/changeset/131461
+        https://bugs.webkit.org/show_bug.cgi?id=99474
+
+        Broke win7 bots (Requested by danakj|gardening on #webkit).
+
+        * DumpRenderTree/chromium/TestEventPrinter.cpp:
+        * DumpRenderTree/chromium/TestEventPrinter.h:
+        (TestEventPrinter):
+        * DumpRenderTree/chromium/TestShell.cpp:
+        (TestShell::dump):
+
 2012-10-16  Zoltan Horvath  <[email protected]>
 
         [chromium] Provide used JSHeap size in chromium's DRT for pageloadtest memory measurements

Modified: trunk/Tools/DumpRenderTree/chromium/TestEventPrinter.cpp (131468 => 131469)


--- trunk/Tools/DumpRenderTree/chromium/TestEventPrinter.cpp	2012-10-16 17:08:28 UTC (rev 131468)
+++ trunk/Tools/DumpRenderTree/chromium/TestEventPrinter.cpp	2012-10-16 17:14:55 UTC (rev 131469)
@@ -30,7 +30,6 @@
 
 #include "config.h"
 #include "TestEventPrinter.h"
-#include "v8.h"
 
 #include <stdio.h>
 #include <stdlib.h>
@@ -58,12 +57,6 @@
     fprintf(stdout, "FAIL: Timed out waiting for notifyDone to be called\n");
 }
 
-void TestEventPrinter::handleDumpMemoryHeader() const
-{
-    v8::HeapStatistics heapStatistics;
-    printf("DumpJSHeap: %zu\n", heapStatistics.used_heap_size());
-}
-
 void TestEventPrinter::handleTextHeader() const
 {
     printf("Content-Type: text/plain\n");

Modified: trunk/Tools/DumpRenderTree/chromium/TestEventPrinter.h (131468 => 131469)


--- trunk/Tools/DumpRenderTree/chromium/TestEventPrinter.h	2012-10-16 17:08:28 UTC (rev 131468)
+++ trunk/Tools/DumpRenderTree/chromium/TestEventPrinter.h	2012-10-16 17:14:55 UTC (rev 131469)
@@ -39,7 +39,6 @@
     ~TestEventPrinter();
     void handleTestHeader(const char* url) const;
     void handleTimedOut() const;
-    void handleDumpMemoryHeader() const;
     void handleTextHeader() const;
     void handleTextFooter() const;
     void handleAudio(const void* audioData, size_t audioSize) const;

Modified: trunk/Tools/DumpRenderTree/chromium/TestShell.cpp (131468 => 131469)


--- trunk/Tools/DumpRenderTree/chromium/TestShell.cpp	2012-10-16 17:08:28 UTC (rev 131468)
+++ trunk/Tools/DumpRenderTree/chromium/TestShell.cpp	2012-10-16 17:14:55 UTC (rev 131469)
@@ -575,7 +575,6 @@
     if (m_params.dumpTree) {
         dumpedAnything = true;
         m_printer.handleTextHeader();
-        m_printer.handleDumpMemoryHeader();
         // Text output: the test page can request different types of output
         // which we handle here.
         if (!shouldDumpAsText) {
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to