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

Side by Side Diff: content/browser/devtools/devtools_frame_trace_recorder.h

Issue 1188773005: DevTools: fix concurrency problems in DevToolsFrameTraceRecorder (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments addressed Created 5 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2015 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_FRAME_TRACE_RECORDER_H_ 5 #ifndef CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_FRAME_TRACE_RECORDER_H_
6 #define CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_FRAME_TRACE_RECORDER_H_ 6 #define CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_FRAME_TRACE_RECORDER_H_
7 7
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/memory/weak_ptr.h" 9 #include "base/memory/weak_ptr.h"
10 10
11 namespace cc { 11 namespace cc {
12 class CompositorFrameMetadata; 12 class CompositorFrameMetadata;
13 } 13 }
14 14
15 namespace content { 15 namespace content {
16 16
17 class DevToolsFrameTraceRecorderData; 17 class DevToolsFrameTraceRecorderData;
18 class RenderFrameHostImpl; 18 class RenderFrameHostImpl;
19 19
20 class DevToolsFrameTraceRecorder { 20 class DevToolsFrameTraceRecorder {
21 public: 21 public:
22 DevToolsFrameTraceRecorder(); 22 DevToolsFrameTraceRecorder();
23 ~DevToolsFrameTraceRecorder(); 23 ~DevToolsFrameTraceRecorder();
24 24
25 void OnSwapCompositorFrame( 25 void OnSwapCompositorFrame(
26 RenderFrameHostImpl* host, 26 RenderFrameHostImpl* host,
27 const cc::CompositorFrameMetadata& frame_metadata); 27 const cc::CompositorFrameMetadata& frame_metadata);
28 28
29 private: 29 private:
30 base::WeakPtr<DevToolsFrameTraceRecorderData> last_event_data_; 30 scoped_refptr<DevToolsFrameTraceRecorderData> pending_frame_data_;
31 DISALLOW_COPY_AND_ASSIGN(DevToolsFrameTraceRecorder); 31 DISALLOW_COPY_AND_ASSIGN(DevToolsFrameTraceRecorder);
32 }; 32 };
33 33
34 } // namespace content 34 } // namespace content
35 35
36 #endif // CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_FRAME_TRACE_RECORDER_H_ 36 #endif // CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_FRAME_TRACE_RECORDER_H_
OLDNEW
« no previous file with comments | « base/trace_event/trace_event.h ('k') | content/browser/devtools/devtools_frame_trace_recorder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698