JFR Browser improvements

 - simple filter to display just First/Middle/Last/Sample N events of the selected type
 - option to display/hide experimental events and properties

Author: jisedlac

visualvm/jfr/src/org/graalvm/visualvm/jfr/views/browser/BrowserNode.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2019, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -49,11 +49,6 @@ abstract class BrowserNode extends CCTNode {
 
     long eventsCount;
 
-//    long time = -1;
-//    String setting;
-//    String value;
-//    String thread;
-    
 
     BrowserNode(String name, Icon icon, BrowserNode parent, List<BrowserNode> children) {
         this.parent = parent;
@@ -134,10 +129,6 @@ public String toString() {
         return name;
     }
 
-//    public String eventsToString() {
-//        return " (" + eventsCount + (eventsCount == 1 ? " event)" : " events)");
-//    }
-    
 
     static final class EventType extends BrowserNode {
 
@@ -175,12 +166,7 @@ static final class Category extends BrowserNode {
     }
 
 
-    static class Root extends BrowserNode implements JFREventVisitor {
-        
-//        private Map<Long, String> types;
-        
-//        private long settingID = 0;
-        
+    static class Root extends BrowserNode implements JFREventVisitor {        
 
         Root(String name) {
             this();
@@ -193,83 +179,28 @@ static class Root extends BrowserNode implements JFREventVisitor {
         }
 
 
-//        void visitEventTypes() {};
-//        
-//        
-//        @Override
-//        public void initTypes() {
-//            types = new HashMap();
-//        }
-//        
-//        @Override
-//        public boolean visitType(JFREventType type) {
-//            types.put(type.getId(), type.getDisplayName());
-//            return false;
-//        }
-        
-        
-        void reloadEvents(JFREventVisitor visitor) {};
-        
         JFREventType type(String typeName) { return null; }
 
 
         @Override
         public boolean visit(String typeName, JFREvent event) {
-//            if ("jdk.ActiveSetting".equals(typeName)) { // NOI18N
-//                try {
-                    JFREventType type = type(typeName);
-//                    String eventName = typeName;
-                    String eventName = type.getName();
-                    
-                    BrowserNode categoryNode = getOrCreateCategory(this, type.getCategory());
-                    BrowserNode eventNode = categoryNode.getChild(eventName);
-                    if (eventNode == null) {
-                        eventNode = new EventType(typeName, type, categoryNode);
-                        categoryNode.addChild(eventNode);
-                    } else {
-                        ((EventType)eventNode).addEvent();
-                    }
-//                    String eventName;
-//                    try {
-//                        eventName = getTypeName(event.getLong("id")); // NOI18N
-//                    } catch (JFRPropertyNotAvailableException e) {
-//                        eventName = event.getValue("settingFor").toString(); // NOI18N
-//                    }
-//
-//                    BrowserNode eventNode = getChild(eventName);
-//                    if (eventNode == null) {
-//                        eventNode = new Event(eventName, this);
-//                        addChild(eventNode);
-//                    }
-//
-//                    String settingName = event.getString("name"); // NOI18N
-//                    String settingValue = event.getString("value"); // NOI18N
-//                    
-//                    String threadName;
-//                    try {
-//                        threadName = event.getThread("eventThread").getName(); // NOI18N
-//                    } catch (JFRPropertyNotAvailableException e) {
-//                        threadName = "-"; // NOI18N
-//                    }
-//                    
-//                    BrowserNode settingNode = new Setting(settingID++, settingName, settingValue, threadName, event.getInstant("eventTime").toEpochMilli(), eventNode);
-//                    eventNode.addChild(settingNode);
-//                } catch (JFRPropertyNotAvailableException e) {
-//                    System.err.println(">>> XX " + e + " -- " + event);
-//                }
-//            }
+            JFREventType type = type(typeName);
+            if (type == null) return false; // experimental type not resolved
+            
+            String eventName = type.getName();
+
+            BrowserNode categoryNode = getOrCreateCategory(this, type.getCategory());
+            BrowserNode eventNode = categoryNode.getChild(eventName);
+            if (eventNode == null) {
+                eventNode = new EventType(typeName, type, categoryNode);
+                categoryNode.addChild(eventNode);
+            } else {
+                ((EventType)eventNode).addEvent();
+            }
 
             return false;
         }
 
-        @Override
-        public void done() {
-//            if (types != null) {
-//                types.clear();
-//                types = null;
-//            }
-        }
-        
 
         private static BrowserNode getOrCreateCategory(BrowserNode parent, List<String> category) {
             List<String> names = new ArrayList(category);
@@ -288,12 +219,6 @@ private static BrowserNode getOrCreateCategory(BrowserNode parent, List<String>
         }
 
 
-//        private String getTypeName(long typeID) {
-//            if (types == null) visitEventTypes();
-//            return types.get(typeID);
-//        }
-        
-        
         @Override
         public int hashCode() {
             return 37;