Revert of cc::ResourcePool - Re-use larger resources for smaller requests

Revert of cc::ResourcePool - Re-use larger resources for smaller requests (patchset #4 id:140001 of https://codereview.chromium.org/2726263003/ )

Reason for revert:
This seems to cause flakiness of several border-radius tests, e.g. https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_tests&tests=compositing%2Foverflow%2Fgrandchild-composited-with-border-radius-ancestor.html

Speculatively revert, to see if it's this CL.

Original issue's description:
> cc::ResourcePool - Re-use larger resources for smaller requests
> 
> Allows cc::ResourcePool to re-use larger resources to fulfill a smaller
> request. This avoids expensive allocations, and is especially useful in
> cases where layer sizes are changing due to animation.
> 
> Also extends the timeout for resource expiration from 1s to 5s, which
> will also cut down on resource allocations.
> 
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2726263003
> Cr-Commit-Position: refs/heads/master@{#456202}
> Committed: https://chromium.googlesource.com/chromium/src/+/830cb220146684c38b9aa4da3f39e96c37a59f33

TBR=enne@chromium.org,danakj@chromium.org,ericrk@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review-Url: https://codereview.chromium.org/2744063002
Cr-Commit-Position: refs/heads/master@{#456257}
Committed: https://chromium.googlesource.com/chromium/src/+/085571c2c97e2915a1e40262f6d1aaf20e9defb7

[Xianzhu, 2017-03-11 02:16:41]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2017-03-11 02:16:42]

Created Revert of cc::ResourcePool - Re-use larger resources for smaller
requests

[commit-bot: I haz the power, 2017-03-11 02:16:55]

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, 2017-03-11 02:17:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489198601554490, "parent_rev":
"1111902f947ac92cb21d6ed443e2c607337308c1", "commit_rev":
"085571c2c97e2915a1e40262f6d1aaf20e9defb7"}

[commit-bot: I haz the power, 2017-03-11 02:18:12]

Description was changed from

==========
Revert of cc::ResourcePool - Re-use larger resources for smaller requests
(patchset #4 id:140001 of https://codereview.chromium.org/2726263003/ )

Reason for revert:
This seems to cause flakiness of several border-radius tests, e.g.
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType...

Speculatively revert, to see if it's this CL.

Original issue's description:
> cc::ResourcePool - Re-use larger resources for smaller requests
> 
> Allows cc::ResourcePool to re-use larger resources to fulfill a smaller
> request. This avoids expensive allocations, and is especially useful in
> cases where layer sizes are changing due to animation.
> 
> Also extends the timeout for resource expiration from 1s to 5s, which
> will also cut down on resource allocations.
> 
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2726263003
> Cr-Commit-Position: refs/heads/master@{#456202}
> Committed:
https://chromium.googlesource.com/chromium/src/+/830cb220146684c38b9aa4da3f39...

TBR=enne@chromium.org,danakj@chromium.org,ericrk@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of cc::ResourcePool - Re-use larger resources for smaller requests
(patchset #4 id:140001 of https://codereview.chromium.org/2726263003/ )

Reason for revert:
This seems to cause flakiness of several border-radius tests, e.g.
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType...

Speculatively revert, to see if it's this CL.

Original issue's description:
> cc::ResourcePool - Re-use larger resources for smaller requests
> 
> Allows cc::ResourcePool to re-use larger resources to fulfill a smaller
> request. This avoids expensive allocations, and is especially useful in
> cases where layer sizes are changing due to animation.
> 
> Also extends the timeout for resource expiration from 1s to 5s, which
> will also cut down on resource allocations.
> 
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2726263003
> Cr-Commit-Position: refs/heads/master@{#456202}
> Committed:
https://chromium.googlesource.com/chromium/src/+/830cb220146684c38b9aa4da3f39...

TBR=enne@chromium.org,danakj@chromium.org,ericrk@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review-Url: https://codereview.chromium.org/2744063002
Cr-Commit-Position: refs/heads/master@{#456257}
Committed:
https://chromium.googlesource.com/chromium/src/+/085571c2c97e2915a1e40262f6d1...
==========

[commit-bot: I haz the power, 2017-03-11 02:18:13]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/085571c2c97e2915a1e40262f6d1...

[Xianzhu, 2017-03-11 04:34:20]

The tests are no longer flaky after this revert:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType...