Disable highlighter on aurora

VERIFIED FIXED in Firefox 9

Status

()

Firefox
Developer Tools
VERIFIED FIXED
6 years ago
4 years ago

People

(Reporter: dcamp, Assigned: dcamp)

Tracking

({verified-aurora, verified-beta})

9 Branch
Firefox 9
x86
All
verified-aurora, verified-beta
Points:
---

Firefox Tracking Flags

(firefox9+ fixed)

Details

(Whiteboard: [qa!])

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

6 years ago
Highlighter isn't ready to ship, disable it on aurora.
(Assignee)

Comment 1

6 years ago
Created attachment 562922 [details] [diff] [review]
Disable highlighter on aurora.
Attachment #562922 - Flags: review?(rcampbell)
Attachment #562922 - Flags: review?(rcampbell)
Attachment #562922 - Flags: review+
Attachment #562922 - Flags: approval-mozilla-aurora?

Updated

6 years ago
Attachment #562922 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Updated

6 years ago
tracking-firefox9: ? → +
(Assignee)

Comment 2

6 years ago
Created attachment 567801 [details] [diff] [review]
fix tests
Attachment #567801 - Flags: review?
(Assignee)

Updated

6 years ago
Attachment #567801 - Flags: review? → review?(rcampbell)
Comment on attachment 567801 [details] [diff] [review]
fix tests

also need to update these files:

./styleinspector/test/browser/browser_bug683672.js:  ok(StyleInspector.isEnabled, "style inspector preference is enabled");
./styleinspector/test/browser/browser_styleinspector.js:  ok(StyleInspector.isEnabled, "style inspector preference is enabled");
./styleinspector/test/browser/browser_styleinspector_bug_672744_search_filter.js:  ok(StyleInspector.isEnabled, "style inspector preference is enabled");
./styleinspector/test/browser/browser_styleinspector_bug_672746_default_styles.js:  ok(StyleInspector.isEnabled, "style inspector preference is enabled");
Attachment #567801 - Flags: review?(rcampbell) → review-
(Assignee)

Comment 4

6 years ago
Created attachment 567832 [details] [diff] [review]
fix tests again
Attachment #567801 - Attachment is obsolete: true
Attachment #567832 - Flags: review?
Comment on attachment 567832 [details] [diff] [review]
fix tests again

ok, I think that'll do it.
Attachment #567832 - Flags: review? → review+
(Assignee)

Comment 6

6 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/4efa58448392
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Depends on: 696167
(Assignee)

Comment 7

6 years ago
Backed out in https://hg.mozilla.org/releases/mozilla-aurora/rev/1d546fa99f1b for test failures, filed bug 696167 with the test changes needed.
(Assignee)

Updated

6 years ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 8

6 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/8b0b938b03a6
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
status-firefox9: --- → fixed
Whiteboard: [qa+]
Verified that highlighter is disabled in Beta(9.0):
Mozilla/5.0 (Windows NT 6.1; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:9.0) Gecko/20100101 Firefox/9.0
Mozilla/5.0 (Windows NT 5.1; rv:9.0) Gecko/20100101 Firefox/9.0 (20111116091359)
Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20100101 Firefox/9.0

Highlighter is available in Aurora (10.0) and Nightly (11.0):
Mozilla/5.0 (Windows NT 6.1; rv:10.0a2) Gecko/20111121 Firefox/10.0a2
Mozilla/5.0 (Windows NT 6.1; rv:11.0a1) Gecko/20111121 Firefox/11.0a1

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:10.0a2) Gecko/20111121 Firefox/10.0a2
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:11.0a1) Gecko/20111121 Firefox/11.0a1

Mozilla/5.0 (Windows NT 5.1; rv:10.0a2) Gecko/20111121 Firefox/10.0a2
Mozilla/5.0 (Windows NT 5.1; rv:11.0a1) Gecko/20111121 Firefox/11.0a1

Mozilla/5.0 (X11; Linux x86_64; rv:10.0a2) Gecko/20111117 Firefox/10.0a2
Mozilla/5.0 (X11; Linux x86_64; rv:11.0a1) Gecko/20111118 Firefox/11.0a1
Status: RESOLVED → VERIFIED
Keywords: verified-aurora, verified-beta
Whiteboard: [qa+] → [qa!]
You need to log in before you can comment on or make changes to this bug.