ChangeLog

2016-07-01  Per Arne Vollan  <pvollan@apple.com>

        ASSERTION FAILED: info.bmBitsPixel == 32

        https://bugs.webkit.org/show_bug.cgi?id=17737

        Reviewed by NOBODY (OOPS!).

        The ::SelectObject call will fail if m_backingStoreBitmap is already selected into a device context.

        This happens when this method is called indirectly from WebView::updateBackingStore during normal

        WM_PAINT handlind. There is no point continuing, since we would just be scrolling a 1x1 bitmap which

        is selected into the device context by default. We can just scroll by repainting the scroll rectangle.

        * WebView.cpp:

        (WebView::scrollBackingStore): Invalidate the scroll rectangle if the ::SelectObject call fails.

2016-07-01  Youenn Fablet  <youennf@gmail.com>

        Add a runtime flag for DOM iterators

202728

Source/WebKit/win/WebView.cpp

@@void WebView::scrollBackingStore(FrameVi

    HWndDC windowDC(m_viewWindow);

    auto bitmapDC = adoptGDIObject(::CreateCompatibleDC(windowDC));

    HGDIOBJ oldBitmap = ::SelectObject(bitmapDC.get(), m_backingStoreBitmap->get());

    if (!oldBitmap) {

        // The ::SelectObject call will fail if m_backingStoreBitmap is already selected into a device context.

        // This happens when this method is called indirectly from WebView::updateBackingStore during normal WM_PAINT handling.

        // There is no point continuing, since we would just be scrolling a 1x1 bitmap which is selected into the device context by default.

        // We can just scroll by repainting the scroll rectangle.

        RECT scrollRect(scrollViewRect);

        ::InvalidateRect(m_viewWindow, &scrollRect, FALSE);

        return;

    }

    // Scroll the bitmap.

    RECT scrollRectWin(scrollViewRect);

@@HRESULT WebView::findString(_In_ BSTR st

    *found = m_page->findString(toString(string), options);

    return S_OK;

}

7591

202676