Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(130)

Issue 453533002: cc: Remove unused arguments from CalculateContentsScale (Closed)

Created:
6 years, 4 months ago by ajuma
Modified:
6 years, 4 months ago
Reviewers:
enne (OOO)
CC:
cc-bugs_chromium.org, chromium-reviews
Project:
chromium
Visibility:
Public.

Description

cc: Remove unused arguments from CalculateContentsScale Layer::CalculateContentsScale and its overrides never use device_scale_factor, page_scale_factor, maximum_animation_contents_scale, or animating_transform_to_screen. This CL removes these arguments. BUG=None Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288197

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -149 lines) Patch
M cc/layers/contents_scaling_layer.h View 1 chunk +0 lines, -4 lines 0 comments Download
M cc/layers/contents_scaling_layer.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M cc/layers/image_layer.h View 1 chunk +0 lines, -4 lines 0 comments Download
M cc/layers/image_layer.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M cc/layers/layer.h View 1 chunk +0 lines, -4 lines 0 comments Download
M cc/layers/layer.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M cc/layers/painted_scrollbar_layer.h View 1 chunk +0 lines, -4 lines 0 comments Download
M cc/layers/painted_scrollbar_layer.cc View 1 chunk +0 lines, -8 lines 0 comments Download
M cc/test/tiled_layer_test_common.h View 1 chunk +0 lines, -4 lines 0 comments Download
M cc/test/tiled_layer_test_common.cc View 2 chunks +0 lines, -8 lines 0 comments Download
M cc/trees/layer_tree_host_common.cc View 7 chunks +5 lines, -41 lines 0 comments Download
M cc/trees/layer_tree_host_common_unittest.cc View 7 chunks +12 lines, -52 lines 0 comments Download
M cc/trees/layer_tree_host_unittest.cc View 1 chunk +0 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
ajuma
6 years, 4 months ago (2014-08-07 19:48:22 UTC) #1
enne (OOO)
lgtm
6 years, 4 months ago (2014-08-07 19:58:56 UTC) #2
enne (OOO)
lgtm
6 years, 4 months ago (2014-08-07 19:58:59 UTC) #3
ajuma
The CQ bit was checked by ajuma@chromium.org
6 years, 4 months ago (2014-08-07 20:01:16 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ajuma@chromium.org/453533002/1
6 years, 4 months ago (2014-08-07 20:06:35 UTC) #5
commit-bot: I haz the power
6 years, 4 months ago (2014-08-08 01:36:26 UTC) #6
Message was sent while issue was closed.
Change committed as 288197

Powered by Google App Engine
This is Rietveld 408576698