Replace if-block on !redirectResponse.isNull() with ASSERT in RawResource::willFollowRedirect()

Replace if-block on !redirectResponse.isNull() with ASSERT in RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
R=mkwst

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

[tyoshino (SeeGerritForStatus), 2016-01-26 08:24:38]

Description was changed from

==========
Replace if-block on !redirectResponse.isNull() with ASSERT in
RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
==========

to

==========
Replace if-block on !redirectResponse.isNull() with ASSERT in
RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
==========

[tyoshino (SeeGerritForStatus), 2016-01-26 08:24:38]

tyoshino@chromium.org changed reviewers:
+ japhet@chromium.org

[tyoshino (SeeGerritForStatus), 2016-01-26 08:24:43]

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

[commit-bot: I haz the power, 2016-01-26 08:24:50]

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

[commit-bot: I haz the power, 2016-01-26 09:40:42]

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

[commit-bot: I haz the power, 2016-01-26 09:40:43]

Dry run: This issue passed the CQ dry run.

[Mike West, 2016-01-26 09:56:22]

On 2016/01/26 at 09:40:43, commit-bot wrote:
> Dry run: This issue passed the CQ dry run.

LGTM.

[tyoshino (SeeGerritForStatus), 2016-01-26 16:12:52]

The CQ bit was checked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2016-01-26 16:12:54]

The CQ bit was unchecked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2016-01-26 16:13:02]

Description was changed from

==========
Replace if-block on !redirectResponse.isNull() with ASSERT in
RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
==========

to

==========
Replace if-block on !redirectResponse.isNull() with ASSERT in
RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
R=mkwst
==========

[tyoshino (SeeGerritForStatus), 2016-01-27 03:25:57]

The CQ bit was checked by tyoshino@chromium.org

[commit-bot: I haz the power, 2016-01-27 03:26:32]

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

[commit-bot: I haz the power, 2016-01-27 03:37:13]

Description was changed from

==========
Replace if-block on !redirectResponse.isNull() with ASSERT in
RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
R=mkwst
==========

to

==========
Replace if-block on !redirectResponse.isNull() with ASSERT in
RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
R=mkwst
==========

[commit-bot: I haz the power, 2016-01-27 03:37:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-27 03:38:38]

Description was changed from

==========
Replace if-block on !redirectResponse.isNull() with ASSERT in
RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
R=mkwst
==========

to

==========
Replace if-block on !redirectResponse.isNull() with ASSERT in
RawResource::willFollowRedirect()

The only caller ResourceLoader::willFollowRedirect() has ASSERT on
redirectResponse. So, we can assert that it's not null here too.

BUG=none
R=mkwst

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

[commit-bot: I haz the power, 2016-01-27 03:38:38]

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