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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 7399016: More trivial cleanupi of ppapi/c headers (Closed)

Created:
9 years, 5 months ago by noelallen1
Modified:
9 years, 5 months ago
CC:
chromium-reviews, piman+watch_chromium.org
Visibility:
Public.

Description

More trivial cleanupi of ppapi/c headers These headers were autogenerated from ppapi/api/*.idl BUG= http://code.google.com/p/chromium/issues/detail?id=76271^M TEST= run try Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=92931

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 (+376 lines, -476 lines) Patch
M ppapi/c/pp_bool.h View 1 2 3 3 chunks +5 lines, -2 lines 0 comments Download
M ppapi/c/pp_completion_callback.h View 1 2 3 6 chunks +9 lines, -9 lines 0 comments Download
M ppapi/c/pp_errors.h View 1 2 3 3 chunks +8 lines, -22 lines 0 comments Download
M ppapi/c/pp_file_info.h View 1 2 3 4 chunks +5 lines, -6 lines 0 comments Download
M ppapi/c/pp_input_event.h View 1 2 3 13 chunks +11 lines, -65 lines 0 comments Download
M ppapi/c/pp_instance.h View 1 2 3 1 chunk +7 lines, -4 lines 0 comments Download
M ppapi/c/pp_macros.h View 1 2 3 1 chunk +10 lines, -2 lines 0 comments Download
M ppapi/c/pp_module.h View 1 2 3 1 chunk +7 lines, -5 lines 0 comments Download
M ppapi/c/pp_point.h View 1 2 3 2 chunks +6 lines, -5 lines 0 comments Download
M ppapi/c/pp_rect.h View 1 2 3 3 chunks +10 lines, -7 lines 0 comments Download
M ppapi/c/pp_resource.h View 1 2 3 2 chunks +4 lines, -1 line 0 comments Download
M ppapi/c/pp_size.h View 1 2 3 1 chunk +6 lines, -3 lines 0 comments Download
M ppapi/c/pp_time.h View 1 2 3 2 chunks +6 lines, -8 lines 0 comments Download
M ppapi/c/pp_var.h View 1 2 3 8 chunks +4 lines, -25 lines 0 comments Download
M ppapi/c/ppb.h View 1 2 3 2 chunks +8 lines, -1 line 0 comments Download
M ppapi/c/ppb_audio.h View 1 2 3 5 chunks +9 lines, -8 lines 0 comments Download
M ppapi/c/ppb_audio_config.h View 1 2 3 8 chunks +11 lines, -27 lines 0 comments Download
M ppapi/c/ppb_core.h View 1 2 3 7 chunks +13 lines, -16 lines 0 comments Download
M ppapi/c/ppb_file_io.h View 1 2 3 12 chunks +14 lines, -23 lines 0 comments Download
M ppapi/c/ppb_file_ref.h View 1 2 3 11 chunks +10 lines, -11 lines 0 comments Download
M ppapi/c/ppb_file_system.h View 1 2 3 5 chunks +11 lines, -11 lines 0 comments Download
M ppapi/c/ppb_graphics_2d.h View 1 2 3 8 chunks +11 lines, -16 lines 0 comments Download
M ppapi/c/ppb_image_data.h View 1 2 3 11 chunks +10 lines, -19 lines 0 comments Download
M ppapi/c/ppb_input_event.h View 1 2 3 25 chunks +57 lines, -73 lines 0 comments Download
M ppapi/c/ppb_instance.h View 1 2 3 3 chunks +13 lines, -12 lines 0 comments Download
M ppapi/c/ppb_messaging.h View 1 2 3 3 chunks +12 lines, -5 lines 0 comments Download
M ppapi/c/ppb_url_loader.h View 1 2 3 10 chunks +13 lines, -16 lines 0 comments Download
M ppapi/c/ppb_url_request_info.h View 1 2 3 17 chunks +8 lines, -23 lines 0 comments Download
M ppapi/c/ppb_url_response_info.h View 1 2 3 10 chunks +9 lines, -16 lines 0 comments Download
M ppapi/c/ppb_var.h View 1 2 3 5 chunks +8 lines, -8 lines 0 comments Download
M ppapi/c/ppp.h View 1 2 3 3 chunks +14 lines, -7 lines 0 comments Download
M ppapi/c/ppp_input_event.h View 1 2 3 2 chunks +18 lines, -0 lines 0 comments Download
M ppapi/c/ppp_instance.h View 1 2 3 8 chunks +16 lines, -13 lines 0 comments Download
M ppapi/c/ppp_messaging.h View 1 2 3 3 chunks +13 lines, -7 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
noelallen1
9 years, 5 months ago (2011-07-16 23:35:08 UTC) #1
dmichael (off chromium)
The IDL generator seems to be sticking a few odd carriage returns in that the ...
9 years, 5 months ago (2011-07-17 02:03:18 UTC) #2
noelallen1
9 years, 5 months ago (2011-07-18 19:17:15 UTC) #3
dmichael (off chromium)
LGTM
9 years, 5 months ago (2011-07-18 19:21:16 UTC) #4
dmichael (off chromium)
Sorry, I missed that PPB_Instance got scripting functions back from IDL. Please fix that and ...
9 years, 5 months ago (2011-07-18 19:51:01 UTC) #5
noelallen1
9 years, 5 months ago (2011-07-18 20:14:44 UTC) #6
noelallen1
9 years, 5 months ago (2011-07-18 21:12:26 UTC) #7
dmichael (off chromium)
9 years, 5 months ago (2011-07-18 21:16:38 UTC) #8 
LGTM with green trybots

Powered by Google App Engine
This is Rietveld 408576698