Revert of Assert that only 0-numbered objects are Released()

Revert of Assert that only 0-numbered objects are Released() (patchset #7 id:120001 of https://codereview.chromium.org/2375343004/ )

Reason for revert:
Broke PDFExtensionTest when rolling DEPS in Chromium.

Original issue's description:
> Assert that only 0-numbered objects are Released()
> 
> This condition holds because numbered objects are brute-force
> deleted by the indirect object holder, rather than being
> released.
> 
> Be careful about recursive deletion, check before advancing,
> since we no longer count on Release() doing this for us.
> Fix a few tests where the test was violating ownership rules.
> 
> This should be the last step before completely removing Release()
> in favor of direct delete everywhere.
> 
> Committed: https://pdfium.googlesource.com/pdfium/+/aba528a362248a54b27a7e9e046e2b65ab83f624

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

Committed: https://pdfium.googlesource.com/pdfium/+/b73c99335bfbd158ad16dd59c9c52396ffd2b54b

[Lei Zhang, 2016-10-04 04:27:56]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2016-10-04 04:27:57]

Created Revert of Assert that only 0-numbered objects are Released()

[commit-bot: I haz the power, 2016-10-04 04:28:02]

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-04 04:28:10]

Description was changed from

==========
Revert of Assert that only 0-numbered objects are Released() (patchset #7
id:120001 of https://codereview.chromium.org/2375343004/ )

Reason for revert:
Broke PDFExtensionTest when rolling DEPS in Chromium.

Original issue's description:
> Assert that only 0-numbered objects are Released()
> 
> This condition holds because numbered objects are brute-force
> deleted by the indirect object holder, rather than being
> released.
> 
> Be careful about recursive deletion, check before advancing,
> since we no longer count on Release() doing this for us.
> Fix a few tests where the test was violating ownership rules.
> 
> This should be the last step before completely removing Release()
> in favor of direct delete everywhere.
> 
> Committed:
https://pdfium.googlesource.com/pdfium/+/aba528a362248a54b27a7e9e046e2b65ab83...

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

to

==========
Revert of Assert that only 0-numbered objects are Released() (patchset #7
id:120001 of https://codereview.chromium.org/2375343004/ )

Reason for revert:
Broke PDFExtensionTest when rolling DEPS in Chromium.

Original issue's description:
> Assert that only 0-numbered objects are Released()
> 
> This condition holds because numbered objects are brute-force
> deleted by the indirect object holder, rather than being
> released.
> 
> Be careful about recursive deletion, check before advancing,
> since we no longer count on Release() doing this for us.
> Fix a few tests where the test was violating ownership rules.
> 
> This should be the last step before completely removing Release()
> in favor of direct delete everywhere.
> 
> Committed:
https://pdfium.googlesource.com/pdfium/+/aba528a362248a54b27a7e9e046e2b65ab83...

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

Committed:
https://pdfium.googlesource.com/pdfium/+/b73c99335bfbd158ad16dd59c9c52396ffd2...
==========

[commit-bot: I haz the power, 2016-10-04 04:28:10]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/b73c99335bfbd158ad16dd59c9c52396ffd2...