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

Issue 10450030: Defer initializing the CopyTextureCHROMIUMResourceManager (Closed)

Created:
8 years, 7 months ago by apatrick_chromium
Modified:
8 years, 7 months ago
Reviewers:
greggman
CC:
chromium-reviews
Visibility:
Public.

Description

Defer initializing the CopyTextureCHROMIUMResourceManager It is takes 10s of milliseconds to initialize. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=139127

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -137 lines) Patch
M gpu/command_buffer/service/gles2_cmd_decoder.cc View 2 chunks +11 lines, -4 lines 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.h View 1 1 chunk +0 lines, -1 line 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc View 1 3 chunks +0 lines, -132 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
apatrick_chromium
8 years, 7 months ago (2012-05-25 00:09:29 UTC) #1
greggman
lgtm (thought did this not break any unit tests?)
8 years, 7 months ago (2012-05-25 05:17:48 UTC) #2
apatrick_chromium
On 2012/05/25 05:17:48, greggman wrote: > lgtm (thought did this not break any unit tests?) ...
8 years, 7 months ago (2012-05-25 18:46:42 UTC) #3
apatrick_chromium
Simpler than I expected. PTAL.
8 years, 7 months ago (2012-05-25 19:06:53 UTC) #4
greggman
lol, LGTM
8 years, 7 months ago (2012-05-25 19:11:57 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/apatrick@chromium.org/10450030/4002
8 years, 7 months ago (2012-05-25 19:14:46 UTC) #6
commit-bot: I haz the power
8 years, 7 months ago (2012-05-25 21:43:44 UTC) #7
Change committed as 139127

Powered by Google App Engine
This is Rietveld 408576698