Index: cc/trees/layer_tree_host_impl.cc |
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc |
index 77c484fd1a4a6bef187c2bec963533ec8a750c4f..d2041ac2735cdc175f5f39950a4ea8202e391120 100644 |
--- a/cc/trees/layer_tree_host_impl.cc |
+++ b/cc/trees/layer_tree_host_impl.cc |
@@ -152,8 +152,14 @@ size_t GetMaxTransferBufferUsageBytes( |
unsigned GetMapImageTextureTarget( |
const ContextProvider::Capabilities& context_capabilities) { |
+// TODO(reveman): This should be a setting passed to the compositor instead |
+// of hard-coded here. The target that need to be used depends on our choice |
+// of GpuMemoryBuffer type. Note: SURFACE_TEXTURE needs EXTERNAL_OES, |
+// IO_SURFACE needs RECTANGLE_ARB. crbug.com/431059 |
+#if defined(OS_ANDROID) |
if (context_capabilities.gpu.egl_image_external) |
return GL_TEXTURE_EXTERNAL_OES; |
+#endif |
if (context_capabilities.gpu.texture_rectangle) |
return GL_TEXTURE_RECTANGLE_ARB; |
@@ -1874,9 +1880,11 @@ void LayerTreeHostImpl::ActivateSyncTree() { |
if (debug_state_.continuous_painting) { |
const RenderingStats& stats = |
rendering_stats_instrumentation_->GetRenderingStats(); |
- paint_time_counter_->SavePaintTime(stats.main_stats.paint_time + |
- stats.main_stats.record_time + |
- stats.impl_stats.rasterize_time); |
+ // TODO(hendrikw): This requires a different metric when we commit directly |
+ // to the active tree. See crbug.com/429311. |
+ paint_time_counter_->SavePaintTime( |
+ stats.impl_stats.commit_to_activate_duration.GetLastTimeDelta() + |
+ stats.impl_stats.draw_duration.GetLastTimeDelta()); |
} |
if (time_source_client_adapter_ && time_source_client_adapter_->Active()) |