1530199 - Remove hasSupportsHighlighters check in supportsEyeDropper

Status: RESOLVED FIXED

(DevTools :: Inspector, enhancement, P3)

Description

[Gabriel [:gl] (ΦωΦ)]

Currently, we have a supportsEyeDropper function in inspector.js that checks if the actor has "supportsHighlighter" method to handle backward support of different actors with potentially different methods. Since the actor method "supportsHighligther" was implemented in Bug 1306937 (back in 2016), we don't need this actor method check anymore since we are always executing await this.inspector.supportsHighlighters().

So, this bug is to clean up the try statement of supportsEyeDropper https://searchfox.org/mozilla-central/source/devtools/client/inspector/inspector.js#1036 by removing the hasSupportsHighlighters check and simplifying it to do a single return:

return await this.inspector.supportsHighlighters()

Comment 1

[Rizwan Syed]

Hi. I am an outreachy applicant and would like to work on this. May you please assign it to me? Thanks :)

Comment 2

[Gabriel [:gl] (ΦωΦ)]

(In reply to jk025523 from comment #1)

Hi. I am an outreachy applicant and would like to work on this. May you please assign it to me? Thanks :)

Hi, feel free to submit a patch.

Comment 3

[lloan alas:[lloanalas]]

Hi, I'm an outreachy applicant. This bug is assigned, if no one has picked it up yet, can I be assigned to it? (in case previous requester was assigned another bug). I was reading the How to Submit a Patch instructions and noticed you have to be assigned to a bug to submit a patch.

Thanks a ton!

Comment 4

[lloan alas:[lloanalas]]

(In reply to lloanalas from comment #3)

Hi, I'm an outreachy applicant. This bug is assigned, if no one has picked it up yet, can I be assigned to it? (in case previous requester was assigned another bug). I was reading the How to Submit a Patch instructions and noticed you have to be assigned to a bug to submit a patch.

Thanks a ton!

Not assigned*

Comment 5

[Rizwan Syed]

Hi, I was not assigned another bug, and when I requested to be assigned this bug, the project maintainer @Gabriel refused to. Why a different decision this time @Gabriel? I was preparing to submit a patch for this bug.

Comment 6

[Gabriel [:gl] (ΦωΦ)]

(In reply to jk025523 from comment #5)

Hi, I was not assigned another bug, and when I requested to be assigned this bug, the project maintainer @Gabriel refused to. Why a different decision this time @Gabriel? I was preparing to submit a patch for this bug.

Oh my mistake, I actually only read the comments in my email and didn't see the previous comments.

Comment 7

[lloan alas:[lloanalas]]

(In reply to Gabriel [:gl] (ΦωΦ) from comment #6)

(In reply to jk025523 from comment #5)

Hi, I was not assigned another bug, and when I requested to be assigned this bug, the project maintainer @Gabriel refused to. Why a different decision this time @Gabriel? I was preparing to submit a patch for this bug.

Oh my mistake, I actually only read the comments in my email and didn't see the previous comments.

I understand. I had started working on it as well. No worries. I was just reading on how to submit a patch as mine is ready.

Thanks regardless :)

Comment 8

[lloan alas:[lloanalas]]

Attachment: Bug 1530199 - Remove hasSupportsHighlighters check in supportsEyeDropper r=gl

Cleans up the try statement within supportsEyeDropper function.

Comment 9

[lloan alas:[lloanalas]]

(In reply to lloanalas from comment #8)

Created attachment 9046834 [details]
cleans up the try statement in supportsEyeDropper - bug 1530199

Cleans up the try statement within supportsEyeDropper function.

Please feel free to disregard. Since I had already done the work, I figured I could use this to figure out how to do an actual commit - to get a hang of the process.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9b1e57540da7
Remove hasSupportsHighlighters check in supportsEyeDropper r=gl

Comment 11

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/9b1e57540da7