Remove Release() from CPDF_PageObject

Remove Release() from CPDF_PageObject

BUG=pdfium:140
R=thestig@chromium.org

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

[Tom Sepez, 2015-04-20 18:31:31]

Lei.

[Tom Sepez, 2015-04-20 18:51:41]

tsepez@chromium.org changed reviewers:
+ thestig@chromium.org

[Tom Sepez, 2015-04-20 18:51:53]

For review.

[Lei Zhang, 2015-04-20 19:21:59]

LGTM

I think you can write BUG=pdfium:140 instead of the full URL.

https://codereview.chromium.org/1093213002/diff/1/core/include/fpdfapi/fpdf_p...
File core/include/fpdfapi/fpdf_pageobj.h (right):

https://codereview.chromium.org/1093213002/diff/1/core/include/fpdfapi/fpdf_p...
core/include/fpdfapi/fpdf_pageobj.h:459: virtual ~CPDF_PageObject() { }
Can we move the dtor impl, however small, into the .cpp file?

[Tom Sepez, 2015-04-20 19:29:01]

https://codereview.chromium.org/1093213002/diff/1/core/include/fpdfapi/fpdf_p...
File core/include/fpdfapi/fpdf_pageobj.h (right):

https://codereview.chromium.org/1093213002/diff/1/core/include/fpdfapi/fpdf_p...
core/include/fpdfapi/fpdf_pageobj.h:459: virtual ~CPDF_PageObject() { }
On 2015/04/20 19:21:59, Lei Zhang wrote:
> Can we move the dtor impl, however small, into the .cpp file?

Done.

[Tom Sepez, 2015-04-20 19:29:17]

Committed patchset #2 (id:20001) manually as
bc580c69403169afa656aabb06a56b26d3b729cf (presubmit successful).