Bug 125862

Summary: Assertion failure seen running some mixed-content tests
Product: WebKit Reporter: mitz
Component: WebKit2Assignee: mitz
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Remove a bogus assertion ap: review+

Description mitz 2013-12-17 11:33:58 PST
Fix an assertion failure seen running some mixed-content tests
Comment 1 mitz 2013-12-17 11:35:12 PST
Created attachment 219434 [details]
Remove a bogus assertion
Comment 2 Alexey Proskuryakov 2013-12-17 11:41:33 PST
Comment on attachment 219434 [details]
Remove a bogus assertion

View in context: https://bugs.webkit.org/attachment.cgi?id=219434&action=review

> Source/WebKit2/UIProcess/PageLoadState.cpp:-275
> -    ASSERT(m_uncommittedState.url.startsWith("https:", false));

This is a new assertion that seem OK to remove for now, but the code looks very surprising and inconsistent. 

canRunInsecureContent is invoked with m_document, and canDisplayInsecureContent is invoked with topFrame->document()
Comment 3 mitz 2013-12-17 11:43:25 PST
Fixed in <http://trac.webkit.org/r160718>.