Remove a few todos in code

MaterialGraphProject/Assets/UnityShaderEditor/Editor/Drawing/Views/GraphEditorView.cs

 
 namespace UnityEditor.MaterialGraph.Drawing
 {
-    // TODO JOCE: Maybe this needs to derive from something we already have?
     public class GraphEditorView : DataWatchContainer
     {
         GraphView m_GraphView;

MaterialGraphProject/Assets/UnityShaderEditor/Editor/Drawing/Views/TitleBarView.cs

 
 namespace UnityEditor.MaterialGraph.Drawing
 {
-    // TODO JOCE: we should not need a title bar drawer. It should just be a visual element in the nodedrawer.
     public class TitleBarView : DataWatchContainer
     {
         TitleBarPresenter m_DataProvider;
             UpdateContainer(m_RightContainer, m_DataProvider.rightItems);
         }
 
-        void UpdateContainer(VisualContainer container, IEnumerable<TitleBarButtonPresenter> itemDatas)
+        static void UpdateContainer(VisualContainer container, IEnumerable<TitleBarButtonPresenter> itemDatas)
         {
             container.ClearChildren();
             foreach (var itemPresenter in itemDatas)