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

Issue 11034006: Remove the CCOcclusionTracker include from CCRenderPass header file (Closed)

Created:
8 years, 2 months ago by danakj
Modified:
8 years, 2 months ago
Reviewers:
jamesr, enne (OOO)
CC:
chromium-reviews, cc-bugs_chromium.org, backer, piman, aelias_OOO_until_Jul13
Visibility:
Public.

Description

Remove the CCOcclusionTracker include from CCRenderPass header file The occlusion tracker still includes WebCore types so removing this header removes another source WebCore dependencies from CCRenderPass. Covered by existing tests; No change in behaviour. R=enne,jamesr BUG=152473 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=159548

Patch Set 1 #

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

Messages

Total messages: 8 (0 generated)
danakj
8 years, 2 months ago (2012-10-01 17:31:54 UTC) #1
enne (OOO)
lgtm
8 years, 2 months ago (2012-10-01 17:42:19 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/danakj@chromium.org/11034006/1
8 years, 2 months ago (2012-10-01 17:43:45 UTC) #3
commit-bot: I haz the power
Retried try job too often for step(s) crypto_unittests, unit_tests, cacheinvalidation_unittests, remoting_unittests, jingle_unittests, nacl_integration, ipc_tests, interactive_ui_tests, ...
8 years, 2 months ago (2012-10-01 17:47:26 UTC) #4
danakj
Something bad has happened. Applying the patch. Failed to apply patch for cc/CCRenderPass.h: While running ...
8 years, 2 months ago (2012-10-01 17:49:42 UTC) #5
danakj
On 2012/10/01 17:49:42, danakj wrote: > Something bad has happened. > > Applying the patch. ...
8 years, 2 months ago (2012-10-01 17:52:23 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/danakj@chromium.org/11034006/1
8 years, 2 months ago (2012-10-01 18:14:26 UTC) #7
commit-bot: I haz the power
8 years, 2 months ago (2012-10-01 20:57:57 UTC) #8
Change committed as 159548

Powered by Google App Engine
This is Rietveld 408576698