Fix nits from earlier CLs

Fix nits from earlier CLs

BUG=None

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

[rbpotter, 2017-02-27 17:23:58]

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

[rbpotter, 2017-02-27 17:24:06]

Description was changed from

==========
Fix nits from earlier CLs

BUG=None
==========

to

==========
Fix nits from earlier CLs

BUG=None
==========

[rbpotter, 2017-02-27 17:24:07]

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

[commit-bot: I haz the power, 2017-02-27 17:24:18]

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

[rbpotter, 2017-02-27 17:25:11]

Fixes for comments from earlier CLs:

https://codereview.chromium.org/2524143003/

https://codereview.chromium.org/2616783002/

https://codereview.chromium.org/2683653005/

[commit-bot: I haz the power, 2017-02-27 18:49:18]

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

[commit-bot: I haz the power, 2017-02-27 18:49:19]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2017-02-27 19:50:22]

lgtm, but you need a security reviewer for the trivial IPC message change.

[rbpotter, 2017-02-27 19:56:45]

rbpotter@chromium.org changed reviewers:
+ tsepez@chromium.org

[Tom Sepez, 2017-02-27 20:20:05]

trivial LGTM

[Lei Zhang, 2017-02-27 20:22:06]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2017-02-27 20:22:40]

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-02-27 20:34:05]

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

[commit-bot: I haz the power, 2017-02-27 20:34:06]

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

[rbpotter, 2017-02-27 20:34:56]

rbpotter@chromium.org changed reviewers:
+ vollick@chromium.org

[rbpotter, 2017-02-27 20:35:19]

Changed ordering of DEPS to alphabetical.

[rbpotter, 2017-03-09 17:47:43]

rbpotter@chromium.org changed reviewers:
+ dcheng@chromium.org
- vollick@chromium.org

[rbpotter, 2017-03-09 17:48:46]

Need review from ui/gfx owner as I changed the ordering
in a DEPS file to be alphabetical and it includes files
from this folder.

[Tom Sepez, 2017-03-09 17:50:52]

RS LGTM

[dcheng, 2017-03-09 19:12:46]

Weird, we should fix the presubmit to not complain about this, but lgtm in the
meantime

[dcheng, 2017-03-09 19:17:20]

Filed https://bugs.chromium.org/p/chromium/issues/detail?id=700108 for the
presubmit check.

[rbpotter, 2017-03-09 20:02:35]

The CQ bit was checked by rbpotter@chromium.org

[commit-bot: I haz the power, 2017-03-09 20:03:08]

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-09 20:12:08]

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

[commit-bot: I haz the power, 2017-03-09 20:12:09]

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

[rbpotter, 2017-03-09 20:19:09]

rbpotter@chromium.org changed reviewers:
+ danakj@chromium.org

[rbpotter, 2017-03-09 20:20:09]

+danakj for ui/gfx/geometry...

CL changes ordering a in DEPS file.

[danakj, 2017-03-09 20:49:05]

https://codereview.chromium.org/2718123002/diff/1/pdf/pdfium/DEPS
File pdf/pdfium/DEPS (right):

https://codereview.chromium.org/2718123002/diff/1/pdf/pdfium/DEPS#newcode7
pdf/pdfium/DEPS:7: "+ui/gfx/codec/jpeg_codec.h",
LGTM

[rbpotter, 2017-03-09 21:27:36]

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

[commit-bot: I haz the power, 2017-03-09 21:28:05]

Dry run: 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-09 22:10:05]

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

[commit-bot: I haz the power, 2017-03-09 22:10:06]

Dry run: This issue passed the CQ dry run.

[rbpotter, 2017-03-09 22:10:33]

The CQ bit was checked by rbpotter@chromium.org

[commit-bot: I haz the power, 2017-03-09 22:10:59]

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-09 22:17:03]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489097433779700, "parent_rev":
"1372b49c583067437e3e42574fd55199bd7d5a28", "commit_rev":
"ea6067deecab4ff2a010bcb01e9bfb76fe7a88bf"}

[commit-bot: I haz the power, 2017-03-09 22:17:44]

Description was changed from

==========
Fix nits from earlier CLs

BUG=None
==========

to

==========
Fix nits from earlier CLs

BUG=None

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

[commit-bot: I haz the power, 2017-03-09 22:17:46]

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