add Qcms fuzzing target dictionary

RESOLVED FIXED in Firefox 63

Status

()

enhancement
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: pdknsk, Assigned: pdknsk)

Tracking

unspecified
mozilla63
Points:
---

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

11 months ago
Why not, I'm thinking. Same as oss-fuzz.
Assignee

Comment 1

11 months ago
Posted 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+
Assignee

Updated

11 months ago
Attachment #8994357 - Flags: checkin?

Comment 2

11 months ago
Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d89d8b34e5bd
add Qcms fuzzing target dictionary. r=decoder

Comment 3

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d89d8b34e5bd
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Hello, did you mean by any chance chekin-needed?
Flags: needinfo?(pdknsk)
Attachment #8994357 - Flags: checkin? → checkin-
Assignee

Comment 5

11 months ago
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.
Assignee

Comment 7

11 months ago
> 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.