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

Issue 810713002: Reduce tile memory size (Closed)

Created:
6 years ago by JungJik
Modified:
6 years ago
Reviewers:
reveman, vmpstr
CC:
chromium-reviews, cc-bugs_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reduce tile memory size One tile memory takes 160 bytes currently. This patch can reduce the tile memory by removing unnecessary variables and packing the booleans to 144 bytes. BUG=341919 Committed: https://crrev.com/0acd2d55393fcbc534b9ed2c61349e5543381658 Cr-Commit-Position: refs/heads/master@{#309504}

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -6 lines) Patch
M cc/resources/tile.h View 1 2 chunks +3 lines, -4 lines 0 comments Download
M cc/resources/tile.cc View 1 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 14 (3 generated)
JungJik
On 2014/12/16 09:54:16, JungJik wrote: > mailto:jungjik.lee@samsung.com changed reviewers: > + mailto:vmpstr@chromium.org Is the issue ...
6 years ago (2014-12-16 10:04:45 UTC) #2
vmpstr
Can you rebase on tip of tree, since some of those variables are being used, ...
6 years ago (2014-12-16 17:40:57 UTC) #3
JungJik
On 2014/12/16 17:40:57, vmpstr wrote: > Can you rebase on tip of tree, since some ...
6 years ago (2014-12-17 11:21:45 UTC) #4
vmpstr
On 2014/12/17 11:21:45, JungJik wrote: > On 2014/12/16 17:40:57, vmpstr wrote: > > Can you ...
6 years ago (2014-12-17 17:38:05 UTC) #5
vmpstr
+reveman, wdyt about merging mts and tile?
6 years ago (2014-12-17 17:39:16 UTC) #7
reveman
On 2014/12/17 17:39:16, vmpstr wrote: > +reveman, wdyt about merging mts and tile? Sgtm if ...
6 years ago (2014-12-17 18:46:00 UTC) #8
JungJik
On 2014/12/17 18:46:00, reveman wrote: > On 2014/12/17 17:39:16, vmpstr wrote: > > +reveman, wdyt ...
6 years ago (2014-12-22 11:12:35 UTC) #9
vmpstr
lgtm
6 years ago (2014-12-22 15:43:35 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/810713002/20001
6 years ago (2014-12-22 23:55:26 UTC) #12
commit-bot: I haz the power
Committed patchset #2 (id:20001)
6 years ago (2014-12-23 00:49:45 UTC) #13
commit-bot: I haz the power
6 years ago (2014-12-23 00:50:52 UTC) #14
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/0acd2d55393fcbc534b9ed2c61349e5543381658
Cr-Commit-Position: refs/heads/master@{#309504}

Powered by Google App Engine
This is Rietveld 408576698