Make createWindow (mostly) work with OOPIF

Source/web/PopupContainer.cpp

 /*
  * Copyright (c) 2011, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 182 matching lines @@
     // points. If the <select>/<input> is transformed, they are not the same.
     int verticalOffset = - m_controlPosition.p4().y() + m_controlPosition.p3().y();
     int verticalForRTLOffset = isRTL ? verticalOffset : 0;
 
     // Assume m_listBox size is already calculated.
     IntSize targetSize(m_listBox->width() + borderSize * 2, m_listBox->height() + borderSize * 2);
 
     IntRect widgetRectInScreen;
     // If the popup would extend past the bottom of the screen, open upwards
     // instead.
-    IntRect screen = screenAvailableRect(m_frameView.get());
+    Widget* rootWidget = m_frameView.get();
+    while (rootWidget->parent())
+        rootWidget = rootWidget->parent();
+    IntRect screen = rootWidget->hostWindow() ? screenAvailableRect(*rootWidget->hostWindow()) : IntRect();

[dcheng, 2015/05/04 17:38:11]

Does m_frameView->root() not work here?

     // Use popupInitialCoordinate.x() + rightOffset because RTL position
     // needs to be considered.
     float pageScaleFactor = m_frameView->frame().page()->pageScaleFactor();
     int popupX = round((popupInitialCoordinate.x() + rightOffset) * pageScaleFactor);
     int popupY = round((popupInitialCoordinate.y() + verticalForRTLOffset) * pageScaleFactor);
     widgetRectInScreen = chromeClient().viewportToScreen(IntRect(popupX, popupY, targetSize.width(), targetSize.height()));
 
     // If we have multiple screens and the browser rect is in one screen, we
     // have to clip the window width to the screen width.
     // When clipping, we also need to set a maximum width for the list box.
@@ skipping 279 matching lines @@
 }
 
 IntPoint PopupContainer::convertSelfToChild(const Widget* child, const IntPoint& point) const
 {
     IntPoint newPoint = point;
     newPoint.moveBy(-child->location());
     return newPoint;
 }
 
 } // namespace blink