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+

mitz
Reported 2013-12-17 11:33:58 PST
Fix an assertion failure seen running some mixed-content tests
Attachments
Remove a bogus assertion (1.32 KB, patch)
2013-12-17 11:35 PST, mitz 		ap: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2013-12-17 11:35:12 PST
Created attachment 219434 [details] Remove a bogus assertion
Alexey Proskuryakov
Comment 2 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()
mitz
Comment 3 2013-12-17 11:43:25 PST
Fixed in <http://trac.webkit.org/r160718>.
Note You need to log in before you can comment on or make changes to this bug.