Closed Bug 1477858 Opened 2 years ago Closed 2 years ago

add Qcms fuzzing target dictionary

Categories

(Core :: GFX: Color Management, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: u473386, Assigned: u473386)

Details

Attachments

(1 file)

Why not, I'm thinking. Same as oss-fuzz.
Attached patch dictionarySplinter Review
Attachment #8994357 - Flags: review?(choller)
Component: General → GFX: Color Management
Product: Firefox → Core
Assignee: nobody → pdknsk
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #8994357 - Flags: review?(choller) → review+
Attachment #8994357 - Flags: checkin?
Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d89d8b34e5bd
add Qcms fuzzing target dictionary. r=decoder
https://hg.mozilla.org/mozilla-central/rev/d89d8b34e5bd
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Hello, did you mean by any chance chekin-needed?
Flags: needinfo?(pdknsk)
Attachment #8994357 - Flags: checkin? → checkin-
Thanks, but it was already commited. I can't use checkin-needed.
Flags: needinfo?(pdknsk)
Attachment #8994357 - Flags: checkin-
(In reply to pdknsk from comment #5)
> Thanks, but it was already commited. I can't use checkin-needed.

You added the checkin to the patch details. When you want a patch to be landed, after it's properly reviewed, you just need to add in the keywords field checkin-needed. 
Let me know if I can help with anything else.
> You added the checkin to the patch details. When you want a patch to be
> landed, after it's properly reviewed, you just need to add in the keywords
> field checkin-needed. 

I had somehow misunderstood from the docs that checkin-needed should be added to the patch. And since checkin-needed is not available as a flag on the patch, I've been using checkin instead. (I thought maybe I don't have permission to use checkin-needed.) I understand now that's its a keyword on the bug, not a flag on the patch. Thanks for the clarification.
You need to log in before you can comment on or make changes to this bug.