[Webkit-unassigned] [Bug 36568] Support BlobBuilder defined in FileAPI/FileWriter

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=36568


Dmitry Titov <dimich at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #52206|review?                     |review-
               Flag|                            |




--- Comment #12 from Dmitry Titov <dimich at chromium.org>  2010-03-31 14:21:13 PST ---
(From update of attachment 52206)
r-, due to the size of the patch.

It is very hard to review 80Kb patch, and if the review happens, the quality of
it usually suffers. We ask contributors to come up with smaller patches,
especially if it is relatively easy to do.

In this case, it's possible to structure the work like this:
1. Have an uber-bug which says "Implement BlobBuilder" and serves as umbrella
for other bugs/patches.
2. Optionally attach the uber-patch with prototype into that bug.
3. Split work - in this case, it could be implementation of internal
implementation objects, like FileBlob, then JSC bindings with a test or two,
then v8 bindings, then more tests.
4. Have separate bugs with those smaller patches, referrign to the umbrella
bug. Normally, we have 1 patch per bug.

I myself didn't appreciate much the 'smaller patches' concept until I became a
reviewer. It takes about the same time to carefully review the patch as it
would take to write it. In the case of a big patch, the process repeats on
every iteration. It is really better and faster to come up with smaller
patches.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.