Put drawSystemFocusRing behind a new runtime flag

RESOLVED FIXED in mozilla29

Status

()

Core
Canvas: 2D
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Rik Cabanier, Assigned: Rik Cabanier)

Tracking

(Blocks: 1 bug, {access, html5})

Trunk
mozilla29
access, html5
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
drawSystemFocusRing and drawFocusIfNeeded share the same runtime flag.
drawFocusIfNeeded will ship as-is but drawSystemFocusRing will most likely change in the future.

Putting drawSystemFocusRing under a new runtime flag so it can be turned on separately.
(Assignee)

Updated

4 years ago
Assignee: nobody → cabanier
Blocks: 935992
Keywords: access, html5
(Assignee)

Comment 1

4 years ago
Created attachment 8360652 [details] [diff] [review]
Added new runtime flag

try run: https://tbpl.mozilla.org/?tree=Try&rev=d233e33b7d4a
Attachment #8360652 - Flags: review?(roc)
Comment on attachment 8360652 [details] [diff] [review]
Added new runtime flag

Review of attachment 8360652 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #8360652 - Flags: review?(roc) → review+
(Assignee)

Updated

4 years ago
Attachment #8360652 - Flags: checkin?(ryanvm)
(Assignee)

Updated

4 years ago
Attachment #8360652 - Flags: checkin?(ryanvm)
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/8d3e02a7f23c
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.