Update DCHECK in WebRange constructor.

Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by
the comment it was intende to check null range only.

Committed: https://crrev.com/aa8d0ff140941ba9e830090421600f47e278dcb5
Cr-Commit-Position: refs/heads/master@{#421252}

[mharanczyk, 2016-09-27 12:57:29]

Description was changed from

==========
Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by
the comment it was intende to check null range only.
==========

to

==========
Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by the comment it was
intende to check null range only.
==========

[mharanczyk, 2016-09-27 12:57:30]

mharanczyk@opera.com changed reviewers:
+ dglazkov@chromium.org

[mharanczyk, 2016-09-27 12:57:52]

Description was changed from

==========
Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by the comment it was
intende to check null range only.
==========

to

==========
Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by
the comment it was intende to check null range only.
==========

[mharanczyk, 2016-09-27 12:59:17]

PTAL.

[dglazkov, 2016-09-27 16:13:52]

The CQ bit was checked by dglazkov@chromium.org

[dglazkov, 2016-09-27 16:13:52]

lgtm

[commit-bot: I haz the power, 2016-09-27 16:14:10]

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-09-27 17:34:10]

Description was changed from

==========
Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by
the comment it was intende to check null range only.
==========

to

==========
Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by
the comment it was intende to check null range only.
==========

[commit-bot: I haz the power, 2016-09-27 17:34:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 17:36:45]

Description was changed from

==========
Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by
the comment it was intende to check null range only.
==========

to

==========
Update DCHECK in WebRange constructor.

Currently creating empty range (0,0) triggers it, judging by
the comment it was intende to check null range only.

Committed: https://crrev.com/aa8d0ff140941ba9e830090421600f47e278dcb5
Cr-Commit-Position: refs/heads/master@{#421252}
==========

[commit-bot: I haz the power, 2016-09-27 17:36:46]

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

[dglazkov, 2016-09-27 17:44:24]

On 2016/09/27 at 12:59:17, mharanczyk wrote:
> PTAL.

Didn't want to block landing this patch, but could you add a unit test that
captures this?

[mharanczyk, 2016-09-28 13:07:33]

On 2016/09/27 17:44:24, dglazkov wrote:
> On 2016/09/27 at 12:59:17, mharanczyk wrote:
> > PTAL.
> 
> Didn't want to block landing this patch, but could you add a unit test that
> captures this?

Thank you for allowing it to land. I will add test soon, just please give me a
day or two.