diff options
| author | boliu <boliu@chromium.org> | 2015-04-01 16:11:44 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-04-01 23:12:09 +0000 |
| commit | 139663213d8cc9dc1bad9c2d7d929cb454416288 (patch) | |
| tree | dacbe7dd0aa5814100a9168a8ead85311673bc8b /content/public/android/java | |
| parent | 87bac62e36ee67c35dec9d82973a56fb7606863e (diff) | |
| download | chromium_src-139663213d8cc9dc1bad9c2d7d929cb454416288.zip chromium_src-139663213d8cc9dc1bad9c2d7d929cb454416288.tar.gz chromium_src-139663213d8cc9dc1bad9c2d7d929cb454416288.tar.bz2 | |
Remove finalizers that only assert
Finalizers have a cost and exceptions in them do not cause a crash. So
finalizers that only assert are pretty much useless. Remove them.
BUG=472713
Review URL: https://codereview.chromium.org/1055633002
Cr-Commit-Position: refs/heads/master@{#323364}
Diffstat (limited to 'content/public/android/java')
| -rw-r--r-- | content/public/android/java/src/org/chromium/content/browser/TracingControllerAndroid.java | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/content/public/android/java/src/org/chromium/content/browser/TracingControllerAndroid.java b/content/public/android/java/src/org/chromium/content/browser/TracingControllerAndroid.java index b29a4db..48cbd5e 100644 --- a/content/public/android/java/src/org/chromium/content/browser/TracingControllerAndroid.java +++ b/content/public/android/java/src/org/chromium/content/browser/TracingControllerAndroid.java @@ -239,12 +239,6 @@ public class TracingControllerAndroid { } } - @Override - protected void finalize() { - // Ensure that destroy() was called. - assert mNativeTracingControllerAndroid == 0; - } - /** * Clean up the C++ side of this class. * After the call, this class instance shouldn't be used. |