[chromium] Add documentation to updater classes
Created attachment 121692 [details] Patch
Comment on attachment 121692 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=121692&action=review Great! Thanks for doing this. > Source/WebCore/platform/graphics/chromium/SkPictureCanvasLayerTextureUpdater.h:43 > +// BitmapSkPictureCanvasLayerTextureUpdater and FrameBufferSkPictureCanvasLayerTextureUpdater I'd prefer you changed this comment so that adding a new SkPictureCanvasLayerTextureUpdater subclass doesn't make it inaccurate or require an update to this file. How about making it a bit more general. Maybe just make it "BitmapSkPic../FramebufferSkPic.. are examples of subclasses that implement different..."
Created attachment 121705 [details] Patch
Thanks. LGTM.
Comment on attachment 121705 [details] Patch Clearing flags on attachment: 121705 Committed r104522: <http://trac.webkit.org/changeset/104522>
All reviewed patches have been landed. Closing bug.