Bug 180926 - [Win] SHOULD NEVER BE REACHED in WebFrameLoaderClient::pageID
Summary: [Win] SHOULD NEVER BE REACHED in WebFrameLoaderClient::pageID
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Fujii Hironori
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-12-18 00:50 PST by Fujii Hironori
Modified: 2017-12-18 18:10 PST (History)
5 users (show)

See Also:


Attachments
Patch (1.66 KB, patch)
2017-12-18 00:59 PST, Fujii Hironori
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fujii Hironori 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
Comment 1 Fujii Hironori 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
Comment 2 Fujii Hironori 2017-12-18 00:59:43 PST
Created attachment 329644 [details]
Patch
Comment 3 WebKit Commit Bot 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>
Comment 4 WebKit Commit Bot 2017-12-18 18:09:14 PST
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2017-12-18 18:10:27 PST
<rdar://problem/36121778>