Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-30 16:22:00 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1349607 - Remove mVsyncTimestamp from the VsyncMarkerPayload r=mstange

Browse Source 
Removing this field since we already have start/end time for each marker payloads.
Also we are assigning the same timestamp to them. We don't need to unnecessarily
duplicate the information here.

Differential Revision: https://phabricator.services.mozilla.com/D8551

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Nazım Can Altınova 2018-11-01 13:56:18 +00:00
parent c51f546bd5
commit b307cbb99b
2 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tools/profiler/core/ProfilerMarkerPayload.cpp
View File

@ -135,8 +135,6 @@ VsyncMarkerPayload::StreamPayload(SpliceableJSONWriter& aWriter,
UniqueStacks& aUniqueStacks) UniqueStacks& aUniqueStacks)
{ {
StreamType("VsyncTimestamp", aWriter); StreamType("VsyncTimestamp", aWriter);
aWriter.DoubleProperty("vsync",
(mVsyncTimestamp - aProcessStartTime).ToMilliseconds());
} }
static const char *GetNetworkState(NetworkLoadType aType) static const char *GetNetworkState(NetworkLoadType aType)

4
tools/profiler/public/ProfilerMarkerPayload.h
View File

@ -200,13 +200,9 @@ class VsyncMarkerPayload : public ProfilerMarkerPayload
public: public:
explicit VsyncMarkerPayload(mozilla::TimeStamp aVsyncTimestamp) explicit VsyncMarkerPayload(mozilla::TimeStamp aVsyncTimestamp)
: ProfilerMarkerPayload(aVsyncTimestamp, aVsyncTimestamp) : ProfilerMarkerPayload(aVsyncTimestamp, aVsyncTimestamp)
, mVsyncTimestamp(aVsyncTimestamp)
{} {}
DECL_STREAM_PAYLOAD DECL_STREAM_PAYLOAD
private:
mozilla::TimeStamp mVsyncTimestamp;
}; };
class NetworkMarkerPayload : public ProfilerMarkerPayload class NetworkMarkerPayload : public ProfilerMarkerPayload