Add OWNERS for expat

Add OWNERS for expat

R=nick@chromium.org,dominicc@chromium.org
BUG=

Committed: https://crrev.com/4dced320561af10d1a2c5264598b855886c236d3
Cr-Commit-Position: refs/heads/master@{#407006}

[jochen (gone - plz use gerrit), 2016-07-21 08:18:58]

Nick, because you added the library initially, Dominic, because you own the
libxml stuff anyways

[ncarter (slow), 2016-07-21 17:12:36]

nick@chromium.org changed reviewers:
+ zea@chromium.org

[ncarter (slow), 2016-07-21 17:12:37]

I'm not really a stakeholder of this directory anymore; though I would be happy
to try to review rolls if nobody else steps up.

Originally we needed expat for sync because of libjingle. I'm not sure that a
sync->libjingle dependency still exists? It looks like the current stakeholders
are webrtc and possibly some android font stuff?

+zea and +tommi -- do you know who cares about expat?

[chromium-reviews, 2016-07-21 18:05:04]

Sync uses libjingle for a few integration tests still, but we're in the
process of trying to get rid of those final dependencies. Someone from
webrtc, as you say, might be best?

On Thu, Jul 21, 2016 at 10:12 AM <nick@chromium.org> wrote:

> I'm not really a stakeholder of this directory anymore; though I would be
> happy
> to try to review rolls if nobody else steps up.
>
> Originally we needed expat for sync because of libjingle. I'm not sure
> that a
> sync->libjingle dependency still exists? It looks like the current
> stakeholders
> are webrtc and possibly some android font stuff?
>
> +zea and +tommi -- do you know who cares about expat?
>
> https://codereview.chromium.org/2163373006/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[ncarter (slow), 2016-07-21 18:47:32]

lgtm

[jochen (gone - plz use gerrit), 2016-07-21 19:35:24]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-07-21 19:35:53]

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-07-21 20:02:32]

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

[commit-bot: I haz the power, 2016-07-21 20:02:35]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[dominicc (has gone to gerrit), 2016-07-22 01:01:05]

The CQ bit was checked by dominicc@chromium.org

[dominicc (has gone to gerrit), 2016-07-22 01:01:05]

lgtm

[commit-bot: I haz the power, 2016-07-22 01:02:15]

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-07-22 03:12:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-22 03:15:15]

Description was changed from

==========
Add OWNERS for expat

R=nick@chromium.org,dominicc@chromium.org
BUG=
==========

to

==========
Add OWNERS for expat

R=nick@chromium.org,dominicc@chromium.org
BUG=

Committed: https://crrev.com/4dced320561af10d1a2c5264598b855886c236d3
Cr-Commit-Position: refs/heads/master@{#407006}
==========

[commit-bot: I haz the power, 2016-07-22 03:15:16]

Patchset 1 (id:??) landed as
https://crrev.com/4dced320561af10d1a2c5264598b855886c236d3
Cr-Commit-Position: refs/heads/master@{#407006}