Document the fact that leveldb sign warnings aren't getting fixed.

Document the fact that leveldb sign warnings aren't getting fixed.

BUG=698013
Review-Url: https://codereview.chromium.org/2788663003
Cr-Commit-Position: refs/heads/master@{#460932}
Committed: https://chromium.googlesource.com/chromium/src/+/022e76c4e2597f27c442eaecacaabdc708480ff2

[pwnall, 2017-03-30 18:13:23]

The CQ bit was checked by pwnall@chromium.org to run a CQ dry run

[pwnall, 2017-03-30 18:13:38]

Description was changed from

==========
Document the fact that leveldb sign warnings aren't getting fixes.

BUG=698013
==========

to

==========
Document the fact that leveldb sign warnings aren't getting fixed.

BUG=698013
==========

[pwnall, 2017-03-30 18:13:55]

pwnall@chromium.org changed reviewers:
+ cmumford@chromium.org

[pwnall, 2017-03-30 18:13:56]

PTAL?

[commit-bot: I haz the power, 2017-03-30 18:14:19]

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

[cmumford, 2017-03-30 18:15:32]

lgtm % typo

https://codereview.chromium.org/2788663003/diff/1/third_party/leveldatabase/B...
File third_party/leveldatabase/BUILD.gn (right):

https://codereview.chromium.org/2788663003/diff/1/third_party/leveldatabase/B...
third_party/leveldatabase/BUILD.gn:134: # leveldb compiles with a limiterd set
of warnings enabled, and is not
typo.

[pwnall, 2017-03-30 20:38:35]

Thank you for the quick review!

https://codereview.chromium.org/2788663003/diff/1/third_party/leveldatabase/B...
File third_party/leveldatabase/BUILD.gn (right):

https://codereview.chromium.org/2788663003/diff/1/third_party/leveldatabase/B...
third_party/leveldatabase/BUILD.gn:134: # leveldb compiles with a limiterd set
of warnings enabled, and is not
On 2017/03/30 18:15:32, cmumford wrote:
> typo.

Done.
Thank you very much for catching this!

[pwnall, 2017-03-30 20:38:39]

The CQ bit was checked by pwnall@chromium.org

[pwnall, 2017-03-30 20:38:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
cmumford@chromium.org
Link to the patchset: https://codereview.chromium.org/2788663003/#ps20001
(title: "Typo fix.")

[commit-bot: I haz the power, 2017-03-30 20:39:41]

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-03-30 23:42:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490906319082800,
"parent_rev": "6970fa31e7037473ce71f9c2b870aca2a0952481", "commit_rev":
"022e76c4e2597f27c442eaecacaabdc708480ff2"}

[commit-bot: I haz the power, 2017-03-30 23:42:57]

Description was changed from

==========
Document the fact that leveldb sign warnings aren't getting fixed.

BUG=698013
==========

to

==========
Document the fact that leveldb sign warnings aren't getting fixed.

BUG=698013

Review-Url: https://codereview.chromium.org/2788663003
Cr-Commit-Position: refs/heads/master@{#460932}
Committed:
https://chromium.googlesource.com/chromium/src/+/022e76c4e2597f27c442eaecacaa...
==========

[commit-bot: I haz the power, 2017-03-30 23:43:02]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/022e76c4e2597f27c442eaecacaa...