When Safari reloads pages with Flash objects after Flash is installed, placeholders...
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Jan 2017 20:58:35 +0000 (20:58 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Jan 2017 20:58:35 +0000 (20:58 +0000)
https://bugs.webkit.org/show_bug.cgi?id=167268
rdar://problem/29857388

Reviewed by Sam Weinig.

Source/WebCore:

* page/Page.cpp:
(WebCore::Page::refreshPlugins):
Call PluginInfoProvider::refresh instead.

(WebCore::Page::clearPluginData):
Add new setter.

* page/Page.h:
Declare new members.

* plugins/PluginInfoProvider.cpp:
(WebCore::PluginInfoProvider::refresh):
Put the logic to clear plug-in data and reload frames here.

* plugins/PluginInfoProvider.h:
Add new members.

Source/WebKit2:

* WebProcess/Plugins/WebPluginInfoProvider.h:
Make refreshPlugins() private.

* WebProcess/WebProcess.cpp:
(WebKit::WebProcess::refreshPlugins):
Call the newly added PluginInfoProvider::refresh().

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@211052 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/Page.cpp
Source/WebCore/page/Page.h
Source/WebCore/plugins/PluginInfoProvider.cpp
Source/WebCore/plugins/PluginInfoProvider.h
Source/WebKit2/ChangeLog
Source/WebKit2/WebProcess/Plugins/WebPluginInfoProvider.h
Source/WebKit2/WebProcess/WebProcess.cpp

index 1976f61..ca7c0be 100644 (file)
@@ -1,3 +1,28 @@
+2017-01-20  Anders Carlsson  <andersca@apple.com>
+
+        When Safari reloads pages with Flash objects after Flash is installed, placeholders don't paint (but do work!)
+        https://bugs.webkit.org/show_bug.cgi?id=167268
+        rdar://problem/29857388
+
+        Reviewed by Sam Weinig.
+
+        * page/Page.cpp:
+        (WebCore::Page::refreshPlugins):
+        Call PluginInfoProvider::refresh instead.
+
+        (WebCore::Page::clearPluginData):
+        Add new setter.
+
+        * page/Page.h:
+        Declare new members.
+
+        * plugins/PluginInfoProvider.cpp:
+        (WebCore::PluginInfoProvider::refresh):
+        Put the logic to clear plug-in data and reload frames here.
+
+        * plugins/PluginInfoProvider.h:
+        Add new members.
+
 2017-01-23  Jer Noble  <jer.noble@apple.com>
 
         REGRESSION (r208149): Video details does not apear and missing scrubber in Control Center
index 7d6af11..845460f 100644 (file)
@@ -539,24 +539,11 @@ void Page::refreshPlugins(bool reload)
 
     Vector<Ref<Frame>> framesNeedingReload;
 
-    for (auto& page : *allPages) {
+    for (auto& page : *allPages)
         pluginInfoProviders.add(&page->pluginInfoProvider());
-        page->m_pluginData = nullptr;
-
-        if (!reload)
-            continue;
-        
-        for (Frame* frame = &page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
-            if (frame->loader().subframeLoader().containsPlugins())
-                framesNeedingReload.append(*frame);
-        }
-    }
 
     for (auto& pluginInfoProvider : pluginInfoProviders)
-        pluginInfoProvider->refreshPlugins();
-
-    for (auto& frame : framesNeedingReload)
-        frame->loader().reload();
+        pluginInfoProvider->refresh(reload);
 }
 
 PluginData& Page::pluginData()
@@ -566,6 +553,11 @@ PluginData& Page::pluginData()
     return *m_pluginData;
 }
 
