Chrome Version: refs/heads/master@{#530170}
OS: Linux

What steps will reproduce the problem?
(1) Open Chrome and select menu > cast
(2) Wait for the cast dialog to stabilize
(3) Click the arrow next to "cast to" to toggle between this and the "select source" view.
(4) Note that on each click, the RenderWidget sends a single ViewHostMsg_ResizeOrRepaint_ACK
(5) Note that for each ViewHostMsg_ResizeOrRepaint_ACK, the Browser performs two surface ID allocations and sends three ViewMsg_SetLocalSurfaceIdForAutoResize responses.

What is the expected result?
I would expect the renderer to receive one ViewMsg_SetLocalSurfaceIdForAutoResize for each ViewHostMsg_ResizeOrRepaint_ACK. I would also expect only one surface allocation.

For a bit more info, here are the stacks showing where we allocate new surfaces (note that this all stems from the same RenderWidgetHostImpl::DelayedAutoResized call triggered by the ViewHostMsg_ResizeOrRepaint_ACK:

----------- stack (A) -------------
#1 0x7ff0b81f9d43 aura::WindowPortLocal::OnDidChangeBounds()
#2 0x7ff0b824b6a2 aura::Window::OnLayerBoundsChanged()
#3 0x7ff0b816054d ui::Layer::SetBoundsFromAnimation()
#4 0x7ff0b82454e2 aura::Window::SetBounds()
#5 0x7ff0bad992de content::RenderWidgetHostViewAura::InternalSetBounds()
#6 0x7ff0bad99201 content::RenderWidgetHostViewAura::SetSize()
#7 0x7ff0bafb935e content::WebContentsViewAura::OnBoundsChanged()
#8 0x7ff0b824b70a aura::Window::OnLayerBoundsChanged()
#9 0x7ff0b816054d ui::Layer::SetBoundsFromAnimation()
#10 0x7ff0b82454e2 aura::Window::SetBounds()
#11 0x7ff0b71a8eaf views::NativeViewHostAura::ShowWidget()
#12 0x7ff0b71064d3 views::NativeViewHost::Layout()
#13 0x7ff0b716e5d7 views::View::BoundsChanged()
#14 0x7ff0b716d85e views::View::SetBoundsRect()
#15 0x7ff0b6b2c6ff views::WebView::OnBoundsChanged()
#16 0x7ff0b716e47d views::View::BoundsChanged()
#17 0x7ff0b716d85e views::View::SetBoundsRect()
#18 0x7ff0b716d6a7 views::View::SetBounds()
#19 0x7ff0b716e5d7 views::View::BoundsChanged()
#20 0x7ff0b716d85e views::View::SetBoundsRect()
#21 0x7ff0b719e23d views::NonClientView::Layout()
#22 0x7ff0b716e5d7 views::View::BoundsChanged()
#23 0x7ff0b716d85e views::View::SetBoundsRect()
#24 0x7ff0b7158756 views::FillLayout::Layout()
#25 0x7ff0b717121c views::View::Layout()
#26 0x7ff0b716e5d7 views::View::BoundsChanged()
#27 0x7ff0b716d85e views::View::SetBoundsRect()
#28 0x7ff0b716e75a views::View::SetSize()
#29 0x7ff0b718d696 views::Widget::OnNativeWidgetSizeChanged()
#30 0x7ff0b824b70a aura::Window::OnLayerBoundsChanged()
#31 0x7ff0b816054d ui::Layer::SetBoundsFromAnimation()
#32 0x7ff0b82454e2 aura::Window::SetBounds()
#33 0x7ff0b71bc874 views::DesktopScreenPositionClient::SetBounds()
#34 0x7ff0b71be48e views::NativeWidgetAura::SetBounds()
#35 0x55d9e6861d0f constrained_window::(anonymous namespace)::UpdateModalDialogPosition()
#36 0x55d9e6861a28 constrained_window::UpdateWebContentsModalDialogPosition()
#37 0x7ff0baf990a1 content::WebContentsImpl::ResizeDueToAutoResize()
#38 0x7ff0bad83d75 content::RenderWidgetHostImpl::DelayedAutoResized()

----------- stack (B) -------------
#1 0x7ff0b81fa6ea aura::WindowPortLocal::AllocateLocalSurfaceId()
#2 0x7ff0bada22a2 content::RenderWidgetHostViewAura::ResizeDueToAutoResize()
#3 0x7ff0baf990e1 content::WebContentsImpl::ResizeDueToAutoResize()
#4 0x7ff0bad83d75 content::RenderWidgetHostImpl::DelayedAutoResized()

Additionally, here are the three stacks showing where we send ViewMsg_SetLocalSurfaceIdForAutoResizes back to the renderer. These also all come from the same base RenderWidgetHostImpl::DelayedAutoResized call:

----------- stack (C) -------------
#1 0x7ff0bad8676f content::RenderWidgetHostImpl::DidAllocateLocalSurfaceIdForAutoResize()
#2 0x7ff0bad99201 content::RenderWidgetHostViewAura::SetSize()
#3 0x7ff0bad9e5e7 content::RenderWidgetHostViewAura::OnBoundsChanged()
#4 0x7ff0b824b70a aura::Window::OnLayerBoundsChanged()
#5 0x7ff0b816054d ui::Layer::SetBoundsFromAnimation()
#6 0x7ff0b82454e2 aura::Window::SetBounds()
#7 0x7ff0bad992de content::RenderWidgetHostViewAura::InternalSetBounds()
#8 0x7ff0bad99201 content::RenderWidgetHostViewAura::SetSize()
#9 0x7ff0bafb935e content::WebContentsViewAura::OnBoundsChanged()
#10 0x7ff0b824b70a aura::Window::OnLayerBoundsChanged()
#11 0x7ff0b816054d ui::Layer::SetBoundsFromAnimation()
#12 0x7ff0b82454e2 aura::Window::SetBounds()
#13 0x7ff0b71a8eaf views::NativeViewHostAura::ShowWidget()
#14 0x7ff0b71064d3 views::NativeViewHost::Layout()
#15 0x7ff0b716e5d7 views::View::BoundsChanged()
#16 0x7ff0b716d85e views::View::SetBoundsRect()
#17 0x7ff0b6b2c6ff views::WebView::OnBoundsChanged()
#18 0x7ff0b716e47d views::View::BoundsChanged()
#19 0x7ff0b716d85e views::View::SetBoundsRect()
#20 0x7ff0b716d6a7 views::View::SetBounds()
#21 0x7ff0b716e5d7 views::View::BoundsChanged()
#22 0x7ff0b716d85e views::View::SetBoundsRect()
#23 0x7ff0b719e23d views::NonClientView::Layout()
#24 0x7ff0b716e5d7 views::View::BoundsChanged()
#25 0x7ff0b716d85e views::View::SetBoundsRect()
#26 0x7ff0b7158756 views::FillLayout::Layout()
#27 0x7ff0b717121c views::View::Layout()
#28 0x7ff0b716e5d7 views::View::BoundsChanged()
#29 0x7ff0b716d85e views::View::SetBoundsRect()
#30 0x7ff0b716e75a views::View::SetSize()
#31 0x7ff0b718d696 views::Widget::OnNativeWidgetSizeChanged()
#32 0x7ff0b824b70a aura::Window::OnLayerBoundsChanged()
#33 0x7ff0b816054d ui::Layer::SetBoundsFromAnimation()
#34 0x7ff0b82454e2 aura::Window::SetBounds()
#35 0x7ff0b71bc874 views::DesktopScreenPositionClient::SetBounds()
#36 0x7ff0b71be48e views::NativeWidgetAura::SetBounds()
#37 0x55d9e6861d0f constrained_window::(anonymous namespace)::UpdateModalDialogPosition()
#38 0x55d9e6861a28 constrained_window::UpdateWebContentsModalDialogPosition()
#39 0x7ff0baf990a1 content::WebContentsImpl::ResizeDueToAutoResize()
#40 0x7ff0bad83d75 content::RenderWidgetHostImpl::DelayedAutoResized()

----------- stack (D) -------------
content::RenderWidgetHostImpl::DidAllocateLocalSurfaceIdForAutoResize()
#2 0x7ff0bad99201 content::RenderWidgetHostViewAura::SetSize()
#3 0x7ff0bafb935e content::WebContentsViewAura::OnBoundsChanged()
#4 0x7ff0b824b70a aura::Window::OnLayerBoundsChanged()
#5 0x7ff0b816054d ui::Layer::SetBoundsFromAnimation()
#6 0x7ff0b82454e2 aura::Window::SetBounds()
#7 0x7ff0b71a8eaf views::NativeViewHostAura::ShowWidget()
#8 0x7ff0b71064d3 views::NativeViewHost::Layout()
#9 0x7ff0b716e5d7 views::View::BoundsChanged()
#10 0x7ff0b716d85e views::View::SetBoundsRect()
#11 0x7ff0b6b2c6ff views::WebView::OnBoundsChanged()
#12 0x7ff0b716e47d views::View::BoundsChanged()
#13 0x7ff0b716d85e views::View::SetBoundsRect()
#14 0x7ff0b716d6a7 views::View::SetBounds()
#15 0x7ff0b716e5d7 views::View::BoundsChanged()
#16 0x7ff0b716d85e views::View::SetBoundsRect()
#17 0x7ff0b719e23d views::NonClientView::Layout()
#18 0x7ff0b716e5d7 views::View::BoundsChanged()
#19 0x7ff0b716d85e views::View::SetBoundsRect()
#20 0x7ff0b7158756 views::FillLayout::Layout()
#21 0x7ff0b717121c views::View::Layout()
#22 0x7ff0b716e5d7 views::View::BoundsChanged()
#23 0x7ff0b716d85e views::View::SetBoundsRect()
#24 0x7ff0b716e75a views::View::SetSize()
#25 0x7ff0b718d696 views::Widget::OnNativeWidgetSizeChanged()
#26 0x7ff0b824b70a aura::Window::OnLayerBoundsChanged()
#27 0x7ff0b816054d ui::Layer::SetBoundsFromAnimation()
#28 0x7ff0b82454e2 aura::Window::SetBounds()
#29 0x7ff0b71bc874 views::DesktopScreenPositionClient::SetBounds()
#30 0x7ff0b71be48e views::NativeWidgetAura::SetBounds()
#31 0x55d9e6861d0f constrained_window::(anonymous namespace)::UpdateModalDialogPosition()
#32 0x55d9e6861a28 constrained_window::UpdateWebContentsModalDialogPosition()
#33 0x7ff0baf990a1 content::WebContentsImpl::ResizeDueToAutoResize()
#34 0x7ff0bad83d75 content::RenderWidgetHostImpl::DelayedAutoResized()

----------- stack (E) -------------
#1 0x7ff0bad8676f content::RenderWidgetHostImpl::DidAllocateLocalSurfaceIdForAutoResize()
#2 0x7ff0baf990e1 content::WebContentsImpl::ResizeDueToAutoResize()
#3 0x7ff0bad83d75 content::RenderWidgetHostImpl::DelayedAutoResized()

Note that it looks like (in this situation at least) we can safely drop the alloc in stack (A) and the notifications in stack (C) and (D), as these will always be covered by the later alloc/noficiation in stacks (B) and (E).