Remove kuint8max.

Remove kuint8max.

BUG=138542
Committed: https://crrev.com/76490fdfedb1d68f2efd8701e5e70c8a4e3ab753
Cr-Commit-Position: refs/heads/master@{#362982}

[Avi (use Gerrit), 2015-11-25 15:53:18]

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

[commit-bot: I haz the power, 2015-11-25 15:53:40]

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

[Avi (use Gerrit), 2015-11-25 17:00:55]

avi@chromium.org changed reviewers:
+ dalecurtis@chromium.org, mark@chromium.org, mmenke@chromium.org,
stevenjb@chromium.org, zea@chromium.org

[Avi (use Gerrit), 2015-11-25 17:00:56]

mark: base
stevenjb: chrome/browser/chromeos, ui/app_list
dalecurtis: media/
mmenke: net/
zea: sync/

[Mark Mentovai, 2015-11-25 17:05:02]

LGTM

[stevenjb, 2015-11-25 17:10:43]

c/b/chromeos lgtm

https://codereview.chromium.org/1477433005/diff/1/chrome/browser/chromeos/dis...
File chrome/browser/chromeos/display/overscan_calibrator.cc (right):

https://codereview.chromium.org/1477433005/diff/1/chrome/browser/chromeos/dis...
chrome/browser/chromeos/display/overscan_calibrator.cc:8: 
nit: no WS here?

[mmenke, 2015-11-25 17:13:33]

net/ LGTM

[DaleCurtis, 2015-11-25 17:15:46]

media/ lgtm

[Avi (use Gerrit), 2015-11-25 17:17:02]

https://codereview.chromium.org/1477433005/diff/1/chrome/browser/chromeos/dis...
File chrome/browser/chromeos/display/overscan_calibrator.cc (right):

https://codereview.chromium.org/1477433005/diff/1/chrome/browser/chromeos/dis...
chrome/browser/chromeos/display/overscan_calibrator.cc:8: 
On 2015/11/25 17:10:43, stevenjb wrote:
> nit: no WS here?

If you combine the C and C++ include block, presubmit complains.

[commit-bot: I haz the power, 2015-11-25 19:22:05]

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

[commit-bot: I haz the power, 2015-11-25 19:22:06]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[Avi (use Gerrit), 2015-11-25 21:34:40]

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

[commit-bot: I haz the power, 2015-11-25 21:37:38]

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

[commit-bot: I haz the power, 2015-11-25 23:43:25]

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

[commit-bot: I haz the power, 2015-11-25 23:43:27]

Dry run: Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_TIMED_OUT, no build
URL)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)

[Nicolas Zea, 2015-11-30 17:58:41]

sync LGTM with some nits

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
File sync/internal_api/public/base/ordinal_unittest.cc (right):

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
sync/internal_api/public/base/ordinal_unittest.cc:7: #include <stdint.h>
nit: move down a line and into abc order (other std librs)

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
File sync/internal_api/public/base/unique_position.h (right):

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
sync/internal_api/public/base/unique_position.h:8: #include <stdint.h>
down a line

[Nicolas Zea, 2015-11-30 17:58:41]

sync LGTM with some nits

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
File sync/internal_api/public/base/ordinal_unittest.cc (right):

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
sync/internal_api/public/base/ordinal_unittest.cc:7: #include <stdint.h>
nit: move down a line and into abc order (other std librs)

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
File sync/internal_api/public/base/unique_position.h (right):

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
sync/internal_api/public/base/unique_position.h:8: #include <stdint.h>
down a line

[Avi (use Gerrit), 2015-11-30 18:07:08]

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
File sync/internal_api/public/base/ordinal_unittest.cc (right):

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
sync/internal_api/public/base/ordinal_unittest.cc:7: #include <stdint.h>
On 2015/11/30 17:58:41, Nicolas Zea wrote:
> nit: move down a line and into abc order (other std librs)

If I do this, the presubmit complains.

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
File sync/internal_api/public/base/unique_position.h (right):

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
sync/internal_api/public/base/unique_position.h:8: #include <stdint.h>
On 2015/11/30 17:58:41, Nicolas Zea wrote:
> down a line

