Summary: | Removing a plug-in element from a page opened in a background tab in Safari crashes | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brady Eidson <beidson> | ||||||||
Component: | WebKit2 | Assignee: | Brady Eidson <beidson> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bdakin, gustavo, philn, thorton, vivekgalatage, xan.lopez | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 95007 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Brady Eidson
2012-08-13 17:26:01 PDT
Created attachment 158157 [details]
Patch v1 - Fix + regression test
Hmmmm I see there's a GTK failure already... ./.libs/libWebCoreInternals.a(./.libs/../Source/WebCore/testing/.libs/libWebCoreInternals_la-InternalSettings.o):InternalSettings.cpp:function WebCore::InternalSettings::Backup::restoreTo(WebCore::Page*, WebCore::Settings*): error: undefined reference to 'WebCore::Page::setCanStartMedia(bool)' ./.libs/libWebCoreInternals.a(./.libs/../Source/WebCore/testing/.libs/libWebCoreInternals_la-InternalSettings.o):InternalSettings.cpp:function WebCore::InternalSettings::setCanStartMedia(bool, int&): error: undefined reference to 'WebCore::Page::setCanStartMedia(bool)' collect2: error: ld returned 1 exit status As far as I can tell Page::setCanStartMedia() is defined in all ports. Dunno what this is about yet. Comment on attachment 158157 [details] Patch v1 - Fix + regression test Attachment 158157 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13486729 (In reply to comment #2) > As far as I can tell Page::setCanStartMedia() is defined in all ports. Dunno what this is about yet. The Windows error makes me wonder if it's not something to do with exports or something? (In reply to comment #4) > (In reply to comment #2) > > As far as I can tell Page::setCanStartMedia() is defined in all ports. Dunno what this is about yet. > > The Windows error makes me wonder if it's not something to do with exports or something? Indeed the win and gtk failures are uncannily the same error... I was under the (perhaps quite mistaken) impression that Mac was the only platform with explicit symbol exporting (WebCore.exp.in). Grumble! Created attachment 158578 [details]
Patch v2 - Shot at fixing GTK
Comment on attachment 158578 [details] Patch v2 - Shot at fixing GTK Attachment 158578 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13516028 (In reply to comment #7) > (From update of attachment 158578 [details]) > Attachment 158578 [details] did not pass win-ews (win): > Output: http://queues.webkit.org/results/13516028 I think I have a response to the Win failure (WebKit2.def for exports) but will let the gtk ews play out on patch v2 first. Created attachment 158593 [details]
Patch v3 - All the right exports in all the right places
Will land after win ews confirms it builds. |