UIP-2379 Memory leak audit

[aaronlademann-wf]

Ultimate problem:

1. The cancel_subscriptions and close_sinks lints were not enabled.
2. AbstractTransitionComponent / ResizeSensorComponent both have lifecycle methods that - if a consumer were to extend from, and not call super - could/would cause memory leaks.
3. The browser_detect dev dependency is abandonware.

How it was fixed:

1. Lints were added / instances of violations were addressed (tests were the only place that had problems)
2. @mustCallSuper annotation was added to critical lifecycle methods that register handlers that could leak if never un-registered / destroyed / closed.
3. Use platform_detect instead.

Testing suggestions:

Verify that tests pass

Potential areas of regression:

None expected

---

FYA: @Workiva/ui-platform-pp

[aviary2-wf]

Raven

Number of Findings: 0

[jacehensley-wf]

+1

[johnbland-wf]

+1

[codecov-io]

Codecov Report

Merging #75 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #75   +/-   ##
======================================
  Coverage    97.7%   97.7%           
======================================
  Files          28      28           
  Lines        1387    1387           
======================================
  Hits         1355    1355           
  Misses         32      32

[greglittlefield-wf]

+1

[greglittlefield-wf]

QA +1

• Tests Pass
• Dev +1’s
• Rosie ran / evaluation displays expected info
• Dependency Scan Clean

Ready for merge and tag.