Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2116)

Unified Diff: chrome/browser/fragmentation_checker_win.h

Issue 8085026: Add a metric to Chrome to measure fragmentation of chrome.dll at startup.BUG=98033TEST=None (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/fragmentation_checker_win.h
===================================================================
--- chrome/browser/fragmentation_checker_win.h (revision 0)
+++ chrome/browser/fragmentation_checker_win.h (revision 0)
@@ -0,0 +1,26 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_FRAGMENTATION_CHECKER_WIN_H_
+#define CHROME_BROWSER_FRAGMENTATION_CHECKER_WIN_H_
+#pragma once
+
+#include "base/file_path.h"
+
+namespace fragmentation_checker {
+
+// Returns the count of the number of extents for the file at |file_path|. On
grt (UTC plus 2) 2011/10/01 02:49:46 Is "the count of the number of" different than "th
robertshield 2011/10/01 05:00:41 Yes: http://www.youtube.com/watch?v=kQC82okzTXI
+// success, returns true and the count is placed in |file_extents_count|.
+bool CountFileExtents(const FilePath& file_path, uint32* file_extents_count);
grt (UTC plus 2) 2011/10/01 02:49:46 Style guide says not to use unsigned types just be
robertshield 2011/10/01 05:00:41 I chose it here because the ioctl returns a dword,
+
+// Records fragmentation metrics for the current module. This records the number
+// of fragments the current module is stored in as well as the total on-disk
grt (UTC plus 2) 2011/10/01 02:49:46 remove " as well as the total on-disk..."
robertshield 2011/10/01 05:00:41 Done.
+// span of those fragments.
+// This will be used to determine whether pursuing more aggressive
+// manual defragmentation is worth the effort.
+void RecordFragmentationMetricForCurrentModule();
+
+} // namespace fragmentation_checker
+
+#endif // CHROME_BROWSER_FRAGMENTATION_CHECKER_WIN_H_
Property changes on: chrome\browser\fragmentation_checker_win.h
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698