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

Issue 11150024: cc: Remove wtf hashmap use from resource provider (Closed)

Created:
8 years, 2 months ago by enne (OOO)
Modified:
8 years, 2 months ago
Reviewers:
jamesr
CC:
chromium-reviews, feature-media-reviews_chromium.org, cc-bugs_chromium.org, tfarina
Visibility:
Public.

Description

cc: Remove wtf hashmap use from resource provider BUG=154451 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=162075

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -101 lines) Patch
M cc/resource_provider.h View 5 chunks +4 lines, -4 lines 0 comments Download
M cc/resource_provider.cc View 24 chunks +14 lines, -94 lines 0 comments Download
M cc/resource_provider_unittest.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M cc/video_layer_impl.h View 1 chunk +1 line, -0 lines 2 comments Download

Messages

Total messages: 6 (0 generated)
enne (OOO)
https://codereview.chromium.org/11150024/diff/1/cc/video_layer_impl.h File cc/video_layer_impl.h (right): https://codereview.chromium.org/11150024/diff/1/cc/video_layer_impl.h#newcode13 cc/video_layer_impl.h:13: #include <wtf/ThreadingPrimitives.h> Sad to add this in, but the ...
8 years, 2 months ago (2012-10-15 21:08:45 UTC) #1
jamesr
lgtm https://codereview.chromium.org/11150024/diff/1/cc/video_layer_impl.h File cc/video_layer_impl.h (right): https://codereview.chromium.org/11150024/diff/1/cc/video_layer_impl.h#newcode13 cc/video_layer_impl.h:13: #include <wtf/ThreadingPrimitives.h> On 2012/10/15 21:08:46, enne wrote: > ...
8 years, 2 months ago (2012-10-15 21:11:44 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/enne@chromium.org/11150024/1
8 years, 2 months ago (2012-10-15 21:18:18 UTC) #3
commit-bot: I haz the power
Retried try job too often for step(s) browser_tests
8 years, 2 months ago (2012-10-16 01:21:16 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/enne@chromium.org/11150024/1
8 years, 2 months ago (2012-10-16 01:29:01 UTC) #5
commit-bot: I haz the power
8 years, 2 months ago (2012-10-16 04:31:27 UTC) #6
Change committed as 162075

Powered by Google App Engine
This is Rietveld 408576698