Bug 1525830 - DiskIOMarkerPayload::StreamPayload should output `mOperation` for the "operation" - r=gregtatum
authorGerald Squelart <gsquelart@mozilla.com>
Thu, 07 Feb 2019 13:42:23 +0000
changeset 457614 5a1d7bbc58cf3bc7daa18237ca67f02d905e5344
parent 457613 dfef705f88a488995598c7ad143af3f2aebba4f7
child 457615 95a7957012db8d8edb93f630f61a5d204cbb2d9c
push id35516
push userrmaries@mozilla.com
push dateFri, 08 Feb 2019 04:23:26 +0000
treeherdermozilla-central@d599d1a73a3a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgregtatum
bugs1525830
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1525830 - DiskIOMarkerPayload::StreamPayload should output `mOperation` for the "operation" - r=gregtatum mFilename is already present in "filename". Differential Revision: https://phabricator.services.mozilla.com/D18921
tools/profiler/core/ProfilerMarkerPayload.cpp
--- a/tools/profiler/core/ProfilerMarkerPayload.cpp
+++ b/tools/profiler/core/ProfilerMarkerPayload.cpp
@@ -65,17 +65,17 @@ void TracingMarkerPayload::StreamPayload
     aWriter.StringProperty("interval", "end");
   }
 }
 
 void DiskIOMarkerPayload::StreamPayload(SpliceableJSONWriter& aWriter,
                                     const TimeStamp& aProcessStartTime,
                                     UniqueStacks& aUniqueStacks) {
   StreamCommonProps("DiskIO", aWriter, aProcessStartTime, aUniqueStacks);
-  aWriter.StringProperty("operation", mFilename.get());
+  aWriter.StringProperty("operation", mOperation.get());
   aWriter.StringProperty("source", mSource);
   if (mFilename) {
     aWriter.StringProperty("filename", mFilename.get());
   }
 }
 
 void UserTimingMarkerPayload::StreamPayload(SpliceableJSONWriter& aWriter,
                                             const TimeStamp& aProcessStartTime,