Closed Bug 1140497 Opened 6 years ago Closed 6 years ago

add more roles into markup map

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: surkov, Assigned: surkov)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
No description provided.
Attachment #8574068 - Flags: review?(mzehe)
Attachment #8574068 - Flags: review?(mzehe) → review+
https://hg.mozilla.org/mozilla-central/rev/7bcbc6e19863
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Comment on attachment 8574068 [details] [diff] [review]
patch

I'll bet good money this makes calling Role() on these types of objects slower, but I'm not sure if that matters much.
(In reply to Trevor Saunders (:tbsaunde) from comment #3)
> Comment on attachment 8574068 [details] [diff] [review]
> patch
> 
> I'll bet good money this makes calling Role() on these types of objects
> slower, but I'm not sure if that matters much.

right, I think that shouldn't be so noticable to make it a bad trade off, I'm really up to the idea to move closer to delcarative stuff
(In reply to alexander :surkov from comment #4)
> (In reply to Trevor Saunders (:tbsaunde) from comment #3)
> > Comment on attachment 8574068 [details] [diff] [review]
> > patch
> > 
> > I'll bet good money this makes calling Role() on these types of objects
> > slower, but I'm not sure if that matters much.
> 
> right, I think that shouldn't be so noticable to make it a bad trade off,
> I'm really up to the idea to move closer to delcarative stuff

it doesn't seem very great to me.
You need to log in before you can comment on or make changes to this bug.