[Cleanup] List trace categories through script as shown by trace record dialog

tools/profile_chrome/chrome_controller.py

Index: tools/profile_chrome/chrome_controller.py
diff --git a/tools/profile_chrome/chrome_controller.py b/tools/profile_chrome/chrome_controller.py
index 91c2504a54c4e34583644db055ad6cd543dd6fb7..18313cb883f5b102e63f276e2c21e8fa4a782566 100644
--- a/tools/profile_chrome/chrome_controller.py
+++ b/tools/profile_chrome/chrome_controller.py
@@ -51,10 +51,18 @@ class ChromeTracingController(controllers.BaseController):
     disabled_by_default_categories = []
     json_data = json.loads(json_category_list)['traceCategoriesList']
     for item in json_data:
-      if item.startswith('disabled-by-default'):
-        disabled_by_default_categories.append(item)
-      else:
-        record_categories.append(item)
+      categories = item.split(',')
+      try:
+        if item.startswith('disabled-by-default'):

[dsinclair, 2014/11/07 00:05:25]

Don't we want to do this check against each item in categories? If the second
category is disabled by default we'd put it in the wrong list?

It seems like like we want to add a loop over categories and do the original if
statement on each of the items in that list?

[r.kasibhatla, 2014/11/07 09:58:47]

Correct. I was trying to club too many conditions in single statement. I have
corrected it now. :)

+          disabled_by_default_categories.append(
+              next(category for category in categories
+                  if category not in disabled_by_default_categories))
+        else:
+          record_categories.append(
+              next(category for category in categories
+                  if category not in record_categories))
+      except StopIteration, e:
+        pass
 
     return record_categories, disabled_by_default_categories