Fix crash when following a link opening a new tab/window.

Fix crash when following a link opening a new tab/window.

Issue http://crrev.com/1408643002 removed the check for NULL entry
when accessing the virtual URL. It is possible to end up with such
an entry when following a link opening a new tab/window. Fix the
crash by checking that GetPossiblyPendingEntryAtIndex() returns a
non-NULL entry.

BUG=548572
Committed: https://crrev.com/bf61f1832c367260d70d96a92fefc31fd6f987e9
Cr-Commit-Position: refs/heads/master@{#356579}

[sdefresne, 2015-10-28 16:37:49]

sdefresne@chromium.org changed reviewers:
+ zea@chromium.org

[sdefresne, 2015-10-28 16:37:49]

I think this should be enough as this is the only code path that checked entry
was NULL or not before accessing it. WDYT?

[Nicolas Zea, 2015-10-28 16:54:16]

lgtm

[Nicolas Zea, 2015-10-28 16:55:21]

Thanks for fixing this so quick btw!

[Nicolas Zea, 2015-10-28 16:55:56]

On 2015/10/28 16:55:21, Nicolas Zea wrote:
> Thanks for fixing this so quick btw!

I'll try to add a regression test for this in a separate patch.

[Nicolas Zea, 2015-10-28 16:57:35]

The CQ bit was checked by zea@chromium.org

[commit-bot: I haz the power, 2015-10-28 16:58:02]

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

[commit-bot: I haz the power, 2015-10-28 17:57:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-28 17:58:22]

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