chore(04-02): remove grace period placeholder from OverlayProvider
- Remove _gracePeriodTimer field and all grace period logic - Grace period management now handled by GpsDataManager - Overlay provider wiring completed in Task 1 (SpeedometerScreen initState) - GPS_GRACE_PERIOD constant remains in TimingConfig (deprecated, unused)
This commit is contained in:
@@ -23,7 +23,6 @@ class OverlayProvider extends ChangeNotifier {
|
|||||||
StreamSubscription<dynamic>? _overlayMessageSubscription;
|
StreamSubscription<dynamic>? _overlayMessageSubscription;
|
||||||
Timer? _overlayStatusCheckTimer;
|
Timer? _overlayStatusCheckTimer;
|
||||||
Timer? _tapCloseTimer;
|
Timer? _tapCloseTimer;
|
||||||
Timer? _gracePeriodTimer;
|
|
||||||
|
|
||||||
// Dependencies — set via update() from ProxyProvider
|
// Dependencies — set via update() from ProxyProvider
|
||||||
GpsDataManager? _gpsManager;
|
GpsDataManager? _gpsManager;
|
||||||
@@ -70,10 +69,6 @@ class OverlayProvider extends ChangeNotifier {
|
|||||||
Future<void> showOverlay() async {
|
Future<void> showOverlay() async {
|
||||||
if (_isDisposed) return;
|
if (_isDisposed) return;
|
||||||
|
|
||||||
// Cancel grace period if user reopens overlay within grace period
|
|
||||||
_gracePeriodTimer?.cancel();
|
|
||||||
_gracePeriodTimer = null;
|
|
||||||
|
|
||||||
try {
|
try {
|
||||||
// Check permission before attempting to show overlay
|
// Check permission before attempting to show overlay
|
||||||
final hasPermission = await _overlayService.checkPermission();
|
final hasPermission = await _overlayService.checkPermission();
|
||||||
@@ -246,13 +241,6 @@ class OverlayProvider extends ChangeNotifier {
|
|||||||
_stopListeningToOverlayMessages();
|
_stopListeningToOverlayMessages();
|
||||||
notifyListeners();
|
notifyListeners();
|
||||||
|
|
||||||
// Start GPS grace period (30s) — infrastructure for Phase 4
|
|
||||||
_gracePeriodTimer?.cancel();
|
|
||||||
_gracePeriodTimer = Timer(TimingConfig.GPS_GRACE_PERIOD, () {
|
|
||||||
Logger.info('GPS grace period expired', 'OverlayProvider');
|
|
||||||
_gracePeriodTimer = null;
|
|
||||||
});
|
|
||||||
|
|
||||||
if (shouldBringToFront) {
|
if (shouldBringToFront) {
|
||||||
_bringAppToFront();
|
_bringAppToFront();
|
||||||
}
|
}
|
||||||
@@ -280,7 +268,6 @@ class OverlayProvider extends ChangeNotifier {
|
|||||||
_overlayMessageSubscription?.cancel();
|
_overlayMessageSubscription?.cancel();
|
||||||
_overlayStatusCheckTimer?.cancel();
|
_overlayStatusCheckTimer?.cancel();
|
||||||
_tapCloseTimer?.cancel();
|
_tapCloseTimer?.cancel();
|
||||||
_gracePeriodTimer?.cancel();
|
|
||||||
Logger.info('OverlayProvider disposed', 'OverlayProvider');
|
Logger.info('OverlayProvider disposed', 'OverlayProvider');
|
||||||
super.dispose();
|
super.dispose();
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user