Title: [132550] trunk/Source/WebCore
- Revision
- 132550
- Author
- [email protected]
- Date
- 2012-10-25 18:41:44 -0700 (Thu, 25 Oct 2012)
Log Message
Remove feed and feeds URL scheme workarounds
https://bugs.webkit.org/show_bug.cgi?id=100442
Reviewed by Dan Bernstein.
* dom/DocumentEventQueue.cpp:
(WebCore::DocumentEventQueue::enqueueOrDispatchScrollEvent):
* page/mac/EventHandlerMac.mm:
(WebCore::EventHandler::needsKeyboardEventDisambiguationQuirks):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (132549 => 132550)
--- trunk/Source/WebCore/ChangeLog 2012-10-26 01:20:57 UTC (rev 132549)
+++ trunk/Source/WebCore/ChangeLog 2012-10-26 01:41:44 UTC (rev 132550)
@@ -1,3 +1,15 @@
+2012-10-25 Anders Carlsson <[email protected]>
+
+ Remove feed and feeds URL scheme workarounds
+ https://bugs.webkit.org/show_bug.cgi?id=100442
+
+ Reviewed by Dan Bernstein.
+
+ * dom/DocumentEventQueue.cpp:
+ (WebCore::DocumentEventQueue::enqueueOrDispatchScrollEvent):
+ * page/mac/EventHandlerMac.mm:
+ (WebCore::EventHandler::needsKeyboardEventDisambiguationQuirks):
+
2012-10-25 Sheriff Bot <[email protected]>
Unreviewed, rolling out r132514.
Modified: trunk/Source/WebCore/dom/DocumentEventQueue.cpp (132549 => 132550)
--- trunk/Source/WebCore/dom/DocumentEventQueue.cpp 2012-10-26 01:20:57 UTC (rev 132549)
+++ trunk/Source/WebCore/dom/DocumentEventQueue.cpp 2012-10-26 01:41:44 UTC (rev 132550)
@@ -40,12 +40,6 @@
namespace WebCore {
-static inline bool shouldDispatchScrollEventSynchronously(Document* document)
-{
- ASSERT_ARG(document, document);
- return applicationIsSafari() && (document->url().protocolIs("feed") || document->url().protocolIs("feeds"));
-}
-
class DocumentEventQueueTimer : public SuspendableTimer {
WTF_MAKE_NONCOPYABLE(DocumentEventQueueTimer);
public:
@@ -98,11 +92,6 @@
bool canBubble = targetType == ScrollEventDocumentTarget;
RefPtr<Event> scrollEvent = Event::create(eventNames().scrollEvent, canBubble, false /* non cancelleable */);
- if (shouldDispatchScrollEventSynchronously(target->document())) {
- target->dispatchEvent(scrollEvent.release());
- return;
- }
-
if (!m_nodesWithQueuedScrollEvents.add(target.get()).isNewEntry)
return;
Modified: trunk/Source/WebCore/page/mac/EventHandlerMac.mm (132549 => 132550)
--- trunk/Source/WebCore/page/mac/EventHandlerMac.mm 2012-10-26 01:20:57 UTC (rev 132549)
+++ trunk/Source/WebCore/page/mac/EventHandlerMac.mm 2012-10-26 01:41:44 UTC (rev 132550)
@@ -711,9 +711,6 @@
{
Document* document = m_frame->document();
- // RSS view needs arrow key keypress events.
- if (applicationIsSafari() && (document->url().protocolIs("feed") || document->url().protocolIs("feeds")))
- return true;
Settings* settings = m_frame->settings();
if (!settings)
return false;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
