| 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 936bd47671c11db707fa32423bf9f0001c33f537..7519413f547b45ea7c424650ce7a604149ad26e1 100644
|
| --- a/cc/trees/layer_tree_host_impl.cc
|
| +++ b/cc/trees/layer_tree_host_impl.cc
|
| @@ -171,6 +171,7 @@ LayerTreeHostImpl::LayerTreeHostImpl(
|
| RenderingStatsInstrumentation* rendering_stats_instrumentation)
|
| : client_(client),
|
| proxy_(proxy),
|
| + output_surface_lost_(true),
|
| input_handler_client_(NULL),
|
| did_lock_scrolling_layer_(false),
|
| should_bubble_scrolls_(false),
|
| @@ -253,7 +254,11 @@ void LayerTreeHostImpl::CommitComplete() {
|
| pending_tree_->set_needs_update_draw_properties();
|
| pending_tree_->UpdateDrawProperties();
|
| // Start working on newly created tiles immediately if needed.
|
| - ManageTiles();
|
| + if (!manage_tiles_needed_)
|
| + client_->NotifyReadyToActivate();
|
| + else
|
| + ManageTiles();
|
| +
|
| } else {
|
| active_tree_->set_needs_update_draw_properties();
|
| }
|
| @@ -1084,10 +1089,7 @@ void LayerTreeHostImpl::DidInitializeVisibleTile() {
|
| }
|
|
|
| void LayerTreeHostImpl::NotifyReadyToActivate() {
|
| - if (pending_tree_) {
|
| - need_to_update_visible_tiles_before_draw_ = true;
|
| - ActivatePendingTree();
|
| - }
|
| + client_->NotifyReadyToActivate();
|
| }
|
|
|
| bool LayerTreeHostImpl::ShouldClearRootRenderPass() const {
|
| @@ -1338,7 +1340,7 @@ const RendererCapabilities& LayerTreeHostImpl::GetRendererCapabilities() const {
|
| }
|
|
|
| bool LayerTreeHostImpl::SwapBuffers(const LayerTreeHostImpl::FrameData& frame) {
|
| - if (frame.has_no_damage)
|
| + if (frame.has_no_damage || output_surface_lost_)
|
| return false;
|
| renderer_->SwapBuffers();
|
| active_tree_->ClearLatencyInfo();
|
| @@ -1381,6 +1383,7 @@ void LayerTreeHostImpl::DidLoseOutputSurface() {
|
| // TODO(jamesr): The renderer_ check is needed to make some of the
|
| // LayerTreeHostContextTest tests pass, but shouldn't be necessary (or
|
| // important) in production. We should adjust the test to not need this.
|
| + output_surface_lost_ = true;
|
| if (renderer_)
|
| client_->DidLoseOutputSurfaceOnImplThread();
|
| }
|
| @@ -1475,6 +1478,8 @@ void LayerTreeHostImpl::ActivatePendingTree() {
|
| CHECK(pending_tree_);
|
| TRACE_EVENT_ASYNC_END0("cc", "PendingTree", pending_tree_.get());
|
|
|
| + need_to_update_visible_tiles_before_draw_ = true;
|
| +
|
| active_tree_->SetRootLayerScrollOffsetDelegate(NULL);
|
| active_tree_->PushPersistedState(pending_tree_.get());
|
| if (pending_tree_->needs_full_tree_sync()) {
|
| @@ -1503,6 +1508,10 @@ void LayerTreeHostImpl::ActivatePendingTree() {
|
| root_layer_scroll_offset_delegate_);
|
| active_tree_->DidBecomeActive();
|
|
|
| + client_->DidActivatePendingTree();
|
| + if (!tree_activation_callback_.is_null())
|
| + tree_activation_callback_.Run();
|
| +
|
| // Reduce wasted memory now that unlinked resources are guaranteed not
|
| // to be used.
|
| client_->ReduceWastedContentsTextureMemoryOnImplThread();
|
| @@ -1519,10 +1528,6 @@ void LayerTreeHostImpl::ActivatePendingTree() {
|
| stats.main_stats.paint_time + stats.main_stats.record_time +
|
| stats.impl_stats.rasterize_time_for_now_bins_on_pending_tree);
|
| }
|
| -
|
| - client_->DidActivatePendingTree();
|
| - if (!tree_activation_callback_.is_null())
|
| - tree_activation_callback_.Run();
|
| }
|
|
|
| void LayerTreeHostImpl::SetVisible(bool visible) {
|
| @@ -1630,6 +1635,7 @@ bool LayerTreeHostImpl::InitializeRenderer(
|
| tile_manager_.reset();
|
| resource_provider_.reset();
|
| output_surface_.reset();
|
| + output_surface_lost_ = true;
|
|
|
| if (!output_surface->BindToClient(this))
|
| return false;
|
| @@ -1675,6 +1681,7 @@ bool LayerTreeHostImpl::InitializeRenderer(
|
|
|
| resource_provider_ = resource_provider.Pass();
|
| output_surface_ = output_surface.Pass();
|
| + output_surface_lost_ = false;
|
|
|
| client_->OnCanDrawStateChanged(CanDraw());
|
|
|
| @@ -2245,6 +2252,7 @@ void LayerTreeHostImpl::AnimatePageScale(base::TimeTicks time) {
|
| active_tree_->SetPageScaleDelta(
|
| page_scale_animation_->PageScaleFactorAtTime(monotonic_time) /
|
| active_tree_->page_scale_factor());
|
| +
|
| gfx::Vector2dF next_scroll =
|
| page_scale_animation_->ScrollOffsetAtTime(monotonic_time);
|
|
|
|
|