Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(46)

Issue 10391049: Merge 116319 - Unreviewed, rolling out r115525. (Closed)

Created:
8 years, 4 months ago by karen
Modified:
8 years, 4 months ago
Reviewers:
nduca
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1132/
Visibility:
Public.

Description

Merge 116319 - Unreviewed, rolling out r115525. http://trac.webkit.org/changeset/115525 https://bugs.webkit.org/show_bug.cgi?id=66683 Too many pages rely on DOMTimeStamp as first argument. Reverting while we consider next steps. Source/WebCore: * WebCore.exp.in: * bindings/js/JSRequestAnimationFrameCallbackCustom.cpp: (WebCore::JSRequestAnimationFrameCallback::handleEvent): * dom/Document.cpp: (WebCore::Document::serviceScriptedAnimations): * dom/Document.h: (Document): * dom/RequestAnimationFrameCallback.h: (RequestAnimationFrameCallback): * dom/RequestAnimationFrameCallback.idl: * dom/ScriptedAnimationController.cpp: (WebCore::ScriptedAnimationController::ScriptedAnimationController): (WebCore::ScriptedAnimationController::serviceScriptedAnimations): (WebCore): (WebCore::ScriptedAnimationController::windowScreenDidChange): (WebCore::ScriptedAnimationController::scheduleAnimation): (WebCore::ScriptedAnimationController::animationTimerFired): * dom/ScriptedAnimationController.h: (ScriptedAnimationController): (WebCore::ScriptedAnimationController::displayRefreshFired): * page/FrameView.cpp: (WebCore::FrameView::serviceScriptedAnimations): * page/FrameView.h: (FrameView): * platform/graphics/DisplayRefreshMonitor.cpp: (WebCore::DisplayRefreshMonitor::DisplayRefreshMonitor): (WebCore::DisplayRefreshMonitor::notifyClients): * platform/graphics/DisplayRefreshMonitor.h: (DisplayRefreshMonitor): * platform/graphics/blackberry/DisplayRefreshMonitorBlackBerry.cpp: (WebCore::DisplayRefreshMonitor::displayLinkFired): * platform/graphics/mac/DisplayRefreshMonitorMac.cpp: (WebCore): (WebCore::DisplayRefreshMonitor::requestRefreshCallback): (WebCore::DisplayRefreshMonitor::displayLinkFired): Source/WebKit/chromium: * src/PageWidgetDelegate.cpp: (WebKit::PageWidgetDelegate::animate): * src/WebViewImpl.cpp: (WebKit::WebViewImpl::updateAnimations): TBR=nduca@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=116643

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -58 lines) Patch
M Source/WebCore/WebCore.exp.in View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/bindings/js/JSRequestAnimationFrameCallbackCustom.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/WebCore/dom/Document.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/dom/Document.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/WebCore/dom/RequestAnimationFrameCallback.h View 3 chunks +3 lines, -1 line 0 comments Download
M Source/WebCore/dom/RequestAnimationFrameCallback.idl View 1 chunk +3 lines, -5 lines 0 comments Download
M Source/WebCore/dom/ScriptedAnimationController.h View 4 chunks +5 lines, -4 lines 0 comments Download
M Source/WebCore/dom/ScriptedAnimationController.cpp View 7 chunks +11 lines, -21 lines 0 comments Download
M Source/WebCore/page/FrameView.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/page/FrameView.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/WebCore/platform/graphics/DisplayRefreshMonitor.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/platform/graphics/DisplayRefreshMonitor.cpp View 2 chunks +4 lines, -4 lines 0 comments Download
M Source/WebCore/platform/graphics/blackberry/DisplayRefreshMonitorBlackBerry.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/platform/graphics/mac/DisplayRefreshMonitorMac.cpp View 4 chunks +6 lines, -8 lines 0 comments Download
M Source/WebKit/chromium/src/PageWidgetDelegate.cpp View 1 chunk +2 lines, -1 line 0 comments Download
M Source/WebKit/chromium/src/WebViewImpl.cpp View 2 chunks +7 lines, -3 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
karen
8 years, 4 months ago (2012-05-10 14:21:34 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698