Closed Bug 1257526 Opened 8 years ago Closed 8 years ago

Return unsupported operation error for screen orientation commands on desktop

Categories

(Remote Protocol :: Marionette, defect)

Version 3
defect
Not set
normal

Tracking

(firefox47 fixed, firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox47 --- fixed
firefox48 --- fixed

People

(Reporter: ato, Assigned: ato)

References

Details

(Keywords: pi-marionette-server)

Attachments

(2 files)

The setScreenOrientation and getScreenOrientation methods on GeckoDriver should throw UnsupportedOperationError on desktop.
Assignee: nobody → ato
Status: NEW → ASSIGNED
Throw unsupported operation errors the same way throughout Marionette.

Review commit: https://reviewboard.mozilla.org/r/40787/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/40787/
Attachment #8731711 - Flags: review?(dburns)
Sweet. This will fix at least a part of bug 1256682.
Blocks: 1256682
Comment on attachment 8731710 [details]
MozReview Request: Bug 1257526 - Return unsupported operation error for screen orientation on desktop; r=automatedtester

https://reviewboard.mozilla.org/r/40785/#review37407

::: testing/marionette/harness/marionette/tests/unit/test_screen_orientation.py:15
(Diff revision 1)
>  unknown_orientation = "Unknown screen orientation: %s"
>  
>  class TestScreenOrientation(MarionetteTestCase):
> +    def setUp(self):
> +        MarionetteTestCase.setUp(self)
> +        self.is_mobile = self.marionette.session_capabilities["platformName"] == "B2G"

Can you change this to ` != "Firefox"`
Attachment #8731710 - Flags: review?(dburns) → review+
Attachment #8731711 - Flags: review?(dburns) → review+
Comment on attachment 8731711 [details]
MozReview Request: Bug 1257526 - Harmonise unsupported operation error usage; r=automatedtester

https://reviewboard.mozilla.org/r/40787/#review37435
Comment on attachment 8731710 [details]
MozReview Request: Bug 1257526 - Return unsupported operation error for screen orientation on desktop; r=automatedtester

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/40785/diff/1-2/
Comment on attachment 8731711 [details]
MozReview Request: Bug 1257526 - Harmonise unsupported operation error usage; r=automatedtester

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/40787/diff/1-2/
Comment on attachment 8731710 [details]
MozReview Request: Bug 1257526 - Return unsupported operation error for screen orientation on desktop; r=automatedtester

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/40785/diff/2-3/
Attachment #8731710 - Attachment description: MozReview Request: Bug 1257526 - Return unsupported operation error for screen orientation on desktop; r?automatedtester → MozReview Request: Bug 1257526 - Return unsupported operation error for screen orientation on desktop; r=automatedtester
Comment on attachment 8731711 [details]
MozReview Request: Bug 1257526 - Harmonise unsupported operation error usage; r=automatedtester

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/40787/diff/2-3/
Attachment #8731711 - Attachment description: MozReview Request: Bug 1257526 - Harmonise unsupported operation error usage; r?automatedtester → MozReview Request: Bug 1257526 - Harmonise unsupported operation error usage; r=automatedtester
(In reply to Carsten Book [:Tomcat] from comment #9)
> backed out for test failures like
> https://treeherder.mozilla.org/logviewer.html#?job_id=24325568&repo=mozilla-
> inbound

Sorry, my mistake.  Issue was caused by a fixup to the patch.
Flags: needinfo?(ato)
https://hg.mozilla.org/mozilla-central/rev/66fd82b72d1e
https://hg.mozilla.org/mozilla-central/rev/0d344473d50f
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.