Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Due to undefined (favorably right to left) argument evaluation order,
the tracker might have been passed and set to NULL before the window
is looked up which results in a NULL pointer dereference.

BUG=

Committed: https://crrev.com/1b4c64279b7c212242738b96643a0700d04d725c
Cr-Commit-Position: refs/heads/master@{#426748}

[jb, 2016-10-20 13:43:25]

Description was changed from

==========
Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Due to undefined (favorably right to left) argument evaluation order,
the tracker might have been passed and set to NULL before the window
is looked up which results in a NULL pointer dereference.

BUG=
==========

to

==========
Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Due to undefined (favorably right to left) argument evaluation order,
the tracker might have been passed and set to NULL before the window
is looked up which results in a NULL pointer dereference.

BUG=
==========

[jb, 2016-10-20 13:43:25]

jb@opera.com changed reviewers:
+ jbauman@chromium.org, sadrul@chromium.org

[jb, 2016-10-20 13:44:03]

PTAL

[sadrul, 2016-10-20 16:42:19]

lgtm

[jbauman, 2016-10-20 19:45:54]

lgtm

[jb, 2016-10-21 06:41:13]

The CQ bit was checked by jb@opera.com

[commit-bot: I haz the power, 2016-10-21 06:41:31]

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-10-21 07:56:21]

Description was changed from

==========
Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Due to undefined (favorably right to left) argument evaluation order,
the tracker might have been passed and set to NULL before the window
is looked up which results in a NULL pointer dereference.

BUG=
==========

to

==========
Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Due to undefined (favorably right to left) argument evaluation order,
the tracker might have been passed and set to NULL before the window
is looked up which results in a NULL pointer dereference.

BUG=
==========

[commit-bot: I haz the power, 2016-10-21 07:56:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-21 13:28:05]

Description was changed from

==========
Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Due to undefined (favorably right to left) argument evaluation order,
the tracker might have been passed and set to NULL before the window
is looked up which results in a NULL pointer dereference.

BUG=
==========

to

==========
Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Due to undefined (favorably right to left) argument evaluation order,
the tracker might have been passed and set to NULL before the window
is looked up which results in a NULL pointer dereference.

BUG=

Committed: https://crrev.com/1b4c64279b7c212242738b96643a0700d04d725c
Cr-Commit-Position: refs/heads/master@{#426748}
==========

[commit-bot: I haz the power, 2016-10-21 13:28:06]

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