[libfuzzer] use source-built expat parser

[libfuzzer] use source-built expat parser

Fuzzer can't progress with system one, you need to instrument sources.

BUG=

Committed: https://crrev.com/8f4bcd2f2244dcde5beb4627e3e9842445948e5b
Cr-Commit-Position: refs/heads/master@{#397277}

[aizatsky, 2016-05-31 22:53:39]

aizatsky@chromium.org changed reviewers:
+ ben@chromium.org, mmoroz@chromium.org, thakis@chromium.org

[mmoroz, 2016-06-01 11:58:23]

LGTM

Now it makes sense why that fuzzer is so boring! Thanks Mike!

[mmoroz, 2016-06-01 11:58:38]

The CQ bit was checked by mmoroz@chromium.org

[commit-bot: I haz the power, 2016-06-01 11:58:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2026113002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2026113002/1

[commit-bot: I haz the power, 2016-06-01 12:04:35]

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

[commit-bot: I haz the power, 2016-06-01 12:04:37]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Nico, 2016-06-01 15:44:14]

lgtm with comment addressed

https://codereview.chromium.org/2026113002/diff/1/third_party/expat/BUILD.gn
File third_party/expat/BUILD.gn (right):

https://codereview.chromium.org/2026113002/diff/1/third_party/expat/BUILD.gn#...
third_party/expat/BUILD.gn:8: # Chromecast doesn't ship expat as a system
library
can you add the comment

      # On Linux, we implicitly already depend on expat via fontconfig;
      # let's not pull it in twice.

from the gyp file?

[aizatsky, 2016-06-01 20:54:02]

The CQ bit was checked by aizatsky@chromium.org

[aizatsky, 2016-06-01 20:54:03]

The patchset sent to the CQ was uploaded after l-g-t-m from mmoroz@chromium.org,
thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/2026113002/#ps20001
(title: "comments")

[commit-bot: I haz the power, 2016-06-01 20:54:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2026113002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2026113002/20001

[commit-bot: I haz the power, 2016-06-01 23:08:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-01 23:09:35]

Description was changed from

==========
[libfuzzer] use source-built expat parser

Fuzzer can't progress with system one, you need to instrument sources.

BUG=
==========

to

==========
[libfuzzer] use source-built expat parser

Fuzzer can't progress with system one, you need to instrument sources.

BUG=

Committed: https://crrev.com/8f4bcd2f2244dcde5beb4627e3e9842445948e5b
Cr-Commit-Position: refs/heads/master@{#397277}
==========

[commit-bot: I haz the power, 2016-06-01 23:09:36]

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

[aizatsky, 2016-06-02 21:02:08]

https://codereview.chromium.org/2026113002/diff/1/third_party/expat/BUILD.gn
File third_party/expat/BUILD.gn (right):

https://codereview.chromium.org/2026113002/diff/1/third_party/expat/BUILD.gn#...
third_party/expat/BUILD.gn:8: # Chromecast doesn't ship expat as a system
library
On 2016/06/01 at 15:44:14, Nico wrote:
> can you add the comment
> 
>       # On Linux, we implicitly already depend on expat via fontconfig;
>       # let's not pull it in twice.
> 
> from the gyp file?

Done. Also added libfuzzer comment.