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] (high review load, please consider other reviewers)
:
Mentors:
Depends on:
Blocks: 648573
  Show dependency treegraph
 
Reported: 2011-04-26 08:36 PDT by Olli Pettay [:smaug] (high review load, please consider other reviewers)
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] (high review load, please consider other reviewers)
mbrubeck: review+
Details | Diff | Review

Description Olli Pettay [:smaug] (high review load, please consider other reviewers) 2011-04-26 08:36:46 PDT

    
Comment 1 Olli Pettay [:smaug] (high review load, please consider other reviewers) 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 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 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 Olli Pettay [:smaug] (high review load, please consider other reviewers) 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.