Remove commit_tab_class helper

app/helpers/tab_helper.rb

@@ -51,12 +51,6 @@ module TabHelper
     end
   end
 
-  def commit_tab_class
-    if ['commits', 'repositories', 'protected_branches'].include? controller.controller_name
-      "active"
-    end
-  end
-
   def branches_tab_class
     if current_page?(branches_project_repository_path(@project)) ||
       controller.controller_name == "protected_branches" ||

app/views/layouts/_project_menu.html.haml

@@ -6,7 +6,7 @@
     - if can? current_user, :download_code, @project
       %li{class: current_controller?(:tree, :blob, :blame) ? 'active' : ''}
         = link_to 'Files', project_tree_path(@project, @project.root_ref)
-      %li{class: commit_tab_class}
+      %li{class: current_controller?(:commit, :commits, :compare, :repositories, :protected_branches) ? 'active' : ''}
         = link_to "Commits", project_commits_path(@project, @project.root_ref)
       %li{class: tab_class(:network)}
         = link_to "Network", graph_project_path(@project)