Issue 401923003: Revert Removing base::TimeTicks argument to DrawLayers

Issue 401923003: Revert Removing base::TimeTicks argument to DrawLayers (Closed)

Created:
6 years, 5 months ago by ccameron

Modified:
6 years, 5 months ago

Reviewers:

CC:
chromium-reviews, cc-bugs_chromium.org

Base URL:
https://chromium.googlesource.com/chromium/src.git@master

Project:
chromium

Visibility:
Public.

Revert Removing base::TimeTicks argument to DrawLayers BUG=394901 TBR=danakj@chromium.org,brianderson@chromium.org,mithro@mithis.com This makes the framerate erratic and YouTube janky. > Removing base::TimeTicks argument to DrawLayers as it isn't really used. > > R=skyostil,brianderson > BUG=346230 > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282456 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284316

Created: 6 years, 5 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+108 lines, -103 lines)			Patch
M	cc/layers/delegated_renderer_layer_impl_unittest.cc	View		26 chunks	+26 lines, -26 lines	0 comments	Download
M	cc/test/layer_tree_test.cc	View	1	1 chunk	+3 lines, -2 lines	0 comments	Download
M	cc/trees/layer_tree_host_impl.h	View		1 chunk	+1 line, -1 line	0 comments	Download
M	cc/trees/layer_tree_host_impl.cc	View	1	2 chunks	+3 lines, -2 lines	0 comments	Download
M	cc/trees/layer_tree_host_impl_unittest.cc	View	1	68 chunks	+68 lines, -68 lines	0 comments	Download
M	cc/trees/single_thread_proxy.h	View	1	1 chunk	+2 lines, -1 line	0 comments	Download
M	cc/trees/single_thread_proxy.cc	View	1	3 chunks	+3 lines, -2 lines	0 comments	Download
M	cc/trees/thread_proxy.cc	View	1	1 chunk	+2 lines, -1 line	0 comments	Download

Total messages: 6 (0 generated)

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ccameron@chromium.org/401923003/1

6 years, 5 months ago (2014-07-18 22:40:35 UTC) #2

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ccameron@chromium.org/401923003/20001

6 years, 5 months ago (2014-07-18 23:03:30 UTC) #4

commit-bot: I haz the power

FYI, CQ is re-trying this CL (attempt #1). The failing builders are: win_chromium_rel on tryserver.chromium ...

6 years, 5 months ago (2014-07-19 03:02:29 UTC) #5

Message was sent while issue was closed.

Change committed as 284316