Enable Tracing on content shell for Android

content/shell/android/java/src/org/chromium/content_shell/ContentShellActivity.java

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.content_shell;
 
 import android.app.Activity;
+import android.content.BroadcastReceiver;
+import android.content.Context;
 import android.content.Intent;
+import android.content.IntentFilter;
 import android.os.Bundle;
 import android.text.TextUtils;
 import android.util.Log;
 import android.view.KeyEvent;
 
 import org.chromium.base.ChromiumActivity;
 import org.chromium.content.app.LibraryLoader;
 import org.chromium.content.browser.ActivityContentVideoViewDelegate;
 import org.chromium.content.browser.ContentVideoView;
 import org.chromium.content.browser.ContentView;
 import org.chromium.content.browser.DeviceUtils;
+import org.chromium.content.browser.TraceController;
 import org.chromium.content.common.CommandLine;
 import org.chromium.ui.gfx.ActivityNativeWindow;
 
 /**
  * Activity for managing the Content Shell.
  */
 public class ContentShellActivity extends ChromiumActivity {
 
     public static final String COMMAND_LINE_FILE = "/data/local/tmp/content-shell-command-line";
     private static final String TAG = ContentShellActivity.class.getName();
 
     private static final String ACTIVE_SHELL_URL_KEY = "activeUrl";
+    private static final String ACTION_START_TRACE = "org.chromium.content_shell.action.PROFILE_START";
+    private static final String ACTION_STOP_TRACE = "org.chromium.content_shell.action.PROFILE_STOP";
     public static final String DEFAULT_SHELL_URL = "http://www.google.com";
     public static final String COMMAND_LINE_ARGS_KEY = "commandLineArgs";
 
     private ShellManager mShellManager;
     private ActivityNativeWindow mActivityNativeWindow;
+    private BroadcastReceiver mReceiver;
 
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
 
         // Initializing the command line must occur before loading the library.
         if (!CommandLine.isInitialized()) {
             CommandLine.initFromFile(COMMAND_LINE_FILE);
             String[] commandLineParams = getCommandLineParamsFromIntent(getIntent());
             if (commandLineParams != null) {
@@ skipping 75 matching lines @@
             }
         }
     }
 
     @Override
     protected void onPause() {
         ContentView view = getActiveContentView();
         if (view != null) view.onActivityPause();
 
         super.onPause();
+        unregisterReceiver(mReceiver);
     }
 
     @Override
     protected void onResume() {
         super.onResume();
 
         ContentView view = getActiveContentView();
         if (view != null) view.onActivityResume();
+        IntentFilter intentFilter = new IntentFilter(ACTION_START_TRACE);
+        intentFilter.addAction(ACTION_STOP_TRACE);
+        mReceiver = new BroadcastReceiver() {
+            @Override
+            public void onReceive(Context context, Intent intent) {
+                String action = intent.getAction();
+                String extra = intent.getStringExtra("file");
+                if (ACTION_START_TRACE.equals(action)) {
+                    if (extra.isEmpty())

[Ted C, 2013/01/04 17:40:48]

sadly the conventions across languages are different, but in java braces are
always required unless the conditional and statement fit on one line.

i.e. this is fine.

if (x > 0) printSomething();

this is not:

if (x > 0)
    printSomething();

this is fine to:

if (x > 0) {
    printSomething();
}

Style guide for android:
http://source.android.com/source/code-style.html#use-standard-brace-style

With the if else format, you're going to need to use braces probably as we like
it to be consistent across the condition checks.

+                        Log.e("ContentShell", "Can not start tracing without specifing saving location");

[Ted C, 2013/01/04 17:40:48]

Although, you should use TAG from above instead of "ContentShell" and that might
get you under the limit.

+                    else {
+                        TraceController.beginTracing(extra);
+                        Log.i("ContentShell", "start tracing");

[Ted C, 2013/01/04 17:40:48]

Use TAG here too and in the else if below.

+                    }
+                } else if (ACTION_STOP_TRACE.equals(action)) {
+                    Log.i("ContentShell", "stop tracing");
+                    TraceController.endTracing();
+                }
+            }
+        };
+        registerReceiver(mReceiver, intentFilter);
     }
 
     @Override
     public void onActivityResult(int requestCode, int resultCode, Intent data) {
         super.onActivityResult(requestCode, resultCode, data);
         mActivityNativeWindow.onActivityResult(requestCode, resultCode, data);
     }
 
     private static String getUrlFromIntent(Intent intent) {
         return intent != null ? intent.getDataString() : null;
@@ skipping 20 matching lines @@
 
     /**
      * @return The {@link ContentView} owned by the currently visible {@link Shell} or null if one
      *         is not showing.
      */
     public ContentView getActiveContentView() {
         Shell shell = getActiveShell();
         return shell != null ? shell.getContentView() : null;
     }
 }