Bug 61478

Summary: Cleanup of commented items from https://bugs.webkit.org/show_bug.cgi?id=61222
Product: WebKit Reporter: Michael Saboff <msaboff>
Component: WebCore Misc.Assignee: Michael Saboff <msaboff>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch with requested style changes. none

Description Michael Saboff 2011-05-25 15:47:21 PDT
There were some comments made to https://bugs.webkit.org/show_bug.cgi?id=61222 that didn't make it into the landed patch.  This includes using /2 instead of >>1, adding a blank line after class include,making method names start with lower case and adding clarifying comments.

-    
-    void pruneDeadResources(); // Flush decoded and encoded data from resources not referenced by Web pages.
-    void pruneLiveResources(); // Flush decoded data from resources still referenced by Web pages.
 
+    // pruneDead*() - Flush decoded and encoded data from resources not referenced by Web pages.
+    // pruneLive*() - Flush decoded data from resources still referenced by Web pages.
+    void pruneDeadResources(); // Automatically decide how much to prune.
+    void pruneLiveResources();
+    void PruneDeadResourcesToPercentage(float prunePercentage);
+    void PruneLiveResourcesToPercentage(float prunePercentage);

Functions should not start with capital letters. 

Added: trunk/Source/WebCore/platform/mac/MemoryPressureHandlerMac.mm (0 => 87228)

--- trunk/Source/WebCore/platform/mac/MemoryPressureHandlerMac.mm	                        (rev 0)
+++ trunk/Source/WebCore/platform/mac/MemoryPressureHandlerMac.mm	2011-05-24 23:43:15 UTC (rev 87228)
...
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#import "config.h"
+#import "MemoryPressureHandler.h"
+#import <WebCore/GCController.h>
+#import <WebCore/MemoryCache.h>
+#import <WebCore/PageCache.h>
+#import <wtf/FastMalloc.h>

There should be a newline after MemoryPressureHandler.h, and this should not be using <WebCore/*> style includes for files in WebCore.
Comment 1 Michael Saboff 2011-05-25 15:48:32 PDT
Created attachment 94874 [details]
Patch with requested style changes.
Comment 2 Michael Saboff 2011-05-25 16:05:46 PDT
Committed r87330: <http://trac.webkit.org/changeset/87330>