Revert of Fix null dereference on MemoryCache.

Revert of Fix null dereference on MemoryCache. (patchset #4 id:280001 of https://codereview.chromium.org/1537343002/ )

Reason for revert:
The change breaks the oilpan build.

Original issue's description:
> Fix null dereference on MemoryCache.
>
> Usually a valid MemoryCacheEntry holds a non-null Resource as |m_resource|. But
> when we hold a valid MemoryCacheEntry beyond destructive statements, it may be
> evicted from the cache and get stale. That means |m_resource| can be null in
> such cases. This CL checks it in order to avoid null dereference.
>
> BUG=488373
>
> Committed: https://crrev.com/5de01d904d77ba5e5beaf4776e45e6eecd47a87b
> Cr-Commit-Position: refs/heads/master@{#367779}

TBR=japhet@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=488373
Committed: https://crrev.com/58c142c6eccd0287edcf0194391f785e45a8de92
Cr-Commit-Position: refs/heads/master@{#367788}

[yhirano, 2016-01-06 06:51:11]

Created Revert of Fix null dereference on MemoryCache.

[commit-bot: I haz the power, 2016-01-06 06:51:28]

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

[yhirano, 2016-01-06 06:52:14]

The CQ bit was unchecked by yhirano@chromium.org

[commit-bot: I haz the power, 2016-01-06 06:52:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-06 06:53:22]

Description was changed from

==========
Revert of Fix null dereference on MemoryCache. (patchset #4 id:280001 of
https://codereview.chromium.org/1537343002/ )

Reason for revert:
The change breaks the oilpan build.

Original issue's description:
> Fix null dereference on MemoryCache.
> 
> Usually a valid MemoryCacheEntry holds a non-null Resource as |m_resource|.
But
> when we hold a valid MemoryCacheEntry beyond destructive statements, it may be
> evicted from the cache and get stale. That means |m_resource| can be null in
> such cases. This CL checks it in order to avoid null dereference.
> 
> BUG=488373
> 
> Committed: https://crrev.com/5de01d904d77ba5e5beaf4776e45e6eecd47a87b
> Cr-Commit-Position: refs/heads/master@{#367779}

TBR=japhet@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=488373
==========

to

==========
Revert of Fix null dereference on MemoryCache. (patchset #4 id:280001 of
https://codereview.chromium.org/1537343002/ )

Reason for revert:
The change breaks the oilpan build.

Original issue's description:
> Fix null dereference on MemoryCache.
>
> Usually a valid MemoryCacheEntry holds a non-null Resource as |m_resource|.
But
> when we hold a valid MemoryCacheEntry beyond destructive statements, it may be
> evicted from the cache and get stale. That means |m_resource| can be null in
> such cases. This CL checks it in order to avoid null dereference.
>
> BUG=488373
>
> Committed: https://crrev.com/5de01d904d77ba5e5beaf4776e45e6eecd47a87b
> Cr-Commit-Position: refs/heads/master@{#367779}

TBR=japhet@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=488373

Committed: https://crrev.com/58c142c6eccd0287edcf0194391f785e45a8de92
Cr-Commit-Position: refs/heads/master@{#367788}
==========

[commit-bot: I haz the power, 2016-01-06 06:53:24]

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

[sof, 2016-01-06 07:07:26]

sigbjornf@opera.com changed reviewers:
+ sigbjornf@opera.com

[sof, 2016-01-06 07:07:27]

'twas fixed by https://crrev.com/367787

[yhirano, 2016-01-06 07:11:14]

On 2016/01/06 07:07:27, sof wrote:
> 'twas fixed by https://crrev.com/367787

I'm sorry for the mess.

[sof, 2016-01-06 07:14:39]

Description was changed from

==========
Revert of Fix null dereference on MemoryCache. (patchset #4 id:280001 of
https://codereview.chromium.org/1537343002/ )

Reason for revert:
The change breaks the oilpan build.

Original issue's description:
> Fix null dereference on MemoryCache.
>
> Usually a valid MemoryCacheEntry holds a non-null Resource as |m_resource|.
But
> when we hold a valid MemoryCacheEntry beyond destructive statements, it may be
> evicted from the cache and get stale. That means |m_resource| can be null in
> such cases. This CL checks it in order to avoid null dereference.
>
> BUG=488373
>
> Committed: https://crrev.com/5de01d904d77ba5e5beaf4776e45e6eecd47a87b
> Cr-Commit-Position: refs/heads/master@{#367779}

TBR=japhet@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=488373

Committed: https://crrev.com/58c142c6eccd0287edcf0194391f785e45a8de92
Cr-Commit-Position: refs/heads/master@{#367788}
==========

to

==========
Revert of Fix null dereference on MemoryCache. (patchset #4 id:280001 of
https://codereview.chromium.org/1537343002/ )

Reason for revert:
The change breaks the oilpan build.

Original issue's description:
> Fix null dereference on MemoryCache.
>
> Usually a valid MemoryCacheEntry holds a non-null Resource as |m_resource|.
But
> when we hold a valid MemoryCacheEntry beyond destructive statements, it may be
> evicted from the cache and get stale. That means |m_resource| can be null in
> such cases. This CL checks it in order to avoid null dereference.
>
> BUG=488373
>
> Committed: https://crrev.com/5de01d904d77ba5e5beaf4776e45e6eecd47a87b
> Cr-Commit-Position: refs/heads/master@{#367779}

TBR=japhet@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=488373

Committed: https://crrev.com/58c142c6eccd0287edcf0194391f785e45a8de92
Cr-Commit-Position: refs/heads/master@{#367788}
==========

[sof, 2016-01-06 07:17:58]

On 2016/01/06 07:11:14, yhirano wrote:
> On 2016/01/06 07:07:27, sof wrote:
> > 'twas fixed by https://crrev.com/367787
> 
> I'm sorry for the mess.

no worries here; an easy issue to overlook & the CQ isn't of much help.