DevTools: Implement "close tabs to the right" for Sources panel tabs

Source/devtools/front_end/ui/TabbedPane.js

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 297 matching lines @@
         var tabs = this._tabs.slice();
         for (var i = 0; i < tabs.length; ++i) {
             if (tabs[i].id !== id)
                 result.push(tabs[i].id);
         }
         return result;
     },
 
     /**
      * @param {string} id
+     * @return {!Array.<string>}
+     */
+    tabsToTheRight: function(id)

[alph, 2015/01/15 17:35:17]

_tabsToTheRight

[lushnikov, 2015/01/18 18:17:41]

Done.

+    {
+        var tabs = this._tabs.slice();

[alph, 2015/01/15 17:35:17]

why do you need slice?

[lushnikov, 2015/01/18 18:17:41]

It came due to copy-pasting of other methods; removed (here and there)

+        var index = -1;
+        for (var i = 0; i < tabs.length; ++i) {
+            if (tabs[i].id === id) {
+                index = i;
+                break;
+            }
+        }
+        if (index === -1)
+            return [];
+        var result = [];

[alph, 2015/01/15 17:35:17]

nit:
return tabs.slice(index + 1).map(function(tab) { return tab.id; });

[lushnikov, 2015/01/18 18:17:41]

Done.

+        for (var i = index + 1; i < tabs.length; ++i)
+            result.push(tabs[i].id);
+        return result;
+    },
+
+    /**
+     * @param {string} id
      * @param {boolean=} userGesture
      * @return {boolean}
      */
     selectTab: function(id, userGesture)
     {
         if (this._currentTabLocked)
             return false;
         var focused = this.hasFocus();
         var tab = this._tabsById[id];
         if (!tab)
@@ skipping 736 matching lines @@
         }
 
         /**
          * @this {WebInspector.TabbedPaneTab}
          */
         function closeAll()
         {
             this._closeTabs(this._tabbedPane.allTabs());
         }
 
+        /**
+         * @this {WebInspector.TabbedPaneTab}
+         */
+        function closeToTheRight()
+        {
+            this._closeTabs(this._tabbedPane.tabsToTheRight(this.id));
+        }
+
         var contextMenu = new WebInspector.ContextMenu(event);
         contextMenu.appendItem(WebInspector.UIString.capitalize("Close"), close.bind(this));
         contextMenu.appendItem(WebInspector.UIString.capitalize("Close ^others"), closeOthers.bind(this));
+        contextMenu.appendItem(WebInspector.UIString.capitalize("Close ^tabs ^to ^the ^right"), closeToTheRight.bind(this));

[alph, 2015/01/15 17:35:17]

Prepositions and articles should not be capitalized.

[lushnikov, 2015/01/18 18:17:41]

Done.

         contextMenu.appendItem(WebInspector.UIString.capitalize("Close ^all"), closeAll.bind(this));
         contextMenu.show();
     },
 
     /**
      * @param {!Event} event
      * @return {boolean}
      */
     _startTabDragging: function(event)
     {
@@ skipping 199 matching lines @@
     /**
      * @param {string} id1
      * @param {string} id2
      * @return {number}
      */
     _tabOrderComparator: function(id1, id2)
     {
         return this._tabOrders[id2] = this._tabOrders[id1];
     }
 }