[Webkit-unassigned] [Bug 26502] [Chromium] Use WebCore::multipleFileUploadText()

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 18 15:52:48 PDT 2009


levin at chromium.org changed:

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

------- Comment #2 from levin at chromium.org  2009-06-18 15:52 PDT -------
(From update of attachment 31470)

> Index: WebCore/ChangeLog
> +2009-06-17  Kent Tamura  <tkent at chromium.org>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Use WebCore::multipleFileUploadText() in Chromium.
> +        <https://bugs.webkit.org/show_bug.cgi?id=26502>
> +

Ideally you would have something about why you are doing the change here?
What does the change accomplish?  (You have this in the bug.)

This change looks good but the changelog needs to be improved, so r- until that
is fixed.  With a better change log, I'd happily give it an r+.

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

More information about the webkit-unassigned mailing list