Title: [254753] trunk/Source/WebCore
- Revision
- 254753
- Author
- cdu...@apple.com
- Date
- 2020-01-17 10:45:34 -0800 (Fri, 17 Jan 2020)
Log Message
Drop dead code related to local storage prewarming
https://bugs.webkit.org/show_bug.cgi?id=206418
Reviewed by Sam Weinig.
Drop dead code related to local storage prewarming. Sihui neutered it in <https://trac.webkit.org/changeset/248734>
but did not remove the code.
* loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::commitData):
* page/DOMWindow.cpp:
(WebCore::DOMWindow::prewarmLocalStorageIfNecessary): Deleted.
* page/DOMWindow.h:
* page/Frame.cpp:
(WebCore::Frame::didPrewarmLocalStorage): Deleted.
(WebCore::Frame::mayPrewarmLocalStorage const): Deleted.
* page/Frame.h:
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (254752 => 254753)
--- trunk/Source/WebCore/ChangeLog 2020-01-17 18:22:38 UTC (rev 254752)
+++ trunk/Source/WebCore/ChangeLog 2020-01-17 18:45:34 UTC (rev 254753)
@@ -1,3 +1,23 @@
+2020-01-17 Chris Dumez <cdu...@apple.com>
+
+ Drop dead code related to local storage prewarming
+ https://bugs.webkit.org/show_bug.cgi?id=206418
+
+ Reviewed by Sam Weinig.
+
+ Drop dead code related to local storage prewarming. Sihui neutered it in <https://trac.webkit.org/changeset/248734>
+ but did not remove the code.
+
+ * loader/DocumentLoader.cpp:
+ (WebCore::DocumentLoader::commitData):
+ * page/DOMWindow.cpp:
+ (WebCore::DOMWindow::prewarmLocalStorageIfNecessary): Deleted.
+ * page/DOMWindow.h:
+ * page/Frame.cpp:
+ (WebCore::Frame::didPrewarmLocalStorage): Deleted.
+ (WebCore::Frame::mayPrewarmLocalStorage const): Deleted.
+ * page/Frame.h:
+
2020-01-17 Adrian Perez de Castro <ape...@igalia.com>
Fix various non-unified build issues introduced since r254327
Modified: trunk/Source/WebCore/loader/DocumentLoader.cpp (254752 => 254753)
--- trunk/Source/WebCore/loader/DocumentLoader.cpp 2020-01-17 18:22:38 UTC (rev 254752)
+++ trunk/Source/WebCore/loader/DocumentLoader.cpp 2020-01-17 18:45:34 UTC (rev 254753)
@@ -1092,9 +1092,6 @@
if (!isLoading())
return;
- if (auto* window = m_frame->document()->domWindow())
- window->prewarmLocalStorageIfNecessary();
-
bool userChosen;
String encoding;
if (overrideEncoding().isNull()) {
Modified: trunk/Source/WebCore/page/DOMWindow.cpp (254752 => 254753)
--- trunk/Source/WebCore/page/DOMWindow.cpp 2020-01-17 18:22:38 UTC (rev 254752)
+++ trunk/Source/WebCore/page/DOMWindow.cpp 2020-01-17 18:45:34 UTC (rev 254753)
@@ -439,28 +439,6 @@
m_performance = nullptr;
}
-void DOMWindow::prewarmLocalStorageIfNecessary()
-{
- auto* page = this->page();
-
- // No need to prewarm for ephemeral sessions since the data is in memory only.
- if (!page || page->usesEphemeralSession())
- return;
-
- if (!page->mainFrame().mayPrewarmLocalStorage())
- return;
-
- auto localStorageResult = this->localStorage();
- if (localStorageResult.hasException())
- return;
-
- auto* localStorage = localStorageResult.returnValue();
- if (!localStorage)
- return;
-
- page->mainFrame().didPrewarmLocalStorage();
-}
-
DOMWindow::~DOMWindow()
{
if (m_suspendedForDocumentSuspension)
Modified: trunk/Source/WebCore/page/DOMWindow.h (254752 => 254753)
--- trunk/Source/WebCore/page/DOMWindow.h 2020-01-17 18:22:38 UTC (rev 254752)
+++ trunk/Source/WebCore/page/DOMWindow.h 2020-01-17 18:45:34 UTC (rev 254753)
@@ -203,8 +203,6 @@
void showModalDialog(const String& urlString, const String& dialogFeaturesString, DOMWindow& activeWindow, DOMWindow& firstWindow, const WTF::Function<void(DOMWindow&)>& prepareDialogFunction);
- void prewarmLocalStorageIfNecessary();
-
void alert(const String& message = emptyString());
bool confirmForBindings(const String& message);
String prompt(const String& message, const String& defaultValue);
Modified: trunk/Source/WebCore/page/Frame.cpp (254752 => 254753)
--- trunk/Source/WebCore/page/Frame.cpp 2020-01-17 18:22:38 UTC (rev 254752)
+++ trunk/Source/WebCore/page/Frame.cpp 2020-01-17 18:45:34 UTC (rev 254753)
@@ -117,9 +117,6 @@
DEFINE_DEBUG_ONLY_GLOBAL(WTF::RefCountedLeakCounter, frameCounter, ("Frame"));
-// We prewarm local storage for at most 5 origins in a given page.
-static const unsigned maxlocalStoragePrewarmingCount { 5 };
-
static inline Frame* parentFromOwnerElement(HTMLFrameOwnerElement* ownerElement)
{
if (!ownerElement)
@@ -988,19 +985,6 @@
tree().removeChild(*child);
}
-void Frame::didPrewarmLocalStorage()
-{
- ASSERT(isMainFrame());
- ASSERT(m_localStoragePrewarmingCount < maxlocalStoragePrewarmingCount);
- ++m_localStoragePrewarmingCount;
-}
-
-bool Frame::mayPrewarmLocalStorage() const
-{
- ASSERT(isMainFrame());
- return m_localStoragePrewarmingCount < maxlocalStoragePrewarmingCount;
-}
-
void Frame::selfOnlyRef()
{
ASSERT(isMainFrame());
Modified: trunk/Source/WebCore/page/Frame.h (254752 => 254753)
--- trunk/Source/WebCore/page/Frame.h 2020-01-17 18:22:38 UTC (rev 254752)
+++ trunk/Source/WebCore/page/Frame.h 2020-01-17 18:45:34 UTC (rev 254753)
@@ -291,9 +291,6 @@
WEBCORE_EXPORT bool isAlwaysOnLoggingAllowed() const;
- void didPrewarmLocalStorage();
- bool mayPrewarmLocalStorage() const;
-
// ========
void selfOnlyRef();
@@ -356,7 +353,6 @@
unsigned m_navigationDisableCount { 0 };
unsigned m_selfOnlyRefCount { 0 };
bool m_hasHadUserInteraction { false };
- unsigned m_localStoragePrewarmingCount { 0 };
protected:
UniqueRef<EventHandler> m_eventHandler;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes