Remove the forceCompositingMode setting.

Source/web/WebPopupMenuImpl.cpp

Index: Source/web/WebPopupMenuImpl.cpp
diff --git a/Source/web/WebPopupMenuImpl.cpp b/Source/web/WebPopupMenuImpl.cpp
index 03abd7d7974f6d4ee427d26acb325c5f8c8780bf..3a9134d708df741f1a82c70525529691308c6f0a 100644
--- a/Source/web/WebPopupMenuImpl.cpp
+++ b/Source/web/WebPopupMenuImpl.cpp
@@ -206,6 +206,9 @@ void WebPopupMenuImpl::layout()
 
 void WebPopupMenuImpl::enterForceCompositingMode(bool enter)
 {
+    // FIXME: Is this function really correct? The other overrides of this function

[enne (OOO), 2014/05/27 23:44:13]

This is the equivalent of WebViewImpl::setRootGraphicsLayer.  Maybe this could
all just get moved to WebPopupMenuImpl::initialize?

Also, there are no callers of enterForceCompositingMode(false) anymore, so the
only branch that's needed here is the one that creates the layer tree view and
root layer.

[ojan, 2014/05/27 23:54:35]

This is called with false in this file unfortunately in
WebPopupMenuImpl::willCloseLayerTreeView. willCloseLayerTreeView in
WebPagePopupImpl and WebViewImpl call setIsAcceleratedCompositingActive instead,
which I think this was this function should be called to match those. But, I
didn't want to risk changing any behavior with this patch.

+    // just used to set forceCompositingMode before we removed that setting.
+    // crbug.com/378029
     if (m_isAcceleratedCompositingActive == enter)
         return;