Last Comment Bug 652814 - Rename nsIDOMTouchPoint to nsIDOMTouch
: Rename nsIDOMTouchPoint to nsIDOMTouch
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: DOM: Events (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla6
Assigned To: Olli Pettay [:smaug] (pto-ish for couple of days)
:
: Andrew Overholt [:overholt]
Mentors:
Depends on:
Blocks: 648573
  Show dependency treegraph
 
Reported: 2011-04-26 08:36 PDT by Olli Pettay [:smaug] (pto-ish for couple of days)
Modified: 2011-04-27 05:55 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (17.68 KB, patch)
2011-04-26 09:20 PDT, Olli Pettay [:smaug] (pto-ish for couple of days)
mbrubeck: review+
Details | Diff | Splinter Review

Description User image Olli Pettay [:smaug] (pto-ish for couple of days) 2011-04-26 08:36:46 PDT

    
Comment 1 User image Olli Pettay [:smaug] (pto-ish for couple of days) 2011-04-26 09:20:48 PDT
Created attachment 528325 [details] [diff] [review]
patch

Matt, I think you could review this. No need to ask
jst or other DOM peers.
Comment 2 User image Matt Brubeck (:mbrubeck) 2011-04-26 10:39:44 PDT
I wonder if we should also rename identifiedPoint to identifiedTouch (in the spec and the implementation)...
Comment 3 User image Matt Brubeck (:mbrubeck) 2011-04-26 10:44:21 PDT
(In reply to comment #2)
> I wonder if we should also rename identifiedPoint to identifiedTouch (in the
> spec and the implementation)...

Pushed this change to the spec:
http://dvcs.w3.org/hg/webevents/rev/fdd08cbcf438
Comment 4 User image Olli Pettay [:smaug] (pto-ish for couple of days) 2011-04-27 05:54:56 PDT
http://hg.mozilla.org/mozilla-central/rev/ac941611adf5

And note to whoever is reading this all, touch event interfaces
are still disabled by default. The interfaces will change.

Note You need to log in before you can comment on or make changes to this bug.