Chromium Code Reviews
[email protected] (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(326)

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 8879009: In ppapi/tests: |... const*| -> |const ...*| (for consistency). (Closed)

Created:
9 years ago by viettrungluu
Modified:
9 years ago
CC:
chromium-reviews
Visibility:
Public.

Description

In ppapi/tests: |... const*| -> |const ...*| (for consistency). Also fix a bunch of |reinterpret_cast|s (to |static_cast|). So we can stop cutting-and-pasting the same mistakes over and over. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113678

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -34 lines) Patch
M ppapi/tests/test_audio.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M ppapi/tests/test_audio_config.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_broker.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_buffer.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_char_set.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_crypto.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_cursor_control.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_graphics_2d.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M ppapi/tests/test_graphics_3d.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_image_data.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_input_event.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M ppapi/tests/test_tcp_socket_private_shared.cc View 1 chunk +2 lines, -4 lines 0 comments Download
M ppapi/tests/test_transport.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_udp_socket_private_shared.cc View 1 chunk +4 lines, -8 lines 0 comments Download
M ppapi/tests/test_uma.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_utils.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_var.cc View 1 chunk +1 line, -1 line 0 comments Download
M ppapi/tests/test_video_decoder.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
viettrungluu
Only one reviewer needed.
9 years ago (2011-12-08 07:19:09 UTC) #1
dmichael (off chromium)
LGTM, thanks!
9 years ago (2011-12-08 18:40:58 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/8879009/1
9 years ago (2011-12-08 19:09:53 UTC) #3
commit-bot: I haz the power
9 years ago (2011-12-08 22:31:23 UTC) #4 
Change committed as 113678

Powered by Google App Engine
This is Rietveld 408576698