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

Unified Diff: skia/ext/SkTraceMemoryDump_chrome.h

Issue 1318823012: SkTraceMemoryDump_Chrome: fix -Wreorder warning (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: skia/ext/SkTraceMemoryDump_chrome.h
diff --git a/skia/ext/SkTraceMemoryDump_chrome.h b/skia/ext/SkTraceMemoryDump_chrome.h
index dcdfc8f97dae6058b9b409119dfb5e03ae4f74b8..749d99d54677aa6ddbd40dddf1b82e26eaeee53d 100644
--- a/skia/ext/SkTraceMemoryDump_chrome.h
+++ b/skia/ext/SkTraceMemoryDump_chrome.h
@@ -24,7 +24,7 @@ class SkTraceMemoryDump_Chrome : public SkTraceMemoryDump {
// This should never outlive the OnMemoryDump call since the
// ProcessMemoryDump is valid only in that timeframe. Optional
// |dump_name_prefix| argument specifies the prefix appended to the dump
- // name skia provides. By defualt it is taken as empty string.
+ // name skia provides. By default it is taken as empty string.
SkTraceMemoryDump_Chrome(
base::trace_event::ProcessMemoryDump* process_memory_dump);
@@ -52,10 +52,10 @@ class SkTraceMemoryDump_Chrome : public SkTraceMemoryDump {
}
private:
- base::trace_event::ProcessMemoryDump* process_memory_dump_;
-
std::string dump_name_prefix_;
+ base::trace_event::ProcessMemoryDump* process_memory_dump_;
+
// Helper to create allocator dumps.
base::trace_event::MemoryAllocatorDump* GetOrCreateAllocatorDump(
const char* dumpName);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698