Trait map optimizations (#7007)

## Description

A couple PRs with trait map optimizations developed as part of trait
coherence effort.

## Checklist

- [ ] I have linked to any relevant issues.
- [ ] I have commented my code, particularly in hard-to-understand
areas.
- [ ] I have updated the documentation where relevant (API docs, the
reference, and the Sway book).
- [ ] If my change requires substantial documentation changes, I have
[requested support from the DevRel
team](https://github.com/FuelLabs/devrel-requests/issues/new/choose)
- [ ] I have added tests that prove my fix is effective or that my
feature works.
- [ ] I have added (or requested a maintainer to add) the necessary
`Breaking*` or `New Feature` labels where relevant.
- [ ] I have done my best to ensure that my PR adheres to [the Fuel Labs
Code Review
Standards](https://github.com/FuelLabs/rfcs/blob/master/text/code-standards/external-contributors.md).
- [ ] I have requested a review from the relevant team or maintainers.

Author: tritao

sway-core/src/semantic_analysis/ast_node/expression/typed_expression.rs

@@ -2573,7 +2573,7 @@ impl ty::TyExpression {
         base_name: &Ident,
         projections: &[ty::ProjectionKind],
     ) -> Result<(TypeId, TypeId), ErrorEmitted> {
-        let ret = module.walk_scope_chain(|lexical_scope| {
+        let ret = module.walk_scope_chain_early_return(|lexical_scope| {
             Self::find_subfield_type_helper(
                 lexical_scope,
                 handler,

sway-core/src/semantic_analysis/namespace/lexical_scope.rs

@@ -678,7 +678,7 @@ impl Items {
                 }
             };
 
-        let _ = module.walk_scope_chain(|lexical_scope| {
+        let _ = module.walk_scope_chain_early_return(|lexical_scope| {
             if let Some((ident, decl)) = lexical_scope.items.symbols.get_key_value(&name) {
                 append_shadowing_error(
                     ident,

sway-core/src/semantic_analysis/namespace/module.rs

@@ -291,7 +291,7 @@ impl Module {
         self.current_lexical_scope_id = parent_scope_id.unwrap(); // panics if pops do not match pushes
     }
 
-    pub fn walk_scope_chain<T>(
+    pub fn walk_scope_chain_early_return<T>(
         &self,
         mut f: impl FnMut(&LexicalScope) -> Result<Option<T>, ErrorEmitted>,
     ) -> Result<Option<T>, ErrorEmitted> {
@@ -310,23 +310,16 @@ impl Module {
         Ok(None)
     }
 
-    pub fn walk_scope_chain_mut<T>(
-        &mut self,
-        mut f: impl FnMut(&mut LexicalScope) -> Result<Option<T>, ErrorEmitted>,
-    ) -> Result<Option<T>, ErrorEmitted> {
-        let mut lexical_scope_opt = Some(self.current_lexical_scope_mut());
+    pub fn walk_scope_chain(&self, mut f: impl FnMut(&LexicalScope)) {
+        let mut lexical_scope_opt = Some(self.current_lexical_scope());
         while let Some(lexical_scope) = lexical_scope_opt {
-            let result = f(lexical_scope)?;
-            if let Some(result) = result {
-                return Ok(Some(result));
-            }
+            f(lexical_scope);
             if let Some(parent_scope_id) = lexical_scope.parent {
-                lexical_scope_opt = self.get_lexical_scope_mut(parent_scope_id);
+                lexical_scope_opt = self.get_lexical_scope(parent_scope_id);
             } else {
                 lexical_scope_opt = None;
             }
         }
-        Ok(None)
     }
 
     pub fn get_items_for_type(
@@ -344,7 +337,7 @@ impl Module {
         symbol: &Ident,
     ) -> Result<(ResolvedDeclaration, ModulePathBuf), ErrorEmitted> {
         let mut last_handler = Handler::default();
-        let ret = self.walk_scope_chain(|lexical_scope| {
+        let ret = self.walk_scope_chain_early_return(|lexical_scope| {
             last_handler = Handler::default();
             Ok(lexical_scope
                 .items