refactor(ads): use injected InlineAdCacheService instead of creating new instance

- Inject InlineAdCacheService into HeadlinesFeedBloc constructor
- Replace InlineAdCacheService() calls with _inlineAdCacheService field
- Remove unused AdService import and usage in HeadlinesFeedPage

Author: fulleni

lib/headlines-feed/bloc/headlines_feed_bloc.dart

@@ -31,10 +31,12 @@ class HeadlinesFeedBloc extends Bloc<HeadlinesFeedEvent, HeadlinesFeedState> {
     userContentPreferencesRepository,
     required FeedDecoratorService feedDecoratorService,
     required AppBloc appBloc,
+    required InlineAdCacheService inlineAdCacheService,
   }) : _headlinesRepository = headlinesRepository,
        _userContentPreferencesRepository = userContentPreferencesRepository,
        _feedDecoratorService = feedDecoratorService,
        _appBloc = appBloc,
+       _inlineAdCacheService = inlineAdCacheService,
        super(const HeadlinesFeedState()) {
     on<HeadlinesFeedFetchRequested>(
       _onHeadlinesFeedFetchRequested,
@@ -65,6 +67,7 @@ class HeadlinesFeedBloc extends Bloc<HeadlinesFeedEvent, HeadlinesFeedState> {
   _userContentPreferencesRepository;
   final FeedDecoratorService _feedDecoratorService;
   final AppBloc _appBloc;
+  final InlineAdCacheService _inlineAdCacheService;
 
   /// The number of headlines to fetch per page.
   static const _headlinesFetchLimit = 10;
@@ -154,7 +157,7 @@ class HeadlinesFeedBloc extends Bloc<HeadlinesFeedEvent, HeadlinesFeedState> {
     Emitter<HeadlinesFeedState> emit,
   ) async {
     // On a full refresh, clear the ad cache to ensure fresh ads are loaded.
-    InlineAdCacheService().clearAllAds();
+    _inlineAdCacheService.clearAllAds();
     emit(state.copyWith(status: HeadlinesFeedStatus.loading));
     try {
       final currentUser = _appBloc.state.user;
@@ -236,7 +239,7 @@ class HeadlinesFeedBloc extends Bloc<HeadlinesFeedEvent, HeadlinesFeedState> {
   ) async {
     // When applying new filters, this is considered a major feed change,
     // so we clear the ad cache to get a fresh set of relevant ads.
-    InlineAdCacheService().clearAllAds();
+    _inlineAdCacheService.clearAllAds();
     emit(
       state.copyWith(
         status: HeadlinesFeedStatus.loading,
@@ -323,7 +326,7 @@ class HeadlinesFeedBloc extends Bloc<HeadlinesFeedEvent, HeadlinesFeedState> {
     Emitter<HeadlinesFeedState> emit,
   ) async {
     // Clearing filters is a major feed change, so clear the ad cache.
-    InlineAdCacheService().clearAllAds();
+    _inlineAdCacheService.clearAllAds();
     emit(
       state.copyWith(
         status: HeadlinesFeedStatus.loading,

lib/headlines-feed/view/headlines_feed_page.dart

@@ -5,7 +5,6 @@ import 'package:core/core.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/account/bloc/account_bloc.dart';
-import 'package:flutter_news_app_mobile_client_full_source_code/ads/ad_service.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/ads/interstitial_ad_manager.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/ads/models/ad_placeholder.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/ads/models/ad_theme_style.dart';
@@ -344,7 +343,6 @@ class _HeadlinesFeedPageState extends State<HeadlinesFeedPage> {
 
                     return FeedAdLoaderWidget(
                       adPlaceholder: item,
-                      adService: context.read<AdService>(),
                       adThemeStyle: AdThemeStyle.fromTheme(Theme.of(context)),
                       adConfig: adConfig,
                     );