Bug 180926

Summary: [Win] SHOULD NEVER BE REACHED in WebFrameLoaderClient::pageID
Product: WebKit Reporter: Fujii Hironori <Hironori.Fujii>
Component: WebKit Misc.Assignee: Fujii Hironori <Hironori.Fujii>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, pvollan, thorton, webkit-bug-importer, wilander
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Fujii Hironori
Reported 2017-12-18 00:50:35 PST
[Win] SHOULD NEVER BE REACHED in WebFrameLoaderClient::pageID I tested with trunk@226017, WinCairo, Debug build. 1) Start MiniBrowser 2) Open http://google.com > SHOULD NEVER BE REACHED > C:\webkit\ga\Source\WebKitLegacy\win\WebCoreSupport\WebFrameLoaderClient.cpp(126) : WebFrameLoaderClient::pageID Maybe caused by Bug 180682. Callstack: > WTF.dll!WTFCrash() Line 272 C++ Symbols loaded. > WebKit.dll!WebFrameLoaderClient::pageID() Line 126 C++ Symbols loaded. > WebKit.dll!WebCore::setCookies(WebCore::Document & document, const WebCore::URL & url, const WTF::String & cookieString) Line 80 C++ Symbols loaded. > WebKit.dll!WebCore::Document::setCookie(const WTF::String & value) Line 4537 C++ Symbols loaded. > WebKit.dll!WebCore::setJSDocumentCookieSetter::__l2::<lambda>() Line 1267 C++ Symbols loaded. > WebKit.dll!WebCore::AttributeSetter::call<WebCore::ExceptionOr<void> <lambda>(void) >(JSC::ExecState & state, JSC::ThrowScope & throwScope, WebCore::setJSDocumentCookieSetter::__l2::WebCore::ExceptionOr<void> <lambda>(void) && functor) Line 104 C++ Symbols loaded. > WebKit.dll!WebCore::setJSDocumentCookieSetter(JSC::ExecState & state, WebCore::JSDocument & thisObject, JSC::JSValue value, JSC::ThrowScope & throwScope) Line 1269 C++ Symbols loaded. > WebKit.dll!WebCore::IDLAttribute<WebCore::JSDocument>::set<&WebCore::setJSDocumentCookieSetter,0>(JSC::ExecState & state, __int64 thisValue, __int64 encodedValue, const char * attributeName) Line 50 C++ Symbols loaded. > WebKit.dll!WebCore::setJSDocumentCookie(JSC::ExecState * state, __int64 thisValue, __int64 encodedValue) Line 1275 C++ Symbols loaded. > JavaScriptCore.dll!JSC::callCustomSetter(JSC::ExecState * exec, bool(*)(JSC::ExecState *, __int64, __int64) setter, bool isAccessor, JSC::JSValue thisValue, JSC::JSValue value) Line 41 C++ Symbols loaded. > JavaScriptCore.dll!JSC::callCustomSetter(JSC::ExecState * exec, JSC::JSValue customGetterSetter, bool isAccessor, JSC::JSObject * base, JSC::JSValue thisValue, JSC::JSValue value) Line 59 C++ Symbols loaded. > JavaScriptCore.dll!JSC::JSObject::putInlineSlow(JSC::ExecState * exec, JSC::PropertyName propertyName, JSC::JSValue value, JSC::PutPropertySlot & slot) Line 794 C++ Symbols loaded. > JavaScriptCore.dll!JSC::JSObject::putInlineForJSObject(JSC::JSCell * cell, JSC::ExecState * exec, JSC::PropertyName propertyName, JSC::JSValue value, JSC::PutPropertySlot & slot) Line 229 C++ Symbols loaded. > JavaScriptCore.dll!JSC::JSCell::putInline(JSC::ExecState * exec, JSC::PropertyName propertyName, JSC::JSValue value, JSC::PutPropertySlot & slot) Line 374 C++ Symbols loaded. > JavaScriptCore.dll!JSC::JSValue::putInline(JSC::ExecState * exec, JSC::PropertyName propertyName, JSC::JSValue value, JSC::PutPropertySlot & slot) Line 912 C++ Symbols loaded. > JavaScriptCore.dll!llint_slow_path_put_by_id(JSC::ExecState * exec, JSC::Instruction * pc) Line 751 C++ Symbols loaded. > [External Code] Annotated Frame > JavaScriptCore.dll!JSC::dateProtoFuncToUTCString(JSC::ExecState * exec) Line 535 C++ Symbols loaded. > [External Code] Annotated Frame
Attachments
Patch (1.66 KB, patch)
2017-12-18 00:59 PST, Fujii Hironori
no flags
Fujii Hironori
Comment 1 2017-12-18 00:53:05 PST
RELEASE_ASSERT_NOT_REACHED of Source/WebKitLegacy/mac/WebCoreSupport/WebFrameLoaderClient.mm was removed in r225934. https://trac.webkit.org/changeset/225934/webkit#file28
Fujii Hironori
Comment 2 2017-12-18 00:59:43 PST
WebKit Commit Bot
Comment 3 2017-12-18 18:09:13 PST
Comment on attachment 329644 [details] Patch Clearing flags on attachment: 329644 Committed r226091: <https://trac.webkit.org/changeset/226091>
WebKit Commit Bot
Comment 4 2017-12-18 18:09:14 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 5 2017-12-18 18:10:27 PST
Note You need to log in before you can comment on or make changes to this bug.