Title: [94371] trunk/Source
Revision
94371
Author
[email protected]
Date
2011-09-01 17:20:30 -0700 (Thu, 01 Sep 2011)

Log Message

Source/WebCore: Cleanup refactoring for https://bugs.webkit.org/show_bug.cgi?id=67160

Don't ifdef out the data member xslStyleSheets in MemoryCache::Statistics to cut down 
the ifdefs in getWebCoreMemoryCacheStatistics() in WebKit2/WebProcess/WebProcess.cpp.

Reviewed by Darin Adler.

No new tests required.  Just small code refactoring.

* loader/cache/MemoryCache.h:

Source/WebKit2: Cut down the ifdefs in WebKit::getWebCoreMemoryCacheStatistics() now that 
MemoryCache::Statistics always has the xslStyleSheets field.

Reviewed by Darin Adler.

* WebProcess/WebProcess.cpp:
(WebKit::getWebCoreMemoryCacheStatistics):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (94370 => 94371)


--- trunk/Source/WebCore/ChangeLog	2011-09-02 00:20:04 UTC (rev 94370)
+++ trunk/Source/WebCore/ChangeLog	2011-09-02 00:20:30 UTC (rev 94371)
@@ -1,3 +1,16 @@
+2011-09-01  Ada Chan  <[email protected]>
+
+        Cleanup refactoring for https://bugs.webkit.org/show_bug.cgi?id=67160
+
+        Don't ifdef out the data member xslStyleSheets in MemoryCache::Statistics to cut down 
+        the ifdefs in getWebCoreMemoryCacheStatistics() in WebKit2/WebProcess/WebProcess.cpp.
+
+        Reviewed by Darin Adler.
+
+        No new tests required.  Just small code refactoring.
+
+        * loader/cache/MemoryCache.h:
+
 2011-09-01  Julien Chaffraix  <[email protected]>
 
         REGRESSION (r84327-r84329): CSS stylesheets fail to load on www.flagstar.com login page

Modified: trunk/Source/WebCore/loader/cache/MemoryCache.h (94370 => 94371)


--- trunk/Source/WebCore/loader/cache/MemoryCache.h	2011-09-02 00:20:04 UTC (rev 94370)
+++ trunk/Source/WebCore/loader/cache/MemoryCache.h	2011-09-02 00:20:30 UTC (rev 94371)
@@ -99,9 +99,7 @@
         TypeStatistic images;
         TypeStatistic cssStyleSheets;
         TypeStatistic scripts;
-#if ENABLE(XSLT)
         TypeStatistic xslStyleSheets;
-#endif
         TypeStatistic fonts;
     };
     

Modified: trunk/Source/WebKit2/ChangeLog (94370 => 94371)


--- trunk/Source/WebKit2/ChangeLog	2011-09-02 00:20:04 UTC (rev 94370)
+++ trunk/Source/WebKit2/ChangeLog	2011-09-02 00:20:30 UTC (rev 94371)
@@ -1,3 +1,13 @@
+2011-09-01  Ada Chan  <[email protected]>
+
+        Cut down the ifdefs in WebKit::getWebCoreMemoryCacheStatistics() now that 
+        MemoryCache::Statistics always has the xslStyleSheets field.
+
+        Reviewed by Darin Adler.
+
+        * WebProcess/WebProcess.cpp:
+        (WebKit::getWebCoreMemoryCacheStatistics):
+
 2011-09-01  Mark Hahnenberg  <[email protected]>
 
         Unzip initialization lists and constructors in JSCell hierarchy (4/7)

Modified: trunk/Source/WebKit2/WebProcess/WebProcess.cpp (94370 => 94371)


--- trunk/Source/WebKit2/WebProcess/WebProcess.cpp	2011-09-02 00:20:04 UTC (rev 94370)
+++ trunk/Source/WebKit2/WebProcess/WebProcess.cpp	2011-09-02 00:20:30 UTC (rev 94371)
@@ -845,66 +845,42 @@
     HashMap<String, uint64_t> counts;
     counts.set(imagesString, memoryCacheStatistics.images.count);
     counts.set(cssString, memoryCacheStatistics.cssStyleSheets.count);
-#if ENABLE(XSLT)
     counts.set(xslString, memoryCacheStatistics.xslStyleSheets.count);
-#else
-    counts.set(xslString, 0);
-#endif
     counts.set(_javascript_String, memoryCacheStatistics.scripts.count);
     result.append(counts);
     
     HashMap<String, uint64_t> sizes;
     sizes.set(imagesString, memoryCacheStatistics.images.size);
     sizes.set(cssString, memoryCacheStatistics.cssStyleSheets.size);
-#if ENABLE(XSLT)
     sizes.set(xslString, memoryCacheStatistics.xslStyleSheets.size);
-#else
-    sizes.set(xslString, 0);
-#endif
     sizes.set(_javascript_String, memoryCacheStatistics.scripts.size);
     result.append(sizes);
     
     HashMap<String, uint64_t> liveSizes;
     liveSizes.set(imagesString, memoryCacheStatistics.images.liveSize);
     liveSizes.set(cssString, memoryCacheStatistics.cssStyleSheets.liveSize);
-#if ENABLE(XSLT)
     liveSizes.set(xslString, memoryCacheStatistics.xslStyleSheets.liveSize);
-#else
-    liveSizes.set(xslString, 0);
-#endif
     liveSizes.set(_javascript_String, memoryCacheStatistics.scripts.liveSize);
     result.append(liveSizes);
     
     HashMap<String, uint64_t> decodedSizes;
     decodedSizes.set(imagesString, memoryCacheStatistics.images.decodedSize);
     decodedSizes.set(cssString, memoryCacheStatistics.cssStyleSheets.decodedSize);
-#if ENABLE(XSLT)
     decodedSizes.set(xslString, memoryCacheStatistics.xslStyleSheets.decodedSize);
-#else
-    decodedSizes.set(xslString, 0);
-#endif
     decodedSizes.set(_javascript_String, memoryCacheStatistics.scripts.decodedSize);
     result.append(decodedSizes);
     
     HashMap<String, uint64_t> purgeableSizes;
     purgeableSizes.set(imagesString, memoryCacheStatistics.images.purgeableSize);
     purgeableSizes.set(cssString, memoryCacheStatistics.cssStyleSheets.purgeableSize);
-#if ENABLE(XSLT)
     purgeableSizes.set(xslString, memoryCacheStatistics.xslStyleSheets.purgeableSize);
-#else
-    purgeableSizes.set(xslString, 0);
-#endif
     purgeableSizes.set(_javascript_String, memoryCacheStatistics.scripts.purgeableSize);
     result.append(purgeableSizes);
     
     HashMap<String, uint64_t> purgedSizes;
     purgedSizes.set(imagesString, memoryCacheStatistics.images.purgedSize);
     purgedSizes.set(cssString, memoryCacheStatistics.cssStyleSheets.purgedSize);
-#if ENABLE(XSLT)
     purgedSizes.set(xslString, memoryCacheStatistics.xslStyleSheets.purgedSize);
-#else
-    purgedSizes.set(xslString, 0);
-#endif
     purgedSizes.set(_javascript_String, memoryCacheStatistics.scripts.purgedSize);
     result.append(purgedSizes);
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to