Correct security critical bit for third_party/ced

Correct security critical bit for third_party/ced

This library is shipped, so should be marked as "Security Critical".

R=jinsukkim@chromium.org,jshin@chromium.org

Review-Url: https://codereview.chromium.org/2842813002
Cr-Commit-Position: refs/heads/master@{#467155}
Committed: https://chromium.googlesource.com/chromium/src/+/c2f72fc86077822222610a0ed35ef068a010733a

[Oliver Chang, 2017-04-25 17:57:32]

Description was changed from

==========
Correct security critical bit for third_party/ced

This library is shipped, so should be marked as "Security Critical".

R=jinsukkim@chromium.org,jshin@chromium.org
==========

to

==========
Correct security critical bit for third_party/ced

This library is shipped, so should be marked as "Security Critical".

R=jinsukkim@chromium.org,jshin@chromium.org
==========

[Oliver Chang, 2017-04-25 17:57:39]

PTAL, thanks!

[Jinsuk Kim, 2017-04-25 22:20:05]

On 2017/04/25 17:57:39, Oliver Chang wrote:
> PTAL, thanks!

Thanks for the change. Would you let me know where the policy for this 'Security
Critical' bit is documented? Not easy for me to find it.

[Oliver Chang, 2017-04-25 22:39:24]

On 2017/04/25 22:20:05, Jinsuk Kim wrote:
> On 2017/04/25 17:57:39, Oliver Chang wrote:
> > PTAL, thanks!
> 
> Thanks for the change. Would you let me know where the policy for this
'Security
> Critical' bit is documented? Not easy for me to find it.

It's here in the README.chromium template:
https://cs.chromium.org/chromium/src/third_party/README.chromium.template?l=9

[Jinsuk Kim, 2017-04-25 22:45:08]

On 2017/04/25 22:39:24, Oliver Chang wrote:
> On 2017/04/25 22:20:05, Jinsuk Kim wrote:
> > On 2017/04/25 17:57:39, Oliver Chang wrote:
> > > PTAL, thanks!
> > 
> > Thanks for the change. Would you let me know where the policy for this
> 'Security
> > Critical' bit is documented? Not easy for me to find it.
> 
> It's here in the README.chromium template:
> https://cs.chromium.org/chromium/src/third_party/README.chromium.template?l=9

lgtm. Thanks.

[Oliver Chang, 2017-04-25 22:45:43]

The CQ bit was checked by ochang@chromium.org

[commit-bot: I haz the power, 2017-04-25 22:46: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, 2017-04-25 22:59:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493160343151130, "parent_rev":
"56bf32f77a418628dd04bc68f4ef8400a3cf0480", "commit_rev":
"c2f72fc86077822222610a0ed35ef068a010733a"}

[commit-bot: I haz the power, 2017-04-25 23:00:07]

Description was changed from

==========
Correct security critical bit for third_party/ced

This library is shipped, so should be marked as "Security Critical".

R=jinsukkim@chromium.org,jshin@chromium.org
==========

to

==========
Correct security critical bit for third_party/ced

This library is shipped, so should be marked as "Security Critical".

R=jinsukkim@chromium.org,jshin@chromium.org

Review-Url: https://codereview.chromium.org/2842813002
Cr-Commit-Position: refs/heads/master@{#467155}
Committed:
https://chromium.googlesource.com/chromium/src/+/c2f72fc86077822222610a0ed35e...
==========

[commit-bot: I haz the power, 2017-04-25 23:00:13]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c2f72fc86077822222610a0ed35e...