MacViews: Get chrome/browser/ui/views/frame to compile

chrome/browser/ui/views/frame/contents_web_view.cc

Index: chrome/browser/ui/views/frame/contents_web_view.cc
diff --git a/chrome/browser/ui/views/frame/contents_web_view.cc b/chrome/browser/ui/views/frame/contents_web_view.cc
index a86e39c4ebfbfa89e97c153253b52e04e5089aa8..ac026a9e74184f2b9638e8d24d72e52d2eeead33 100644
--- a/chrome/browser/ui/views/frame/contents_web_view.cc
+++ b/chrome/browser/ui/views/frame/contents_web_view.cc
@@ -83,7 +83,10 @@ void ContentsWebView::OnLayerRecreated(ui::Layer* old_layer,
 void ContentsWebView::CloneWebContentsLayer() {
   if (!web_contents())
     return;
+#if defined(USE_AURA)
+  // TODO(andresantoso): Figure out how to port this for MacViews.
   cloned_layer_tree_ = wm::RecreateLayers(web_contents()->GetNativeView());

[tapted, 2014/11/04 02:33:43]

So, after poking some more, I think this won't be needed for MacViews. Closing
an NSWindow doesn't animate - it just disappears - so there's no "orphaned"
layer context that needs to be displayed during the close animation. I think you
can say something to that effect here, and remove the TODO

An alternative without #ifdefs might be something like `cloned_layer_tree_ =
super::holder_->native_wrapper_->RecreateLayers();`, but #ifdefs are maybe OK
here.

[Andre, 2014/11/05 02:04:23]

Done.
Thanks for figuring it out.

+#endif
   if (!cloned_layer_tree_ || !cloned_layer_tree_->root()) {
     cloned_layer_tree_.reset();
     return;