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

Side by Side Diff: ppapi/api/ppb_graphics_2d.idl

Issue 9731003: PPAPI: Remove out-dated or unnecessary TODOs from public files. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 * Use of this source code is governed by a BSD-style license that can be 2 * Use of this source code is governed by a BSD-style license that can be
3 * found in the LICENSE file. 3 * found in the LICENSE file.
4 */ 4 */
5 5
6 /** 6 /**
7 * Defines the <code>PPB_Graphics2D</code> struct representing a 2D graphics 7 * Defines the <code>PPB_Graphics2D</code> struct representing a 2D graphics
8 * context within the browser. 8 * context within the browser.
9 */ 9 */
10 10
(...skipping 221 matching lines...) Expand 10 before | Expand all | Expand 10 after
232 * the image has been painted on the screen. 232 * the image has been painted on the screen.
233 * 233 *
234 * @return Returns <code>PP_OK</code> on success or 234 * @return Returns <code>PP_OK</code> on success or
235 * <code>PP_ERROR_BADRESOURCE</code> if the graphics context is invalid, 235 * <code>PP_ERROR_BADRESOURCE</code> if the graphics context is invalid,
236 * <code>PP_ERROR_BADARGUMENT</code> if the callback is null and flush is 236 * <code>PP_ERROR_BADARGUMENT</code> if the callback is null and flush is
237 * being called from the main thread of the module, or 237 * being called from the main thread of the module, or
238 * <code>PP_ERROR_INPROGRESS</code> if a flush is already pending that has 238 * <code>PP_ERROR_INPROGRESS</code> if a flush is already pending that has
239 * not issued its callback yet. In the failure case, nothing will be updated 239 * not issued its callback yet. In the failure case, nothing will be updated
240 * and no callback will be scheduled. 240 * and no callback will be scheduled.
241 */ 241 */
242
243 /* TODO(darin): We should ensure that the completion callback always runs, so
244 * that it is easier for consumers to manage memory referenced by a callback.
dmichael (off chromium) 2012/03/19 17:41:52 polina's work for "required" callbacks fixed this
245 *
246 * TODO(): Add back in the synchronous mode description once we have support
247 * for it.
dmichael (off chromium) 2012/03/19 17:41:52 That's just part of planned work.
248 */
249 int32_t Flush( 242 int32_t Flush(
250 [in] PP_Resource graphics_2d, 243 [in] PP_Resource graphics_2d,
251 [in] PP_CompletionCallback callback); 244 [in] PP_CompletionCallback callback);
252 }; 245 };
253 246
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698