Closed
Bug 1575511
Opened 5 years ago
Closed 5 years ago
Remove the highlight feature of Marionette screenshots
Categories
(Remote Protocol :: Marionette, task, P1)
Tracking
(firefox70 fixed)
RESOLVED
FIXED
mozilla70
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: whimboo, Assigned: whimboo)
References
Details
Attachments
(1 file)
This feature was used in the past by some l10n tests to highlight elements for localized builds of Firefox, eg. for missing access keys, or cut-off elements.
Given that we no longer run those tests there isn't actually a need to keep this code in Marionette. It would also help me a lot with the refactoring on bug 1559592.
Lets just kill the code!
Assignee | ||
Comment 1•5 years ago
|
||
The feature was used in the past to highlight broken elements for l10n
specific tests. Given that those tests don't exist anymore (for already
a long time) the highlight feature doesn't have to be kept alive.
Also it isn't covered by the WebDriver spec, and as such a custom feature
which is not worth keeping its code working. Especially with the Fission
work upcoming.
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4ab60925635c [marionette] Remove feature to highlight elements in screenshots. r=webdriver-reviewers,ato
Comment 3•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•