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

Issue 12457031: cc: Changed total_pixels_rasterized to be a return value (Closed)

Created:
7 years, 9 months ago by vmpstr
Modified:
7 years, 9 months ago
Reviewers:
enne (OOO)
CC:
chromium-reviews, cc-bugs_chromium.org
Visibility:
Public.

Description

cc: Changed total_pixels_rasterized to be a return value It's a bit awkward calling Raster and passing an output parameter that is never used. Instead, it makes sense to make it a return value, and simply not capture it if its not required. BUG=223590 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=190569

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -17 lines) Patch
M cc/layers/picture_layer_impl_unittest.cc View 1 chunk +1 line, -3 lines 0 comments Download
M cc/resources/picture_pile_impl.h View 1 chunk +3 lines, -3 lines 0 comments Download
M cc/resources/picture_pile_impl.cc View 6 chunks +9 lines, -8 lines 0 comments Download
M cc/resources/tile_manager.cc View 1 chunk +2 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
vmpstr
Please take a look.
7 years, 9 months ago (2013-03-25 17:23:34 UTC) #1
enne (OOO)
lgtm
7 years, 9 months ago (2013-03-25 18:09:39 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vmpstr@chromium.org/12457031/1
7 years, 9 months ago (2013-03-25 18:12:18 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vmpstr@chromium.org/12457031/1
7 years, 9 months ago (2013-03-26 06:50:54 UTC) #4
commit-bot: I haz the power
7 years, 9 months ago (2013-03-26 07:20:26 UTC) #5
Message was sent while issue was closed.
Change committed as 190569

Powered by Google App Engine
This is Rietveld 408576698