Bug 17737 - ASSERTION FAILED: info.bmBitsPixel == 32
Summary: ASSERTION FAILED: info.bmBitsPixel == 32
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Per Arne Vollan
URL: http://www.edmondsun.com/opinion/loca...
Keywords: InRadar, PlatformOnly
Depends on:
Blocks:
 
Reported: 2008-03-09 21:43 PDT by Matt Lilek
Modified: 2019-06-26 22:45 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.25 KB, patch)
2016-07-01 02:32 PDT, Per Arne Vollan
no flags Details | Formatted Diff | Diff
Patch (2.31 KB, patch)
2016-07-01 02:58 PDT, Per Arne Vollan
no flags Details | Formatted Diff | Diff
Patch (2.30 KB, patch)
2016-07-01 03:01 PDT, Per Arne Vollan
bfulgham: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Lilek 2008-03-09 21:43:17 PDT
On Windows, I'm hitting this assert resizing the window with the URL in a background tab.

1. Load <http://reddit.com>
2. Right click and open the URL in a new tab (""God did it!" may soon be an acceptable answer to questions on a Science test in Oklahoma if this bill passes.")
3. Resize the window a bit (I believe until the tab in the background would not need horizontal scrollbars if it were visible)

ASSERT(info.bmBitsPixel == 32);
http://trac.webkit.org/projects/webkit/browser/trunk/WebCore/platform/graphics/win/GraphicsContextCGWin.cpp#L48

According to the debugger, info.bmBitsPixel is actually 1.

>	WebKit.dll!WebCore::CGContextWithHDC(HDC__ * hdc=0xc4010892)  Line 48 + 0x22 bytes	C++
 	WebKit.dll!WebCore::GraphicsContext::GraphicsContext(HDC__ * hdc=0xc4010892)  Line 66 + 0x52 bytes	C++
 	WebKit.dll!WebView::paintIntoBackingStore(WebCore::FrameView * frameView=0x01dc01f0, HDC__ * bitmapDC=0xc4010892, const WebCore::IntRect & dirtyRect={...})  Line 905	C++
 	WebKit.dll!WebView::updateBackingStore(WebCore::FrameView * frameView=0x01dc01f0, HDC__ * dc=0xc4010892, bool backingStoreCompletelyDirty=false)  Line 798 + 0x1d bytes	C++
 	WebKit.dll!WebView::scrollBackingStore(WebCore::FrameView * frameView=0x01dc01f0, int dx=0, int dy=9, const WebCore::IntRect & scrollViewRect={...}, const WebCore::IntRect & clipRect={...})  Line 721	C++
 	WebKit.dll!WebChromeClient::scrollBackingStore(int dx=0, int dy=9, const WebCore::IntRect & scrollViewRect={...}, const WebCore::IntRect & clipRect={...})  Line 431	C++
 	WebKit.dll!WebCore::Chrome::scrollBackingStore(int dx=0, int dy=9, const WebCore::IntRect & scrollViewRect={...}, const WebCore::IntRect & clipRect={...})  Line 299 + 0x28 bytes	C++
 	WebKit.dll!WebCore::ScrollView::scrollBackingStore(int dx=0, int dy=9, const WebCore::IntRect & scrollViewRect={...}, const WebCore::IntRect & clipRect={...})  Line 829	C++
 	WebKit.dll!WebCore::ScrollView::ScrollViewPrivate::scrollBackingStore(const WebCore::IntSize & scrollDelta={...})  Line 167	C++
 	WebKit.dll!WebCore::ScrollView::ScrollViewPrivate::valueChanged(WebCore::Scrollbar * bar=0x04e8da18)  Line 150	C++
 	WebKit.dll!WebCore::Scrollbar::setValue(int v=637)  Line 60 + 0x1e bytes	C++
 	WebKit.dll!WebCore::ScrollView::updateScrollbars(const WebCore::IntSize & desiredOffset={...})  Line 560	C++
 	WebKit.dll!WebCore::ScrollView::resizeContents(int w=1049, int h=1349)  Line 285	C++
 	WebKit.dll!WebCore::FrameView::adjustViewSize()  Line 257 + 0x47 bytes	C++
 	WebKit.dll!WebCore::FrameView::layout(bool allowSubtree=true)  Line 492	C++
 	WebKit.dll!WebCore::FrameView::layoutIfNeededRecursive()  Line 1091	C++
 	WebKit.dll!WebView::updateBackingStore(WebCore::FrameView * frameView=0x01dc01f0, HDC__ * dc=0x61010d94, bool backingStoreCompletelyDirty=true)  Line 786	C++
 	WebKit.dll!WebView::paint(HDC__ * dc=0x00000000, long options=0)  Line 853	C++
 	WebKit.dll!WebViewWndProc(HWND__ * hWnd=0x00100246, unsigned int message=15, unsigned int wParam=0, long lParam=0)  Line 1615	C++
 	user32.dll!7e418734() 	
 	[Frames below may be incorrect and/or missing, no symbols loaded for user32.dll]
Comment 1 Adam Roben (:aroben) 2008-03-10 13:09:12 PDT
I can't seem to reproduce this on ToT.
Comment 2 Per Arne Vollan 2016-07-01 02:32:27 PDT
Created attachment 282530 [details]
Patch
Comment 3 Per Arne Vollan 2016-07-01 02:58:57 PDT
Created attachment 282534 [details]
Patch
Comment 4 Per Arne Vollan 2016-07-01 03:01:43 PDT
Created attachment 282535 [details]
Patch
Comment 5 Brent Fulgham 2016-07-01 09:30:31 PDT
Comment on attachment 282535 [details]
Patch

r=me
Comment 6 Per Arne Vollan 2016-07-01 10:29:12 PDT
(In reply to comment #5)
> Comment on attachment 282535 [details]
> Patch
> 
> r=me

Thank you!
Comment 7 Per Arne Vollan 2016-07-01 11:22:53 PDT
Committed r202744: <https://trac.webkit.org/changeset/202744>
Comment 8 Brent Fulgham 2016-07-05 08:49:58 PDT
<rdar://problem/6051446>