Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(652)

Side by Side Diff: content/public/android/java/src/org/chromium/content/app/ContentApplication.java

Issue 386983002: Correct the usage of finalize in TracingControllerAndroid class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: updated as per comments Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.content.app; 5 package org.chromium.content.app;
6 6
7 import android.os.Looper; 7 import android.os.Looper;
8 import android.os.MessageQueue; 8 import android.os.MessageQueue;
9 9
10 import org.chromium.base.BaseChromiumApplication; 10 import org.chromium.base.BaseChromiumApplication;
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 49
50 /** 50 /**
51 * For emulated process environment only. On a production device, the applic ation process is 51 * For emulated process environment only. On a production device, the applic ation process is
52 * simply killed without calling this method. We don't need to unregister th e broadcast 52 * simply killed without calling this method. We don't need to unregister th e broadcast
53 * receiver in the latter case. 53 * receiver in the latter case.
54 */ 54 */
55 @Override 55 @Override
56 public void onTerminate() { 56 public void onTerminate() {
57 try { 57 try {
58 getTracingController().unregisterReceiver(this); 58 getTracingController().unregisterReceiver(this);
59 getTracingController().destroy();
59 } catch (SecurityException e) { 60 } catch (SecurityException e) {
60 // Happens if the process is isolated. Ignore. 61 // Happens if the process is isolated. Ignore.
61 } 62 }
62 63
63 super.onTerminate(); 64 super.onTerminate();
64 } 65 }
65 66
66 } 67 }
OLDNEW
« no previous file with comments | « AUTHORS ('k') | content/public/android/java/src/org/chromium/content/browser/TracingControllerAndroid.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698