Ditto. If I do this, the presubmit complains.

[mmenke, 2015-11-30 18:09:52]

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
File sync/internal_api/public/base/ordinal_unittest.cc (right):

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
sync/internal_api/public/base/ordinal_unittest.cc:7: #include <stdint.h>
On 2015/11/30 18:07:08, Avi wrote:
> On 2015/11/30 17:58:41, Nicolas Zea wrote:
> > nit: move down a line and into abc order (other std librs)
> 
> If I do this, the presubmit complains.

Indeed. Style guide is standard C headers before standard C++ ones, with a line
break between.

[Nicolas Zea, 2015-11-30 23:59:58]

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
File sync/internal_api/public/base/ordinal_unittest.cc (right):

https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
sync/internal_api/public/base/ordinal_unittest.cc:7: #include <stdint.h>
On 2015/11/30 18:09:52, mmenke wrote:
> On 2015/11/30 18:07:08, Avi wrote:
> > On 2015/11/30 17:58:41, Nicolas Zea wrote:
> > > nit: move down a line and into abc order (other std librs)
> > 
> > If I do this, the presubmit complains.
> 
> Indeed. Style guide is standard C headers before standard C++ ones, with a
line
> break between.

Ah, didn't realize that. Disregard then.

[Avi (use Gerrit), 2015-12-01 14:59:07]

On 2015/11/30 23:59:58, Nicolas Zea wrote:
>
https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
> File sync/internal_api/public/base/ordinal_unittest.cc (right):
> 
>
https://codereview.chromium.org/1477433005/diff/20001/sync/internal_api/publi...
> sync/internal_api/public/base/ordinal_unittest.cc:7: #include <stdint.h>
> On 2015/11/30 18:09:52, mmenke wrote:
> > On 2015/11/30 18:07:08, Avi wrote:
> > > On 2015/11/30 17:58:41, Nicolas Zea wrote:
> > > > nit: move down a line and into abc order (other std librs)
> > > 
> > > If I do this, the presubmit complains.
> > 
> > Indeed. Style guide is standard C headers before standard C++ ones, with a
> line
> > break between.
> 
> Ah, didn't realize that. Disregard then.

No worries, mate. I didn't know that either until the presubmit bopped me on the
head.

[Avi (use Gerrit), 2015-12-02 00:32:26]

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

[commit-bot: I haz the power, 2015-12-02 00:33:09]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1477433005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1477433005/40001

[commit-bot: I haz the power, 2015-12-02 02:52:46]

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

[commit-bot: I haz the power, 2015-12-02 02:52:47]

Dry run: This issue passed the CQ dry run.

[Avi (use Gerrit), 2015-12-02 17:16:11]

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

[commit-bot: I haz the power, 2015-12-02 17:16:43]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1477433005/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1477433005/60001

[commit-bot: I haz the power, 2015-12-02 21:08:19]

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

[commit-bot: I haz the power, 2015-12-02 21:08:20]

Dry run: This issue passed the CQ dry run.

[Avi (use Gerrit), 2015-12-03 15:31:56]

The CQ bit was checked by avi@chromium.org

[Avi (use Gerrit), 2015-12-03 15:31:56]

The patchset sent to the CQ was uploaded after l-g-t-m from mark@chromium.org,
stevenjb@chromium.org, dalecurtis@chromium.org, mmenke@chromium.org,
zea@chromium.org
Link to the patchset: https://codereview.chromium.org/1477433005/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2015-12-03 15:34:20]

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

[commit-bot: I haz the power, 2015-12-03 15:41:33]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-12-03 15:42:42]

Description was changed from

==========
Remove kuint8max.

BUG=138542
==========

to

==========
Remove kuint8max.

BUG=138542

Committed: https://crrev.com/76490fdfedb1d68f2efd8701e5e70c8a4e3ab753
Cr-Commit-Position: refs/heads/master@{#362982}
==========

[commit-bot: I haz the power, 2015-12-03 15:42:43]

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