Remove net::IPAddressNumber.

All consumers have been migrated to net::IPAddress over the last months so it is now safe to remove the net::IPAddressNumber type. This also removes the IPAddress length constants and switches consumers to the (identically named) net::IPAddress member constants or the IsIPv*() helpers where possible.

The other changes are mostly removals of leftover #includes.

BUG=496258
Committed: https://crrev.com/56506d0a3226db78d2a905a4bc5ff6b222469e1d
Cr-Commit-Position: refs/heads/master@{#391310}

[martijnc, 2016-05-01 18:03:47]

The CQ bit was checked by martijn@martijnc.be to run a CQ dry run

[martijnc, 2016-05-01 20:20:39]

The CQ bit was unchecked by martijn@martijnc.be

[martijnc, 2016-05-01 20:20:45]

The CQ bit was checked by martijn@martijnc.be to run a CQ dry run

[commit-bot: I haz the power, 2016-05-01 21:49:14]

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

[commit-bot: I haz the power, 2016-05-01 22:58:44]

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

[commit-bot: I haz the power, 2016-05-01 22:58:45]

Dry run: This issue passed the CQ dry run.

[martijnc, 2016-05-02 17:39:49]

Description was changed from

==========
Remove net::IPAddressNumber.

BUG=496258
==========

to

==========
All consumers have been migrated to net::IPAddress over the last months so it is
now safe to remove the net::IPAddressNumber type. This also removes the
IPAddress length constants and switches consumers to the (identically named)
net::IPAddress member constants or the IsIPv*() helpers where possible.

The other changes are mostly removals of leftover #includes.

BUG=496258
==========

[martijnc, 2016-05-02 17:39:49]

martijn@martijnc.be changed reviewers:
+ eroman@chromium.org

[martijnc, 2016-05-02 17:45:32]

It took a little longer than I expected but net::IPAddressNumber is (almost)
gone :)

I was able to build all targets on Linux and all trybots seem to pass.

Can you review?

Thank you for the reviews, your feedback throughout the migration has been very
helpful!

[eroman, 2016-05-02 19:42:56]

LGTM!

Thanks for completing this migration so thoughtfully!

[martijnc, 2016-05-02 21:02:38]

martijn@martijnc.be changed reviewers:
+ nparker@chromium.org, rdevlin.cronin@chromium.org

[martijnc, 2016-05-02 21:06:11]

+nparker for the extensions change.
+rdevlin.cronin for the safe_browsing change.

Can you take a look? Thanks!

[Devlin, 2016-05-02 21:30:10]

On 2016/05/02 21:06:11, martijnc wrote:
> +nparker for the extensions change.
> +rdevlin.cronin for the safe_browsing change.
> 
> Can you take a look? Thanks!

I think these are supposed to be switched? (nparker owns safe browsing, I own
extensions.)

Extensions lgtm.

[Nathan Parker, 2016-05-02 21:41:41]

lgtm

for safe_browsing

[martijnc, 2016-05-03 17:28:33]

On 2016/05/02 at 21:30:10, rdevlin.cronin wrote:
> On 2016/05/02 21:06:11, martijnc wrote:
> > +nparker for the extensions change.
> > +rdevlin.cronin for the safe_browsing change.
> > 
> > Can you take a look? Thanks!
> 
> I think these are supposed to be switched? (nparker owns safe browsing, I own
extensions.)
> 
> Extensions lgtm.

Yes, sorry, I accidentally switched those. It was indeed my intention to add you
for extensions and nparker for safe browsing.

[martijnc, 2016-05-03 17:28:49]

The CQ bit was checked by martijn@martijnc.be

[commit-bot: I haz the power, 2016-05-03 17:29:12]

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

[commit-bot: I haz the power, 2016-05-03 18:44:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-03 18:46:39]

Description was changed from

==========
All consumers have been migrated to net::IPAddress over the last months so it is
now safe to remove the net::IPAddressNumber type. This also removes the
IPAddress length constants and switches consumers to the (identically named)
net::IPAddress member constants or the IsIPv*() helpers where possible.

The other changes are mostly removals of leftover #includes.

BUG=496258
==========

to

==========
All consumers have been migrated to net::IPAddress over the last months so it is
now safe to remove the net::IPAddressNumber type. This also removes the
IPAddress length constants and switches consumers to the (identically named)
net::IPAddress member constants or the IsIPv*() helpers where possible.

The other changes are mostly removals of leftover #includes.

BUG=496258

Committed: https://crrev.com/56506d0a3226db78d2a905a4bc5ff6b222469e1d
Cr-Commit-Position: refs/heads/master@{#391310}
==========

[commit-bot: I haz the power, 2016-05-03 18:46:40]

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