Remove can_be_in_same_chunk (#8029)

This is unused. Closes PACK-2995
vercel · Apr 24, 2024 · 9783846 · 9783846
1 parent e63b1a8
commit 9783846
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 43 deletions.
diff --git a/crates/turbopack-browser/src/chunking_context.rs b/crates/turbopack-browser/src/chunking_context.rs
@@ -310,24 +310,6 @@ impl ChunkingContext for BrowserChunkingContext {
         Ok(Vc::cell(source_maps))
     }
 
-    #[turbo_tasks::function]
-    async fn can_be_in_same_chunk(
-        &self,
-        asset_a: Vc<Box<dyn Module>>,
-        asset_b: Vc<Box<dyn Module>>,
-    ) -> Result<Vc<bool>> {
-        let parent_dir = asset_a.ident().path().parent().await?;
-
-        let path = asset_b.ident().path().await?;
-        if let Some(rel_path) = parent_dir.get_path_to(&path) {
-            if !rel_path.starts_with("node_modules/") && !rel_path.contains("/node_modules/") {
-                return Ok(Vc::cell(true));
-            }
-        }
-
-        Ok(Vc::cell(false))
-    }
-
     #[turbo_tasks::function]
     async fn asset_path(
         &self,

diff --git a/crates/turbopack-core/src/chunk/chunking_context.rs b/crates/turbopack-core/src/chunk/chunking_context.rs
@@ -9,7 +9,6 @@ use crate::{
     chunk::{ChunkItem, ModuleId},
     environment::Environment,
     ident::AssetIdent,
-    module::Module,
     output::{OutputAsset, OutputAssets},
 };
 
@@ -62,12 +61,6 @@ pub trait ChunkingContext {
     /// Reference Source Map Assets for chunks
     fn reference_chunk_source_maps(self: Vc<Self>, chunk: Vc<Box<dyn OutputAsset>>) -> Vc<bool>;
 
-    fn can_be_in_same_chunk(
-        self: Vc<Self>,
-        asset_a: Vc<Box<dyn Module>>,
-        asset_b: Vc<Box<dyn Module>>,
-    ) -> Vc<bool>;
-
     /// Returns a URL (relative or absolute, depending on the asset prefix) to
     /// the static asset based on its `ident`.
     fn asset_url(self: Vc<Self>, ident: Vc<AssetIdent>) -> Result<Vc<String>>;

diff --git a/crates/turbopack-nodejs/src/chunking_context.rs b/crates/turbopack-nodejs/src/chunking_context.rs
@@ -274,24 +274,6 @@ impl ChunkingContext for NodeJsChunkingContext {
         Vc::cell(true)
     }
 
-    #[turbo_tasks::function]
-    async fn can_be_in_same_chunk(
-        &self,
-        asset_a: Vc<Box<dyn Module>>,
-        asset_b: Vc<Box<dyn Module>>,
-    ) -> Result<Vc<bool>> {
-        let parent_dir = asset_a.ident().path().parent().await?;
-
-        let path = asset_b.ident().path().await?;
-        if let Some(rel_path) = parent_dir.get_path_to(&path) {
-            if !rel_path.starts_with("node_modules/") && !rel_path.contains("/node_modules/") {
-                return Ok(Vc::cell(true));
-            }
-        }
-
-        Ok(Vc::cell(false))
-    }
-
     #[turbo_tasks::function]
     async fn asset_path(
         &self,