Title: [163647] trunk/Source
- Revision
- 163647
- Author
- barraclo...@apple.com
- Date
- 2014-02-07 14:27:06 -0800 (Fri, 07 Feb 2014)
Log Message
Should report user input to PageThrottler
https://bugs.webkit.org/show_bug.cgi?id=128398
Reviewed by Tim Horton.
Make sure we wake from AppNap if there is user interaction.
Source/WebCore:
* page/PageThrottler.h:
(WebCore::PageThrottler::didReceiveUserInput):
(WebCore::PageThrottler::pluginDidEvaluate):
- added, these call reportInterestingEvent()
Source/WebKit2:
* WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp:
(WebKit::NPRuntimeObjectMap::evaluate):
- reportInterestingEvent() -> pluginDidEvaluate()
* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::mouseEvent):
(WebKit::WebPage::wheelEvent):
(WebKit::WebPage::keyEvent):
- call didReceiveUserInput()
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (163646 => 163647)
--- trunk/Source/WebCore/ChangeLog 2014-02-07 22:12:03 UTC (rev 163646)
+++ trunk/Source/WebCore/ChangeLog 2014-02-07 22:27:06 UTC (rev 163647)
@@ -1,3 +1,17 @@
+2014-02-07 Gavin Barraclough <barraclo...@apple.com>
+
+ Should report user input to PageThrottler
+ https://bugs.webkit.org/show_bug.cgi?id=128398
+
+ Reviewed by Tim Horton.
+
+ Make sure we wake from AppNap if there is user interaction.
+
+ * page/PageThrottler.h:
+ (WebCore::PageThrottler::didReceiveUserInput):
+ (WebCore::PageThrottler::pluginDidEvaluate):
+ - added, these call reportInterestingEvent()
+
2014-02-07 Jer Noble <jer.no...@apple.com>
Unreviewed build fix for 32-bit iOS.
Modified: trunk/Source/WebCore/page/PageThrottler.h (163646 => 163647)
--- trunk/Source/WebCore/page/PageThrottler.h 2014-02-07 22:12:03 UTC (rev 163646)
+++ trunk/Source/WebCore/page/PageThrottler.h 2014-02-07 22:27:06 UTC (rev 163647)
@@ -50,7 +50,8 @@
void setIsVisuallyIdle(bool);
- void reportInterestingEvent();
+ void didReceiveUserInput() { reportInterestingEvent(); }
+ void pluginDidEvaluate() { reportInterestingEvent(); }
private:
enum PageThrottleState {
@@ -63,6 +64,8 @@
void addActivityToken(PageActivityAssertionToken&);
void removeActivityToken(PageActivityAssertionToken&);
+ void reportInterestingEvent();
+
void startThrottleHysteresisTimer();
void stopThrottleHysteresisTimer();
void throttleHysteresisTimerFired(Timer<PageThrottler>&);
Modified: trunk/Source/WebKit2/ChangeLog (163646 => 163647)
--- trunk/Source/WebKit2/ChangeLog 2014-02-07 22:12:03 UTC (rev 163646)
+++ trunk/Source/WebKit2/ChangeLog 2014-02-07 22:27:06 UTC (rev 163647)
@@ -1,3 +1,21 @@
+2014-02-07 Gavin Barraclough <barraclo...@apple.com>
+
+ Should report user input to PageThrottler
+ https://bugs.webkit.org/show_bug.cgi?id=128398
+
+ Reviewed by Tim Horton.
+
+ Make sure we wake from AppNap if there is user interaction.
+
+ * WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp:
+ (WebKit::NPRuntimeObjectMap::evaluate):
+ - reportInterestingEvent() -> pluginDidEvaluate()
+ * WebProcess/WebPage/WebPage.cpp:
+ (WebKit::WebPage::mouseEvent):
+ (WebKit::WebPage::wheelEvent):
+ (WebKit::WebPage::keyEvent):
+ - call didReceiveUserInput()
+
2014-02-07 Dean Jackson <d...@apple.com>
WebGL doesn't update with remotely hosted layers
@@ -60,7 +78,7 @@
Reviewed by Tim Horton.
* Shared/WebPreferencesStore.h:
- - pageVisibilityBasedProcessSuppressionEnabled shoud default to true.
+ - pageVisibilityBasedProcessSuppressionEnabled should default to true.
2014-02-07 Tim Horton <timothy_hor...@apple.com>
Modified: trunk/Source/WebKit2/WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp (163646 => 163647)
--- trunk/Source/WebKit2/WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp 2014-02-07 22:12:03 UTC (rev 163646)
+++ trunk/Source/WebKit2/WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp 2014-02-07 22:27:06 UTC (rev 163647)
@@ -191,7 +191,7 @@
if (m_pluginView && !m_pluginView->isBeingDestroyed()) {
if (Page* page = m_pluginView->frame()->page())
- page->pageThrottler().reportInterestingEvent();
+ page->pageThrottler().pluginDidEvaluate();
}
ExecState* exec = globalObject->globalExec();
Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp (163646 => 163647)
--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp 2014-02-07 22:12:03 UTC (rev 163646)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp 2014-02-07 22:27:06 UTC (rev 163647)
@@ -118,6 +118,7 @@
#include <WebCore/MainFrame.h>
#include <WebCore/MouseEvent.h>
#include <WebCore/Page.h>
+#include <WebCore/PageThrottler.h>
#include <WebCore/PlatformKeyboardEvent.h>
#include <WebCore/PluginDocument.h>
#include <WebCore/PrintContext.h>
@@ -1665,6 +1666,8 @@
void WebPage::mouseEvent(const WebMouseEvent& mouseEvent)
{
+ m_page->pageThrottler().didReceiveUserInput();
+
#if ENABLE(CONTEXT_MENUS)
// Don't try to handle any pending mouse events if a context menu is showing.
if (m_isShowingContextMenu) {
@@ -1745,6 +1748,8 @@
void WebPage::wheelEvent(const WebWheelEvent& wheelEvent)
{
+ m_page->pageThrottler().didReceiveUserInput();
+
bool handled = false;
if (canHandleUserEvents()) {
@@ -1774,6 +1779,8 @@
void WebPage::keyEvent(const WebKeyboardEvent& keyboardEvent)
{
+ m_page->pageThrottler().didReceiveUserInput();
+
bool handled = false;
if (canHandleUserEvents()) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes