[HBD] Remove PluginCache and reload plugin list when origin changes

third_party/WebKit/Source/core/page/Page.h

Index: third_party/WebKit/Source/core/page/Page.h
diff --git a/third_party/WebKit/Source/core/page/Page.h b/third_party/WebKit/Source/core/page/Page.h
index 6f75adeca66167b95faa128a92e802f37340a819..a9b6edf573679618bd8b10ac9839798bed1d93cb 100644
--- a/third_party/WebKit/Source/core/page/Page.h
+++ b/third_party/WebKit/Source/core/page/Page.h
@@ -119,7 +119,7 @@ public:
     ViewportDescription viewportDescription() const;
 
     static void refreshPlugins();
-    PluginData* pluginData() const;
+    PluginData* pluginData();
 
     EditorClient& editorClient() const { return *m_editorClient; }
     SpellCheckerClient& spellCheckerClient() const { return *m_spellCheckerClient; }
@@ -260,6 +260,8 @@ private:
 
     bool m_isCursorVisible;
 
+    RefPtr<SecurityOrigin> m_mainFrameOrigin;

[dcheng, 2016/08/03 14:34:58]

I would prefer we find a way to write this without caching the main frame origin
like this. For one, it's not actually accurate since it only gets updated if
pluginData is called.

[trizzofo, 2016/08/03 19:11:18]

We are considering storing it in PluginData. Take a look at tommycli's comment
in Page.h in Patch Set 7:

https://codereview.chromium.org/2156803002/diff/120001/third_party/WebKit/Sou...

Would that be a better approach?

[tommycli, 2016/08/03 20:53:39]

If we put m_origin in PluginData, hopefully it's then clear that it's only the
PluginData associated with that origin. I agree that it's confusing as-is. 

I suggested to Tomas to put m_origin in Page, so it was my bad suggestion.

+
 #if ENABLE(ASSERT)
     bool m_isPainting;
 #endif