Attachment 373201 Details for Bug 198323 – Patch only for trying EWS

[patch] Patch only for trying EWS

bug-198323-20190701113621.patch (text/plain), 5.79 KB, created by Fujii Hironori on 2019-06-30 19:36:23 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Fujii Hironori

Created: 2019-06-30 19:36:23 PDT

Size: 5.79 KB

patch

obsolete

>Subversion Revision: 246960
>diff --git a/Source/WebKitLegacy/win/ChangeLog b/Source/WebKitLegacy/win/ChangeLog
>index bacc0d4de8ebfe4f0d2da8044caff71255ee81ed..847ff0740bf9aa94b3c97fdf597be0736fa2d2a7 100644
>--- a/Source/WebKitLegacy/win/ChangeLog
>+++ b/Source/WebKitLegacy/win/ChangeLog
>@@ -1,3 +1,27 @@
>+2019-06-30  Fujii Hironori  <Hironori.Fujii@sony.com>
>+
>+        [WinCairo] ASSERTION FAILED: info.bmBitsPixel == 32 in createCairoContextWithHDC
>+        https://bugs.webkit.org/show_bug.cgi?id=198323
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        WebView::paint binds m_backingStoreBitmap to a DC by using
>+        SelectObject. m_backingStoreBitmap can't be bound to multiple DCs
>+        at the same time. However, WebView::paint can be called
>+        recursively. If such case, SelectObject was failing to bind
>+        m_backingStoreBitmap.
>+
>+        Call WebCore::Page::updateRendering before binding
>+        m_backingStoreBitmap instead of after it.
>+
>+        Reverted r202744 change which is not needed by this change.
>+
>+        * WebView.cpp:
>+        (WebView::scrollBackingStore): Removed r202744's change.
>+        (WebView::updateBackingStore): Removed m_page->updateRendering().
>+        (WebView::paint): Do m_page->updateRendering() before binding m_backingStoreBitmap.
>+        * WebView.h: Removed unused WebView::isPainting.
>+
> 2019-06-30  Fujii Hironori  <Hironori.Fujii@sony.com>
> 
>         [Win] Multiline mode of tooltip control does word-wrapping very slowly
>diff --git a/Source/WebKitLegacy/win/WebView.cpp b/Source/WebKitLegacy/win/WebView.cpp
>index 002c6ebc5bff70c68a8d59bf49a90f5ffdb7e379..207a238c223224bf828fd9569501c67b1e82c3ba 100644
>--- a/Source/WebKitLegacy/win/WebView.cpp
>+++ b/Source/WebKitLegacy/win/WebView.cpp
>@@ -1015,15 +1015,7 @@ void WebView::scrollBackingStore(FrameView* frameView, int logicalDx, int logica
>     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;
>-    }
>+    ASSERT(oldBitmap);
> 
>     // Scroll the bitmap.
>     RECT scrollRectWin(scrollViewRect);
>@@ -1164,6 +1156,7 @@ void WebView::updateBackingStore(FrameView* frameView, HDC dc, bool backingStore
>         bitmapDCObject = adoptGDIObject(::CreateCompatibleDC(windowDC));
>         bitmapDC = bitmapDCObject.get();
>         oldBitmap = ::SelectObject(bitmapDC, m_backingStoreBitmap->get());
>+        ASSERT(oldBitmap);
> #if USE(DIRECT2D)
>         HRESULT hr = m_backingStoreGdiInterop->GetDC(D2D1_DC_INITIALIZE_MODE_COPY, &bitmapDC);
>         RELEASE_ASSERT(SUCCEEDED(hr));
>@@ -1171,9 +1164,6 @@ void WebView::updateBackingStore(FrameView* frameView, HDC dc, bool backingStore
>     }
> 
>     if (m_backingStoreBitmap && (m_backingStoreDirtyRegion || backingStoreCompletelyDirty)) {
>-        // Do a layout first so that everything we render to the backing store is always current.
>-        m_page->updateRendering();
>-
>         Vector<IntRect> paintRects;
>         if (!backingStoreCompletelyDirty && m_backingStoreDirtyRegion) {
>             RECT regionBox;
>@@ -1305,6 +1295,8 @@ void WebView::paint(HDC dc, LPARAM options)
>         return;
>     }
> 
>+    m_page->updateRendering();
>+
>     Frame* coreFrame = core(m_mainFrame);
>     if (!coreFrame)
>         return;
>@@ -1341,8 +1333,6 @@ void WebView::paint(HDC dc, LPARAM options)
>         return;
>     }
> 
>-    m_paintCount++;
>-
>     auto bitmapDC = adoptGDIObject(::CreateCompatibleDC(hdc));
>     HGDIOBJ oldBitmap = ::SelectObject(bitmapDC.get(), m_backingStoreBitmap->get());
> 
>@@ -1377,8 +1367,6 @@ void WebView::paint(HDC dc, LPARAM options)
>     m_backingStoreGdiInterop->ReleaseDC(nullptr);
> #endif
> 
>-    m_paintCount--;
>-
>     if (active())
>         cancelDeleteBackingStoreSoon();
>     else
>diff --git a/Source/WebKitLegacy/win/WebView.h b/Source/WebKitLegacy/win/WebView.h
>index 1084d4063e29c3697a5f7af8ef7c52f51af90a9a..8363772c0063024ca335b5a3334cb5a4a51b283d 100644
>--- a/Source/WebKitLegacy/win/WebView.h
>+++ b/Source/WebKitLegacy/win/WebView.h
>@@ -475,8 +475,6 @@ public:
>     const char* interpretKeyEvent(const WebCore::KeyboardEvent*);
>     bool handleEditingKeyboardEvent(WebCore::KeyboardEvent&);
> 
>-    bool isPainting() const { return m_paintCount > 0; }
>-
>     void setToolTip(const WTF::String&);
> 
>     void registerForIconNotification(bool listen);
>@@ -670,7 +668,6 @@ protected:
>     COMPtr<IDropTargetHelper> m_dropTargetHelper;
>     UChar m_currentCharacterCode { 0 };
>     bool m_isBeingDestroyed { false };
>-    unsigned m_paintCount { 0 };
>     bool m_hasSpellCheckerDocumentTag { false };
>     bool m_didClose { false };
>     bool m_hasCustomDropTarget { false };
>diff --git a/LayoutTests/imported/blink/animations/display-inline-style-adjust-expected.html b/LayoutTests/imported/blink/animations/display-inline-style-adjust-expected.html
>index c237fdef0e0ecd702930683ebc969248a760039b..05fd50d4d566e48744db45ca45afca82a2779742 100644
>--- a/LayoutTests/imported/blink/animations/display-inline-style-adjust-expected.html
>+++ b/LayoutTests/imported/blink/animations/display-inline-style-adjust-expected.html
>@@ -8,5 +8,5 @@ div {
> </style>
> 
> <div>
>-  <span>This sentence should span a single line.</span>
>+  <span>This sentence shoul</span><span>d span a single line.</span>
> </div>

Flags:

ews-watchlist: commit-queue-

Actions: View | Formatted Diff | Diff

Attachments on bug 198323: 370834 | 373004 | 373012 | 373201 | 373209 | 373379