Stop looking at .js files

Summary: It was only used by a step removed in {D5139671}. I looked at remaining steps and I think nothing else needs it.

Reviewed By: dlreeves

Differential Revision: D6731028

fbshipit-source-id: 80d3a9dc7166b3325fc3230e7e7f3d6431c15e81

hphp/hack/src/server/serverEnv.ml

@@ -124,7 +124,6 @@ let file_filter f =
   (* Filter the relative path *)
   let f = Relative_path.strip_root_if_possible f in
   (FindUtils.is_php f && not (FilesToIgnore.should_ignore f))
-  || FindUtils.is_js f
 
 let list_files env oc =
   let acc = List.fold_right

hphp/hack/src/server/serverEnvBuild.ml

@@ -28,11 +28,6 @@ let watchman_expression_terms = [
       J.strlist ["suffix"; "hh"];
       J.strlist ["suffix"; "hhi"];
       J.strlist ["suffix"; "xhp"];
-      (* FIXME: This is clearly wrong, but we do it to match the
-       * behavior on the server-side. We need to investigate if
-       * tracking js files is truly necessary.
-       *)
-      J.strlist ["suffix"; "js"];
     ];
   ];
   J.pred "not" @@ [

hphp/hack/src/utils/findUtils.ml

@@ -30,10 +30,6 @@ let is_php path =
   not (is_dot_file path) &&
   List.exists extensions (Filename.check_suffix path)
 
-let is_js path =
-  not (is_dot_file path) &&
-  Filename.check_suffix path ".js"
-
 (** Returns whether one of the ancestral directories of path has the given
  * name. *)
 let rec has_ancestor path ancestor_name =

hphp/hack/src/utils/findUtils.mli

@@ -8,6 +8,6 @@
  *
  *)
 
+val is_dot_file: string -> bool
 val is_php: string -> bool
-val is_js: string -> bool
 val has_ancestor: string -> string -> bool