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

Unified Diff: webkit/plugins/ppapi/ppb_graphics_2d_impl.cc

Issue 10836225: Remove a log statement in PPAPI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/plugins/ppapi/ppb_graphics_2d_impl.cc
diff --git a/webkit/plugins/ppapi/ppb_graphics_2d_impl.cc b/webkit/plugins/ppapi/ppb_graphics_2d_impl.cc
index c1d3829638da60734a5eb7688408b301cd452d7a..40a88dff63e451195f43dc38ad7223a57b76985b 100644
--- a/webkit/plugins/ppapi/ppb_graphics_2d_impl.cc
+++ b/webkit/plugins/ppapi/ppb_graphics_2d_impl.cc
@@ -248,11 +248,8 @@ void PPB_Graphics2D_Impl::PaintImageData(PP_Resource image_data,
operation.paint_image = image_resource;
if (!ValidateAndConvertRect(src_rect, image_resource->width(),
image_resource->height(),
- &operation.paint_src_rect)) {
- Log(PP_LOGLEVEL_ERROR,
- "PPB_Graphics2D.PaintImageData: Rectangle is outside bounds.");
+ &operation.paint_src_rect))
return;
- }
// Validate the bitmap position using the previously-validated rect, there
// should be no painted area outside of the image.
@@ -278,11 +275,8 @@ void PPB_Graphics2D_Impl::Scroll(const PP_Rect* clip_rect,
if (!ValidateAndConvertRect(clip_rect,
image_data_->width(),
image_data_->height(),
- &operation.scroll_clip_rect)) {
- Log(PP_LOGLEVEL_ERROR,
- "PPB_Graphics2D.Scroll: Rectangle is outside bounds.");
+ &operation.scroll_clip_rect))
return;
- }
// If we're being asked to scroll by more than the clip rect size, just
// ignore this scroll command and say it worked.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698