+void Page::clearPluginData()
+{
+    m_pluginData = nullptr;
+}
+
 bool Page::showAllPlugins() const
 {
     if (m_showAllPlugins)
index 0d2c735..f966db6 100644 (file)
@@ -166,6 +166,7 @@ public:
 
     static void refreshPlugins(bool reload);
     WEBCORE_EXPORT PluginData& pluginData();
+    void clearPluginData();
 
     WEBCORE_EXPORT void setCanStartMedia(bool);
     bool canStartMedia() const { return m_canStartMedia; }
index 0932b90..96a210e 100644 (file)
 #include "config.h"
 #include "PluginInfoProvider.h"
 
+#include "MainFrame.h"
+#include "Page.h"
+#include "SubframeLoader.h"
+
 namespace WebCore {
 
 PluginInfoProvider::~PluginInfoProvider()
@@ -33,6 +37,26 @@ PluginInfoProvider::~PluginInfoProvider()
     ASSERT(m_pages.isEmpty());
 }
 
+void PluginInfoProvider::refresh(bool reloadPages)
+{
+    Vector<Ref<Frame>> framesNeedingReload;
+
+    for (auto& page : m_pages) {
+        page->clearPluginData();
+
+        if (!reloadPages)
+            continue;
+
+        for (Frame* frame = &page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
+            if (frame->loader().subframeLoader().containsPlugins())
+                framesNeedingReload.append(*frame);
+        }
+    }
+
+    for (auto& frame : framesNeedingReload)
+        frame->loader().reload();
+}
+
 void PluginInfoProvider::addPage(Page& page)
 {
     ASSERT(!m_pages.contains(&page));
index 68be151..dafa165 100644 (file)
@@ -34,16 +34,17 @@ class WEBCORE_EXPORT PluginInfoProvider : public RefCounted<PluginInfoProvider>
 public:
     virtual ~PluginInfoProvider();
 
-    void refreshPlugins(bool reloadPages);
+    void refresh(bool reloadPages);
 
     void addPage(Page&);
     void removePage(Page&);
 
-    virtual void refreshPlugins() = 0;
     virtual void getPluginInfo(Page&, Vector<PluginInfo>&) = 0;
     virtual void getWebVisiblePluginInfo(Page&, Vector<PluginInfo>&) = 0;
 
 private:
+    virtual void refreshPlugins() = 0;
+
     HashSet<Page*> m_pages;
 };
 
index c16d8ce..929694e 100644 (file)
@@ -1,3 +1,18 @@
+2017-01-20  Anders Carlsson  <andersca@apple.com>
+
+        When Safari reloads pages with Flash objects after Flash is installed, placeholders don't paint (but do work!)
+        https://bugs.webkit.org/show_bug.cgi?id=167268
+        rdar://problem/29857388
+
+        Reviewed by Sam Weinig.
+
+        * WebProcess/Plugins/WebPluginInfoProvider.h:
+        Make refreshPlugins() private.
+
+        * WebProcess/WebProcess.cpp:
+        (WebKit::WebProcess::refreshPlugins):
+        Call the newly added PluginInfoProvider::refresh().
+
 2017-01-23  Antti Koivisto  <antti@apple.com>
 
         Use priorities in speculative revalidation
index af7f96d..30fda4f 100644 (file)
@@ -42,13 +42,12 @@ public:
     void clearPluginClientPolicies();
 #endif
 
-    void refreshPlugins() override;
-
 private:
     WebPluginInfoProvider();
 
     void getPluginInfo(WebCore::Page&, Vector<WebCore::PluginInfo>&) override;
     void getWebVisiblePluginInfo(WebCore::Page&, Vector<WebCore::PluginInfo>&) override;
+    void refreshPlugins() override;
 
 #if ENABLE(NETSCAPE_PLUGIN_API)
     void populatePluginCache(const WebCore::Page&);
index 965c8f6..e1c26b9 100644 (file)
@@ -907,7 +907,7 @@ void WebProcess::clearPluginClientPolicies()
 void WebProcess::refreshPlugins()
 {
 #if ENABLE(NETSCAPE_PLUGIN_API)
-    WebPluginInfoProvider::singleton().refreshPlugins();
+    WebPluginInfoProvider::singleton().refresh(false);
 #endif
 }