Revert of [WeakMemoryCache] Remove Resource from MemoryCache just before destructed

Revert of [WeakMemoryCache] Remove Resource from MemoryCache just before destructed (patchset #3 id:40001 of https://codereview.chromium.org/2066763002/ )

Reason for revert:
Reverting field-trial CLs before going to beta, as planned.

Original issue's description:
> [WeakMemoryCache] Remove Resource from MemoryCache just before destructed
>
> Previously, if Weak MemoryCache is enabled, Resource could be silently
> destructed and MemoryCache's live/dead sizes were not updated.
> This may cause larger |m_live/deadSize| and more aggressive pruning.
>
> This CL calls MemoryCache::remove() just before a Resource is destructed
> to make the MemoryCache sizes correct.
>
> This CL should be reverted before going to beta.
>
> BUG=603462
>
> Committed: https://crrev.com/4706b08cb45742e3b0dcc73ba443522d0cce28da
> Cr-Commit-Position: refs/heads/master@{#401536}

TBR=japhet@chromium.org,yhirano@chromium.org,oilpan-reviews@chromium.org,haraken@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=603462
Committed: https://crrev.com/bb84af54556198947087f4ec33209e5c4ea5bbc3
Cr-Commit-Position: refs/heads/master@{#403087}

[hiroshige, 2016-06-30 02:35:20]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2016-06-30 02:35:20]

Created Revert of [WeakMemoryCache] Remove Resource from MemoryCache just before
destructed

[commit-bot: I haz the power, 2016-06-30 02:35:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[haraken, 2016-06-30 02:35:45]

LGTM

[commit-bot: I haz the power, 2016-06-30 03:19:07]

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

[commit-bot: I haz the power, 2016-06-30 03:19:08]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[hiroshige, 2016-06-30 03:30:46]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2016-06-30 03:31:58]

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-06-30 04:29:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-30 04:29:56]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 04:31:31]

Description was changed from

==========
Revert of [WeakMemoryCache] Remove Resource from MemoryCache just before
destructed (patchset #3 id:40001 of https://codereview.chromium.org/2066763002/
)

Reason for revert:
Reverting field-trial CLs before going to beta, as planned.

Original issue's description:
> [WeakMemoryCache] Remove Resource from MemoryCache just before destructed
> 
> Previously, if Weak MemoryCache is enabled, Resource could be silently
> destructed and MemoryCache's live/dead sizes were not updated.
> This may cause larger |m_live/deadSize| and more aggressive pruning.
> 
> This CL calls MemoryCache::remove() just before a Resource is destructed
> to make the MemoryCache sizes correct.
> 
> This CL should be reverted before going to beta.
> 
> BUG=603462
> 
> Committed: https://crrev.com/4706b08cb45742e3b0dcc73ba443522d0cce28da
> Cr-Commit-Position: refs/heads/master@{#401536}

TBR=japhet@chromium.org,yhirano@chromium.org,oilpan-reviews@chromium.org,hara...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=603462
==========

to

==========
Revert of [WeakMemoryCache] Remove Resource from MemoryCache just before
destructed (patchset #3 id:40001 of https://codereview.chromium.org/2066763002/
)

Reason for revert:
Reverting field-trial CLs before going to beta, as planned.

Original issue's description:
> [WeakMemoryCache] Remove Resource from MemoryCache just before destructed
>
> Previously, if Weak MemoryCache is enabled, Resource could be silently
> destructed and MemoryCache's live/dead sizes were not updated.
> This may cause larger |m_live/deadSize| and more aggressive pruning.
>
> This CL calls MemoryCache::remove() just before a Resource is destructed
> to make the MemoryCache sizes correct.
>
> This CL should be reverted before going to beta.
>
> BUG=603462
>
> Committed: https://crrev.com/4706b08cb45742e3b0dcc73ba443522d0cce28da
> Cr-Commit-Position: refs/heads/master@{#401536}

TBR=japhet@chromium.org,yhirano@chromium.org,oilpan-reviews@chromium.org,hara...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=603462

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

[commit-bot: I haz the power, 2016-06-30 04:31:32]

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