[Webkit-unassigned] [Bug 20255] New: Coding style fixes to EventHandler

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 1 08:39:30 PDT 2008


           Summary: Coding style fixes to EventHandler
           Product: WebKit
           Version: 526+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Minor
          Priority: P2
         Component: WebKit Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: joe at staikos.net

Just a few misc problems I noticed when looking at EventHandler:

All of the fields are public, even though some of them have accessors!  That's
just bad practice.

The comment about m_capturingMouseEventsNode in handleMousePressEvent is pretty
unclear.  It took me a while to figure out what it meant.

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