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

Unified Diff: Source/core/inspector/TimelineTraceEventProcessor.h

Issue 46663010: DevTools: Show GPU utilization bar on timeline (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Addressing comments. Created 7 years, 2 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: Source/core/inspector/TimelineTraceEventProcessor.h
diff --git a/Source/core/inspector/TimelineTraceEventProcessor.h b/Source/core/inspector/TimelineTraceEventProcessor.h
index 8b4d99d5c114c106b72337f580c04d6fdd76cd75..b2b634dc34c2e93cbffcfd101e6b6e3804eca79b 100644
--- a/Source/core/inspector/TimelineTraceEventProcessor.h
+++ b/Source/core/inspector/TimelineTraceEventProcessor.h
@@ -81,7 +81,7 @@ private:
class TimelineTraceEventProcessor : public ThreadSafeRefCounted<TimelineTraceEventProcessor> {
public:
- TimelineTraceEventProcessor(WeakPtr<InspectorTimelineAgent>, InspectorClient*);
+ TimelineTraceEventProcessor(WeakPtr<InspectorTimelineAgent>, InspectorClient*, bool collectGpuEvents);
caseq 2013/11/01 13:04:20 do we have to pass this? can we pull it from timel
alph 2013/11/01 16:40:19 Done.
~TimelineTraceEventProcessor();
void shutdown();
@@ -219,6 +219,8 @@ private:
void onLayerDeleted(const TraceEvent&);
void onDrawLazyPixelRef(const TraceEvent&);
void onLazyPixelRefDeleted(const TraceEvent&);
+ void onGpuTaskBegin(const TraceEvent&);
+ void onGpuTaskEnd(const TraceEvent&);
WeakPtr<InspectorTimelineAgent> m_timelineAgent;
TimelineTimeConverter m_timeConverter;
@@ -240,6 +242,8 @@ private:
unsigned long long m_layerId;
double m_paintSetupStart;
double m_paintSetupEnd;
+ double m_gpuTaskStart;
+ unsigned m_gpuTaskRendererPid;
struct ImageInfo {
int backendNodeId;

Powered by Google App Engine
This is Rietveld 408576698