[Webkit-unassigned] [Bug 115830] New: HTMLEmbedElement::updateWidget should not schedule, load, or cancel a page load

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=115830

           Summary: HTMLEmbedElement::updateWidget should not schedule,
                    load, or cancel a page load
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Plug-ins
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: andersca at apple.com, simon.fraser at apple.com


We shouldn’t do things like:

    #1 0x7f26f14cf151 in WebCore::FrameLoader::completed() third_party/WebKit/Source/WebCore/loader/FrameLoader.cpp:1084
    #2 0x7f26f14ceada in WebCore::FrameLoader::checkCompleted() third_party/WebKit/Source/WebCore/loader/FrameLoader.cpp:782
    #3 0x7f26f157a31f in WebCore::CachedResourceLoader::loadDone(WebCore::CachedResource*) third_party/WebKit/Source/WebCore/loader/cache/CachedResourceLoader.cpp:748
    #4 0x7f26f153e54a in WebCore::SubresourceLoader::releaseResources() third_party/WebKit/Source/WebCore/loader/SubresourceLoader.cpp:322
    #5 0x7f26f1533491 in WebCore::ResourceLoader::cancel(WebCore::ResourceError const&) third_party/WebKit/Source/WebCore/loader/ResourceLoader.cpp:420
    #6 0x7f26f150aa69 in WebCore::MainResourceLoader::cancel(WebCore::ResourceError const&) third_party/WebKit/Source/WebCore/loader/MainResourceLoader.cpp:129
    #7 0x7f26f150a4f3 in WebCore::MainResourceLoader::cancel() third_party/WebKit/Source/WebCore/loader/MainResourceLoader.cpp:111
    #8 0x7f26f14989c8 in WebCore::DocumentLoader::stopLoading() third_party/WebKit/Source/WebCore/loader/DocumentLoader.cpp:257
    #9 0x7f26f151440c in WebCore::NavigationScheduler::schedule(WTF::PassOwnPtr<WebCore::ScheduledNavigation>) third_party/WebKit/Source/WebCore/loader/NavigationScheduler.cpp:440
    #10 0x7f26f1514cba in WebCore::NavigationScheduler::scheduleLocationChange(WebCore::SecurityOrigin*, WTF::String const&, WTF::String const&, bool, bool) third_party/WebKit/Source/WebCore/loader/NavigationScheduler.cpp:361
    #11 0x7f26f15361de in WebCore::SubframeLoader::loadOrRedirectSubframe(WebCore::HTMLFrameOwnerElement*, WebCore::KURL const&, WTF::AtomicString const&, bool, bool) third_party/WebKit/Source/WebCore/loader/SubframeLoader.cpp:339
    #12 0x7f26f153878b in WebCore::SubframeLoader::requestObject(WebCore::HTMLPlugInImageElement*, WTF::String const&, WTF::AtomicString const&, WTF::String const&, WTF::Vector<WTF::String, 0ul> const&, WTF::Vector<WTF::String, 0ul> const&) third_party/WebKit/Source/WebCore/loader/SubframeLoader.cpp:242
    #13 0x7f26f5dcec01 in WebCore::HTMLEmbedElement::updateWidget(WebCore::PluginCreationOption) third_party/WebKit/Source/WebCore/html/HTMLEmbedElement.cpp:173

Ideally, these things are done asynchronously in some queue instead.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.