[Webkit-unassigned] [Bug 43251] Web Inspector: add support for DOM-style events to WebInspector Extensions API

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 6 09:57:34 PDT 2010


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





--- Comment #3 from Joseph Pecoraro <joepeck at webkit.org>  2010-08-06 09:57:34 PST ---
(From update of attachment 63605)
> diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog

> +++ b/LayoutTests/ChangeLog
> +        Web Inspector: added tests for event handling, updated expectations to include new API methods.
> +        https://bugs.webkit.org/show_bug.cgi?id=43344

I just glanced at this and felt I should not a couple things:

  1) wrong bug number! The bug # for this was 43251
  2) ChangeLogs have a typical format which tools like
     webkit-patch and prepare-ChangeLog will generate
     for you. Don't edit the bug title & link. If you have
     comments, put them beneath the bug link.

Glancing at some of your other commits, it looks like you've
been replacing the bug title with your comments. Not a big
deal, just something I noticed.

Examples of good style (on the front page of trac.webkit.org):
http://trac.webkit.org/changeset/64855
http://trac.webkit.org/changeset/64852

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



More information about the webkit-unassigned mailing list