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

Side by Side Diff: android_webview/browser/child_frame.h

Issue 2347563003: Added FrameFuture class (Closed)
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 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 ANDROID_WEBVIEW_BROWSER_CHILD_FRAME_H_ 5 #ifndef ANDROID_WEBVIEW_BROWSER_CHILD_FRAME_H_
6 #define ANDROID_WEBVIEW_BROWSER_CHILD_FRAME_H_ 6 #define ANDROID_WEBVIEW_BROWSER_CHILD_FRAME_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "android_webview/browser/compositor_id.h" 10 #include "android_webview/browser/compositor_id.h"
11 #include "base/macros.h" 11 #include "base/macros.h"
12 #include "base/memory/ref_counted.h"
13 #include "content/public/browser/android/synchronous_compositor.h"
12 #include "ui/gfx/geometry/rect.h" 14 #include "ui/gfx/geometry/rect.h"
13 #include "ui/gfx/geometry/size.h" 15 #include "ui/gfx/geometry/size.h"
14 #include "ui/gfx/transform.h" 16 #include "ui/gfx/transform.h"
15 17
16 namespace cc { 18 namespace cc {
boliu 2016/09/16 16:26:12 you can remove this forward declaration
ojars 2016/09/19 22:02:37 Done.
17 class CompositorFrame; 19 class CompositorFrame;
18 } 20 }
19 21
20 namespace android_webview { 22 namespace android_webview {
21 23
22 class ChildFrame { 24 class ChildFrame {
23 public: 25 public:
24 ChildFrame(uint32_t output_surface_id, 26 // ChildFrame(uint32_t output_surface_id,
boliu 2016/09/16 16:26:12 commented out code
ojars 2016/09/19 22:02:37 Done.
25 std::unique_ptr<cc::CompositorFrame> frame, 27 // std::unique_ptr<cc::CompositorFrame> frame,
26 const CompositorID& compositor_id, 28 ChildFrame(
27 bool viewport_rect_for_tile_priority_empty, 29 scoped_refptr<content::SynchronousCompositor::FrameFuture> frame_future,
boliu 2016/09/16 16:26:12 const &
ojars 2016/09/19 22:02:37 Done.
28 const gfx::Transform& transform_for_tile_priority, 30 const CompositorID& compositor_id,
29 bool offscreen_pre_raster, 31 bool viewport_rect_for_tile_priority_empty,
30 bool is_layer); 32 const gfx::Transform& transform_for_tile_priority,
33 bool offscreen_pre_raster,
34 bool is_layer);
31 ~ChildFrame(); 35 ~ChildFrame();
32 36
33 const uint32_t output_surface_id; 37 // const uint32_t output_surface_id;
boliu 2016/09/16 16:26:12 ditto
ojars 2016/09/19 22:02:37 Done.
34 std::unique_ptr<cc::CompositorFrame> frame; 38 // std::unique_ptr<cc::CompositorFrame> frame;
39 scoped_refptr<content::SynchronousCompositor::FrameFuture> frame_future;
35 // The id of the compositor this |frame| comes from. 40 // The id of the compositor this |frame| comes from.
36 const CompositorID compositor_id; 41 const CompositorID compositor_id;
37 const bool viewport_rect_for_tile_priority_empty; 42 const bool viewport_rect_for_tile_priority_empty;
38 const gfx::Transform transform_for_tile_priority; 43 const gfx::Transform transform_for_tile_priority;
39 const bool offscreen_pre_raster; 44 const bool offscreen_pre_raster;
40 const bool is_layer; 45 const bool is_layer;
41 46
42 private: 47 private:
43 DISALLOW_COPY_AND_ASSIGN(ChildFrame); 48 DISALLOW_COPY_AND_ASSIGN(ChildFrame);
44 }; 49 };
45 50
46 } // namespace webview 51 } // namespace webview
47 52
48 #endif // ANDROID_WEBVIEW_BROWSER_CHILD_FRAME_H_ 53 #endif // ANDROID_WEBVIEW_BROWSER_CHILD_FRAME_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698