UIP-2298 Include explicitly-used transitive imports, as warned by pub publish

[greglittlefield-wf]

Ultimate problem:

When running pub publish, there were warnings about libraries being used but not declared as dependencies:

Suggestions:
* line 21, column 1 of lib/src/util/react_wrappers.dart: This package doesn't depend on js.
  import 'package:js/js.dart';
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
* line 23, column 1 of lib/transformer.dart: This package doesn't depend on path.
  import 'package:path/path.dart' as p;
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
* line 17, column 1 of test/over_react/component_declaration/component_type_checking_test.dart: This package doesn't depend on js.
  import 'package:js/js.dart';
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
* line 20, column 1 of test/over_react/util/react_wrappers_test.dart: This package doesn't depend on js.
  import 'package:js/js.dart';
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
* line 21, column 1 of test/test_util/dom_util.dart: This package doesn't depend on browser_detect.
  import 'package:browser_detect/browser_detect.dart';
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
* line 21, column 1 of test/test_util/react_util.dart: This package doesn't depend on js.
  import 'package:js/js.dart';
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
* line 26, column 1 of test/wsd_test_util/common_component_tests.dart: This package doesn't depend on browser_detect.
  import 'package:browser_detect/browser_detect.dart';
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
* line 17, column 1 of test/wsd_test_util/test_js_component.dart: This package doesn't depend on js.
  import 'package:js/js.dart';
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Package has 8 warnings.

How it was fixed:

Add packages to pubspec.yaml.

Testing suggestions:

• Pull this branch, run pub publish --dry-run, and verify that there are no warnings
• Verify that pub get completes successfully (should be done by CI)

Potential areas of regression:

Dependencies

---

FYA: @greglittlefield-wf @aaronlademann-wf @jacehensley-wf @clairesarsam-wf @joelleibow-wf

[aviary-wf]

Raven

Number of Findings: 0

[codecov-io]

Codecov Report

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

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   97.69%   97.69%           
=======================================
  Files          28       28           
  Lines        1380     1380           
=======================================
  Hits         1348     1348           
  Misses         32       32

[jacehensley-wf]

+1

[leviwith-wf]

QA +10

• no warnings in pub publish --dry run
• pub get works

---

• Testing instruction
• Dev +1's
• Dev/QA +10
• All unit tests pass
• Rosie ran/Rosie comment displays expected info
• Dependency Scan Clean

Merging.