add more roles into markup map

RESOLVED FIXED in Firefox 39

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: surkov, Assigned: surkov)

Tracking

unspecified
mozilla39
Points:
---

Firefox Tracking Flags

(firefox39 fixed)

Details

Attachments

(1 attachment)

Posted 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: 4 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.