Build rule for pdf_codec_fax_fuzzer

Build rule for pdf_codec_fax_fuzzer

Committed: https://crrev.com/a6dc4f682f6db2b4792963ef5a26929d8a728f2a
Cr-Commit-Position: refs/heads/master@{#419859}

[kcwu, 2016-09-20 17:42:11]

kcwu@chromium.org changed reviewers:
+ thestig@chromium.org

[Lei Zhang, 2016-09-20 17:53:47]

lgtm

https://codereview.chromium.org/2354003002/diff/1/pdf/pdfium/fuzzers/BUILD.gn
File pdf/pdfium/fuzzers/BUILD.gn (right):

https://codereview.chromium.org/2354003002/diff/1/pdf/pdfium/fuzzers/BUILD.gn...
pdf/pdfium/fuzzers/BUILD.gn:14: fuzzer_test("pdf_codec_fax_fuzzer") {
Can you move this down to after pdfium_fuzzer, to keep it together with the
other pdf_foo_fuzzer targets?

[kcwu, 2016-09-20 18:00:49]

https://codereview.chromium.org/2354003002/diff/1/pdf/pdfium/fuzzers/BUILD.gn
File pdf/pdfium/fuzzers/BUILD.gn (right):

https://codereview.chromium.org/2354003002/diff/1/pdf/pdfium/fuzzers/BUILD.gn...
pdf/pdfium/fuzzers/BUILD.gn:14: fuzzer_test("pdf_codec_fax_fuzzer") {
On 2016/09/20 17:53:46, Lei Zhang wrote:
> Can you move this down to after pdfium_fuzzer, to keep it together with the
> other pdf_foo_fuzzer targets?

Done.

[kcwu, 2016-09-20 18:01:04]

The CQ bit was checked by kcwu@chromium.org

[kcwu, 2016-09-20 18:01:05]

The patchset sent to the CQ was uploaded after l-g-t-m from thestig@chromium.org
Link to the patchset: https://codereview.chromium.org/2354003002/#ps20001
(title: "move order")

[commit-bot: I haz the power, 2016-09-20 18:01:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-20 20:18:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-20 20:18:56]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Lei Zhang, 2016-09-20 20:31:12]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2016-09-20 20:32:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-20 21:43:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-20 21:45:06]

Description was changed from

==========
Build rule for pdf_codec_fax_fuzzer
==========

to

==========
Build rule for pdf_codec_fax_fuzzer

Committed: https://crrev.com/a6dc4f682f6db2b4792963ef5a26929d8a728f2a
Cr-Commit-Position: refs/heads/master@{#419859}
==========

[commit-bot: I haz the power, 2016-09-20 21:45:07]

Patchset 2 (id:??) landed as
https://crrev.com/a6dc4f682f6db2b4792963ef5a26929d8a728f2a
Cr-Commit-Position: refs/heads/master@{#419859}