This sounds like potential fix for a variant of this issue #400

I understand that Screen size changed all the time is annoying and noisy, but it's what the OS/framework triggers anyway.
I'd be fine adding a enableWindowMetricBreadcrumbs flag but I'd rather prefer the debouncing in another PR, because it can be reused across different sources of duplication.
I'd rather prefer a debouncing algorithm that works generically instead of a local var which requires the implementation everywhere, how does that sound?

@marandaneto I'm not sure that debouncing is right solution here. It's also incorrect, that each time client sends event to Sentry means that framework reports about screen size change.
According to the Flutter docs, onMetricsChanged method can be called whenever the devicePixelRatio, physicalSize, padding, viewInsets, PlatformDispatcher.views, or systemGestureInsets values change (docs). As you can see, there are few more values that has no relation to screen size and can trigger event on update. As i mentioned in the description, i have 30+ events each time user opens a keyboard, while screen size, obviously, persists the same (it's viewInsets changes when keyboard opens).

However, i agree, that variable + if statement doesn't look like an enterprise solution, i'm going to spend some time to refactor it to something more generic & easy to reuse, but i still believe that this solution should aim to send only distinct events, not debounced.

@marandaneto @brustolin I've refactored this to use stream controller with distinct to avoid duplicates

@lavinov-mercury thanks, I'll review it early next week.

@lavinov-mercury missing changelog entry.
Thanks for doing this.

@marandaneto I've updated changelog, could you please check, if it's done right?

@marandaneto I've updated changelog, could you please check, if it's done right?

All good, thanks.
CI is going to fail due to #938 but its a different issue.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.81%. Comparing base (bdc6e0f) to head (f3acb7b).
Report is 1336 commits behind head on main.

@@           Coverage Diff           @@
##             main     #888   +/-   ##
=======================================
  Coverage   89.81%   89.81%           
=======================================
  Files         104      104           
  Lines        3211     3211           
=======================================
  Hits         2884     2884           
  Misses        327      327           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@marandaneto Hi! It failed to merge as branch became outdated again, could you please start workflows?