Move the content-dependent RecreateLayer logic from aura::Window to RWHVA

ui/views/widget/desktop_aura/desktop_native_widget_aura.cc

Index: ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
diff --git a/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc b/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
index a2ab44c6204fd9b6833ab71a0be7ff4014cd56b1..812c068857e8fceb08e14722c415398054136680 100644
--- a/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
+++ b/ui/views/widget/desktop_aura/desktop_native_widget_aura.cc
@@ -779,12 +779,8 @@ void DesktopNativeWidgetAura::GetHitTestMask(gfx::Path* mask) const {
   native_widget_delegate_->GetHitTestMask(mask);
 }
 
-scoped_refptr<ui::Texture> DesktopNativeWidgetAura::CopyTexture() {
-  // The layer we create doesn't have an external texture, so this should never

[danakj, 2013/09/11 17:55:05]

You kept this comment elsewhere, do yuo want to keep it here? Should we have it
in every empty function? Should we move this comment to the declaration of the
pure virtual method to say "do nothing if there's no external content"?

[piman, 2013/09/12 01:02:33]

Added larger comments to WindowDelegate::DidRecreateLayer (the interface), and
removed all redundant comments in implementation.

-  // get invoked.
-  NOTREACHED();
-  return scoped_refptr<ui::Texture>();
-}
+void DesktopNativeWidgetAura::DidRecreateLayer(ui::Layer* old_layer,
+                                               ui::Layer* new_layer) {}
 
 ////////////////////////////////////////////////////////////////////////////////
 // DesktopNativeWidgetAura, ui::EventHandler implementation: