[SCM] WebKit Debian packaging branch, debian/experimental, updated. debian/1.3.8-1-142-g786665c
tonyg at chromium.org
tonyg at chromium.org
Mon Dec 27 16:28:40 UTC 2010
The following commit has been merged in the debian/experimental branch:
commit bf199588fa4a80e8dd062e45a599acd83796b1ce
Author: tonyg at chromium.org <tonyg at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Wed Dec 22 01:41:04 2010 +0000
2010-12-21 Sheriff Bot <webkit.review.bot at gmail.com>
Unreviewed, rolling out r74420.
http://trac.webkit.org/changeset/74420
https://bugs.webkit.org/show_bug.cgi?id=51436
causes editing/pasteboard/paste-noscript-xhtml.xhtml to crash
(Requested by tonyg-cr on #webkit).
* dom/DocumentParser.cpp:
* dom/DocumentParser.h:
* html/parser/HTMLDocumentParser.cpp:
* html/parser/HTMLDocumentParser.h:
* html/parser/HTMLParserScheduler.cpp:
(WebCore::HTMLParserScheduler::HTMLParserScheduler):
* html/parser/HTMLParserScheduler.h:
(WebCore::HTMLParserScheduler::isScheduledForResume):
* page/PageGroupLoadDeferrer.cpp:
(WebCore::PageGroupLoadDeferrer::PageGroupLoadDeferrer):
(WebCore::PageGroupLoadDeferrer::~PageGroupLoadDeferrer):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@74443 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index e43f601..7706fc1 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,24 @@
+2010-12-21 Sheriff Bot <webkit.review.bot at gmail.com>
+
+ Unreviewed, rolling out r74420.
+ http://trac.webkit.org/changeset/74420
+ https://bugs.webkit.org/show_bug.cgi?id=51436
+
+ causes editing/pasteboard/paste-noscript-xhtml.xhtml to crash
+ (Requested by tonyg-cr on #webkit).
+
+ * dom/DocumentParser.cpp:
+ * dom/DocumentParser.h:
+ * html/parser/HTMLDocumentParser.cpp:
+ * html/parser/HTMLDocumentParser.h:
+ * html/parser/HTMLParserScheduler.cpp:
+ (WebCore::HTMLParserScheduler::HTMLParserScheduler):
+ * html/parser/HTMLParserScheduler.h:
+ (WebCore::HTMLParserScheduler::isScheduledForResume):
+ * page/PageGroupLoadDeferrer.cpp:
+ (WebCore::PageGroupLoadDeferrer::PageGroupLoadDeferrer):
+ (WebCore::PageGroupLoadDeferrer::~PageGroupLoadDeferrer):
+
2010-12-21 Kenneth Russell <kbr at google.com>
Reviewed by Darin Fisher.
diff --git a/WebCore/dom/DocumentParser.cpp b/WebCore/dom/DocumentParser.cpp
index f4eebcc..5f8e04f 100644
--- a/WebCore/dom/DocumentParser.cpp
+++ b/WebCore/dom/DocumentParser.cpp
@@ -68,13 +68,5 @@ void DocumentParser::detach()
m_document = 0;
}
-void DocumentParser::suspendParsing()
-{
-}
-
-void DocumentParser::resumeParsing()
-{
-}
-
};
diff --git a/WebCore/dom/DocumentParser.h b/WebCore/dom/DocumentParser.h
index bedae8d..aa18a64 100644
--- a/WebCore/dom/DocumentParser.h
+++ b/WebCore/dom/DocumentParser.h
@@ -91,10 +91,6 @@ public:
void setDocumentWasLoadedAsPartOfNavigation() { m_documentWasLoadedAsPartOfNavigation = true; }
bool documentWasLoadedAsPartOfNavigation() const { return m_documentWasLoadedAsPartOfNavigation; }
- // FIXME: The names are not very accurate :(
- virtual void suspendParsing();
- virtual void resumeParsing();
-
protected:
DocumentParser(Document*);
diff --git a/WebCore/html/parser/HTMLDocumentParser.cpp b/WebCore/html/parser/HTMLDocumentParser.cpp
index 9b9780c..c32f9db 100644
--- a/WebCore/html/parser/HTMLDocumentParser.cpp
+++ b/WebCore/html/parser/HTMLDocumentParser.cpp
@@ -519,14 +519,4 @@ bool HTMLDocumentParser::usePreHTML5ParserQuirks(Document* document)
return document->settings() && document->settings()->usePreHTML5ParserQuirks();
}
-void HTMLDocumentParser::suspendParsing()
-{
- m_parserScheduler->suspend();
-}
-
-void HTMLDocumentParser::resumeParsing()
-{
- m_parserScheduler->resume();
-}
-
}
diff --git a/WebCore/html/parser/HTMLDocumentParser.h b/WebCore/html/parser/HTMLDocumentParser.h
index 27bb11b..05053df 100644
--- a/WebCore/html/parser/HTMLDocumentParser.h
+++ b/WebCore/html/parser/HTMLDocumentParser.h
@@ -93,8 +93,6 @@ private:
virtual void executeScriptsWaitingForStylesheets();
virtual int lineNumber() const;
virtual TextPosition0 textPosition() const;
- virtual void suspendParsing();
- virtual void resumeParsing();
// HTMLScriptRunnerHost
virtual void watchForLoad(CachedResource*);
diff --git a/WebCore/html/parser/HTMLParserScheduler.cpp b/WebCore/html/parser/HTMLParserScheduler.cpp
index 56db1aa..6e67697 100644
--- a/WebCore/html/parser/HTMLParserScheduler.cpp
+++ b/WebCore/html/parser/HTMLParserScheduler.cpp
@@ -65,7 +65,6 @@ HTMLParserScheduler::HTMLParserScheduler(HTMLDocumentParser* parser)
, m_parserTimeLimit(parserTimeLimit(m_parser->document()->page()))
, m_parserChunkSize(parserChunkSize(m_parser->document()->page()))
, m_continueNextChunkTimer(this, &HTMLParserScheduler::continueNextChunkTimerFired)
- , m_isSuspendedWithActiveTimer(false)
{
}
@@ -93,22 +92,4 @@ void HTMLParserScheduler::continueNextChunkTimerFired(Timer<HTMLParserScheduler>
m_parser->resumeParsingAfterYield();
}
-void HTMLParserScheduler::suspend()
-{
- ASSERT(!m_isSuspendedWithActiveTimer);
- if (!m_continueNextChunkTimer.isActive())
- return;
- m_isSuspendedWithActiveTimer = true;
- m_continueNextChunkTimer.stop();
-}
-
-void HTMLParserScheduler::resume()
-{
- ASSERT(!m_continueNextChunkTimer.isActive());
- if (!m_isSuspendedWithActiveTimer)
- return;
- m_isSuspendedWithActiveTimer = false;
- m_continueNextChunkTimer.startOneShot(0);
-}
-
}
diff --git a/WebCore/html/parser/HTMLParserScheduler.h b/WebCore/html/parser/HTMLParserScheduler.h
index 3a20b2b..5be33b0 100644
--- a/WebCore/html/parser/HTMLParserScheduler.h
+++ b/WebCore/html/parser/HTMLParserScheduler.h
@@ -71,10 +71,7 @@ public:
return true;
}
- bool isScheduledForResume() const { return m_isSuspendedWithActiveTimer || m_continueNextChunkTimer.isActive(); }
-
- void suspend();
- void resume();
+ bool isScheduledForResume() const { return m_continueNextChunkTimer.isActive(); }
private:
HTMLParserScheduler(HTMLDocumentParser*);
@@ -86,7 +83,6 @@ private:
double m_parserTimeLimit;
int m_parserChunkSize;
Timer<HTMLParserScheduler> m_continueNextChunkTimer;
- bool m_isSuspendedWithActiveTimer;
};
}
diff --git a/WebCore/page/PageGroupLoadDeferrer.cpp b/WebCore/page/PageGroupLoadDeferrer.cpp
index 680c52f..98a4720 100644
--- a/WebCore/page/PageGroupLoadDeferrer.cpp
+++ b/WebCore/page/PageGroupLoadDeferrer.cpp
@@ -22,7 +22,6 @@
#include "PageGroupLoadDeferrer.h"
#include "AsyncScriptRunner.h"
-#include "DocumentParser.h"
#include "Frame.h"
#include "Page.h"
#include "PageGroup.h"
@@ -50,8 +49,6 @@ PageGroupLoadDeferrer::PageGroupLoadDeferrer(Page* page, bool deferSelf)
for (Frame* frame = otherPage->mainFrame(); frame; frame = frame->tree()->traverseNext()) {
frame->document()->suspendActiveDOMObjects(ActiveDOMObject::WillShowDialog);
frame->document()->asyncScriptRunner()->suspend();
- if (DocumentParser* parser = frame->document()->parser())
- parser->suspendParsing();
}
}
}
@@ -72,8 +69,6 @@ PageGroupLoadDeferrer::~PageGroupLoadDeferrer()
for (Frame* frame = page->mainFrame(); frame; frame = frame->tree()->traverseNext()) {
frame->document()->resumeActiveDOMObjects();
frame->document()->asyncScriptRunner()->resume();
- if (DocumentParser* parser = frame->document()->parser())
- parser->resumeParsing();
}
}
}
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list