Put drawSystemFocusRing behind a new runtime flag

RESOLVED FIXED in mozilla29

Status

()

Product:
Core
Component:
Canvas: 2D
Status:
RESOLVED FIXED
Reported:
5 years ago
Modified:
5 years ago

People

(Reporter: cabanier, Assigned: cabanier)

Tracking

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

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

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Added new runtime flag
4.55 KB, patch
roc
: review+
Details | Diff | Splinter Review
(Assignee)

Description

•
5 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

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

Comment 1

•
5 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 2

•
5 years ago 
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

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

Updated

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

Updated

•
5 years ago
Keywords: checkin-needed

Comment 4

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