• beidson@apple.com's avatar
    Page cache should support pages with plugins. · 343e49ba
    beidson@apple.com authored
    <rdar://problem/5190122> and https://bugs.webkit.org/show_bug.cgi?id=13634
    Reviewed by Anders Carlsson.
    By making plugin elements renderers go display:none when entering the page cache,
    we destroy the plug-in in a cross platform way as well as handle invalidating script
    objects created by that plugin.
    By restoring the original style when leaving the page cache and forcing a style recalc
    on the plugin element, the plugin is gracefully reinstantiated when the user goes back.
    Test: plugins/netscape-plugin-page-cache-works.html
    * dom/Document.cpp:
    (WebCore::Document::documentDidBecomeActive): Copy this collection before iterating over
      it, as the callbacks might result in mutating the set.
    * dom/Node.h:
    (WebCore::Node::clearHasCustomStyleForRenderer): Expose the ability to stop using a 
      custom renderer and go back to the default renderer.
    * history/CachedFrame.cpp:
    (WebCore::CachedFrame::CachedFrame): Move the document inactivation call to the same place
      we suspend active DOM objects. It is important this call be *after* the beforeunload event
      is dispatched, and was coincidental non of the elements that using Document activation
      had run in to this problem yet.
    * history/PageCache.cpp:
    (WebCore::PageCache::canCachePageContainingThisFrame): If the page contains plugins but
      the PageCacheSupportsPlugins setting is true, allow this page.
    Kill and recreate the plugin by listening for Document activation callbacks and setting a custom
    display:none render style:
    * html/HTMLPlugInImageElement.cpp:
    (WebCore::HTMLPlugInImageElement::~HTMLPlugInImageElement): Unregister for document activation
    (WebCore::HTMLPlugInImageElement::createRenderer): Once a renderer (ie. plugin instance) is
      created, this element needs Document (in)activation callbacks.
    (WebCore::HTMLPlugInImageElement::willMoveToNewOwnerDocument): Manage Document activation 
      callback registration.
    (WebCore::HTMLPlugInImageElement::didMoveToNewOwnerDocument): Ditto.
    (WebCore::HTMLPlugInImageElement::documentWillBecomeInactive): Clone the element's current style
      and set the clone's display value to None. Start using this custom style and force a style
      recall. This destroys the renderer and therefore the plugin instance.
    (WebCore::HTMLPlugInImageElement::documentDidBecomeActive): Stop using the custom style and
      force a style recall to reinstantiate the plugin.
    (WebCore::HTMLPlugInImageElement::customStyleForRenderer): Return the stand-in style that has
      display:none set.
    * html/HTMLPlugInImageElement.h:
    Add a setting that allows runtime configuration of whether or not the page cache supports plugins:
    * page/Settings.cpp:
    * page/Settings.h:
    Expose a WebKit preference for the page cache supporting plugins (on by default).
    Reviewed by Anders Carlsson.
    * WebView/WebPreferenceKeysPrivate.h:
    * WebView/WebPreferences.mm:
    (-[WebPreferences pageCacheSupportsPlugins]):
    (-[WebPreferences setPageCacheSupportsPlugins:]):
    * WebView/WebPreferencesPrivate.h:
    * WebView/WebView.mm:
    (-[WebView _preferencesChanged:]):
    Expose a WebKit2 preference for the page cache supporting plugins (on by default).
    Reviewed by Anders Carlsson.
    * Shared/WebPreferencesStore.h:
    * UIProcess/API/C/WKPreferences.cpp:
    * UIProcess/API/C/WKPreferencesPrivate.h:
    * WebProcess/WebPage/WebPage.cpp:
    Reviewed by Anders Carlsson.
    * plugins/netscape-plugin-page-cache-works-expected.txt: Added.
    * plugins/netscape-plugin-page-cache-works.html: Added.
    * plugins/resources/go-back.html: Added.
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@102619 268f45cc-cd09-0410-ab3c-d52691b4dbfc
netscape-plugin-page-cache-works-expected.txt 1.33 KB