Chromium Code Reviews
Help | Chromium Project | Sign in
(77)

Issue 11363191: Cache more GL state both service and client side. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 5 months ago by greggman
Modified:
1 year, 5 months ago
CC:
chromium-reviews_chromium.org, apatrick_chromium
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Cache more GL state both service and client side.

BUG=160370
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=167613

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1493 lines, -1019 lines) Lint Patch
M gpu/command_buffer/build_gles2_cmd_buffer.py View 1 19 chunks +240 lines, -82 lines 0 comments ? errors Download
A gpu/command_buffer/client/client_context_state.h View 1 chunk +39 lines, -0 lines 0 comments 1 errors Download
A + gpu/command_buffer/client/client_context_state.cc View 1 chunk +13 lines, -10 lines 0 comments 0 errors Download
A gpu/command_buffer/client/client_context_state_autogen.h View 1 chunk +27 lines, -0 lines 0 comments ? errors Download
A gpu/command_buffer/client/client_context_state_impl_autogen.h View 1 1 chunk +123 lines, -0 lines 0 comments 0 errors Download
M gpu/command_buffer/client/gles2_implementation.h View 6 chunks +13 lines, -33 lines 0 comments 1 errors Download
M gpu/command_buffer/client/gles2_implementation.cc View 1 12 chunks +100 lines, -111 lines 0 comments ? errors Download
M gpu/command_buffer/client/gles2_implementation_impl_autogen.h View 6 chunks +15 lines, -16 lines 0 comments ? errors Download
M gpu/command_buffer/client/gles2_implementation_unittest.cc View 1 2 3 4 3 chunks +38 lines, -5 lines 0 comments ? errors Download
M gpu/command_buffer/client/gles2_implementation_unittest_autogen.h View 11 chunks +23 lines, -30 lines 0 comments ? errors Download
M gpu/command_buffer/service/context_state.h View 1 chunk +7 lines, -0 lines 0 comments 0 errors Download
M gpu/command_buffer/service/context_state_impl_autogen.h View 1 chunk +626 lines, -0 lines 0 comments ? errors Download
M gpu/command_buffer/service/gles2_cmd_decoder.cc View 1 2 7 chunks +9 lines, -8 lines 0 comments 0 errors Download
M gpu/command_buffer/service/gles2_cmd_decoder_autogen.h View 20 chunks +201 lines, -707 lines 0 comments 0 errors Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest.cc View 1 chunk +0 lines, -3 lines 0 comments ? errors Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest_1_autogen.h View 5 chunks +8 lines, -8 lines 0 comments 0 errors Download
M gpu/command_buffer/service/gles2_cmd_decoder_unittest_2_autogen.h View 2 chunks +6 lines, -6 lines 0 comments 0 errors Download
M gpu/gpu_common.gypi View 1 2 3 1 chunk +4 lines, -0 lines 0 comments ? errors Download
M ppapi/native_client/src/shared/ppapi_proxy/ppapi_proxy_untrusted.gyp View 1 chunk +1 line, -0 lines 0 comments ? errors Download
Commit:

Messages

Total messages: 10
greggman
Hey Al, PTAL Hey Brett, can you please bless the ppapi file change? Thank you
1 year, 5 months ago #1
brettw
ppapi/ lgtm, I didn't look at the rest.
1 year, 5 months ago #2
apatrick_chromium
lgtm
1 year, 5 months ago #3
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gman@chromium.org/11363191/2001
1 year, 5 months ago #4
I haz the power (commit-bot)
Retried try job too often for step(s) compile
1 year, 5 months ago #5
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gman@chromium.org/11363191/25
1 year, 5 months ago #6
I haz the power (commit-bot)
Retried try job too often for step(s) base_unittests, browser_tests, cacheinvalidation_unittests, check_deps, chrome_frame_net_tests, chrome_frame_unittests, content_browsertests, content_unittests, ...
1 year, 5 months ago #7
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gman@chromium.org/11363191/2009
1 year, 5 months ago #8
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gman@chromium.org/11363191/5008
1 year, 5 months ago #9
I haz the power (commit-bot)
1 year, 5 months ago #10
Change committed as 167613
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6