Chromium Code Reviews| Index: ppapi/examples/2d/paint_manager_example.cc |
| diff --git a/ppapi/examples/2d/paint_manager_example.cc b/ppapi/examples/2d/paint_manager_example.cc |
| index 86bdbe79ed8f9f2a8e820cd682f24c57cbb413d7..430f42259cccbecf26ad2589285ded3891d0cc2a 100644 |
| --- a/ppapi/examples/2d/paint_manager_example.cc |
| +++ b/ppapi/examples/2d/paint_manager_example.cc |
| @@ -78,7 +78,9 @@ class MyInstance : public pp::Instance, public pp::PaintManager::Client { |
| // PaintManager::Client implementation. |
| virtual bool OnPaint(pp::Graphics2D& graphics_2d, |
| const std::vector<pp::Rect>& paint_rects, |
| - const pp::Rect& paint_bounds) { |
| + const pp::Rect& paint_bounds_unused) { |
| + pp::Rect paint_bounds(paint_manager_.GetEffectiveSize()); |
|
brettw
2012/08/21 20:56:10
Ignore the changes in this file, this is for testi
|
| + |
| // Make an image just large enough to hold all dirty rects. We won't |
| // actually paint all of these pixels below, but rather just the dirty |
| // ones. Since image allocation can be somewhat heavyweight, we wouldn't |
| @@ -96,6 +98,7 @@ class MyInstance : public pp::Instance, public pp::PaintManager::Client { |
| // Note that the aggregator used by the paint manager won't give us |
| // multiple regions that overlap, so we don't have to worry about double |
| // painting in this code. |
| +/* |
| for (size_t i = 0; i < paint_rects.size(); i++) { |
| // Since our image is just the invalid region, we need to offset the |
| // areas we paint by that much. This is just a light blue background. |
| @@ -106,6 +109,7 @@ class MyInstance : public pp::Instance, public pp::PaintManager::Client { |
| paint_rects[i].height(), |
| 0xFFAAAAFF); |
| } |
| + */ |
| // Paint the square black. Because we're lazy, we do this outside of the |
| // loop above. |
| @@ -117,7 +121,7 @@ class MyInstance : public pp::Instance, public pp::PaintManager::Client { |
| square.height(), |
| 0xFF000000); |
| - graphics_2d.PaintImageData(updated_image, paint_bounds.point()); |
| + graphics_2d.ReplaceContents(&updated_image); |
| return true; |
| } |