Bug 1534808 - Backed out changeset 33f142760b69 - Don't check for JavaScript before showing debugger source tab title r=jlast a=backout
authorDavid Walsh <dwalsh@mozilla.com>
Mon, 18 Mar 2019 16:18:24 +0000
changeset 525635 3a2c5cdf6b064bfdfed38b7a0ecb07cc7bf6d722
parent 525634 14a745d24befa142078249c91fd29a1df7f54c83
child 525636 df5b195834f1dca643bfb34a6977e6581407ab44
push id2032
push userffxbld-merge
push dateMon, 13 May 2019 09:36:57 +0000
treeherdermozilla-release@455c1065dcbe [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjlast, backout
bugs1534808
milestone67.0
Bug 1534808 - Backed out changeset 33f142760b69 - Don't check for JavaScript before showing debugger source tab title r=jlast a=backout I talked to Harald and he doesn't want this. Differential Revision: https://phabricator.services.mozilla.com/D23743
devtools/client/debugger/new/src/components/Editor/Tab.js
--- a/devtools/client/debugger/new/src/components/Editor/Tab.js
+++ b/devtools/client/debugger/new/src/components/Editor/Tab.js
@@ -18,17 +18,16 @@ import type { Source } from "../../types
 import actions from "../../actions";
 
 import {
   getDisplayPath,
   getFileURL,
   getRawSourceURL,
   getSourceQueryString,
   getTruncatedFileName,
-  isJavaScript,
   isPretty,
   shouldBlackbox
 } from "../../utils/source";
 import { copyToTheClipboard } from "../../utils/clipboard";
 import { getTabMenuItems } from "../../utils/tabs";
 
 import {
   getSelectedSource,
@@ -145,17 +144,17 @@ class Tab extends PureComponent<Props> {
           disabled: !shouldBlackbox(source),
           click: () => toggleBlackBox(source)
         }
       },
       {
         item: {
           ...tabMenuItems.prettyPrint,
           click: () => togglePrettyPrint(tab),
-          disabled: isPretty(source) || !isJavaScript(source)
+          disabled: isPretty(sourceTab)
         }
       }
     ];
 
     showMenu(e, buildMenu(items));
   }
 
   isProjectSearchEnabled() {