* Migrate core/utils/string.js to ES6 const/let
* Migrate core/utils/string.js to goog.module
* clang-format core/utils/string.js
* Migrate core/utils/string.js to current exports convention
* Convert two remaining vars to lets in core/utils/string.js
* Migrate core/variables.js to ES6 const/let
* Migrate core/variables.js to goog.module
* Migrate core/variables.js to named requires
* clang-format core/variables.js
* Add JSDoc and make testdeps function test-only
* Fix return type for getTestDeps
* Migrate to new style of exposing methods for mocking in core/variables.js
* Migrate core/procedures.js to ES6 const/let
* Migrate core/procedures.js to goog.module
* Migrate core/procedures.js to named requires
* clang-format core/procedures.js
* Rename xml to utilsXml to disambiguate from Xml
* Use goog.requireType when importing I* interfaces
Interfaces have no code, so should never be referred to outside of
(JSDoc) comments, and so the modules that define only interfaces never
need to be goog.require'd - goog.requireType is always sufficient.
This commit fixes imports of all modules whose name matches
/(.*\.)?I[A-Z]*/ - i.e., the hungarian-notation named ones in
core/interfaces/.
* Use goog.requireType when only using import for type specifications
Where a module is imported only to used in JSDoc comments it can
(and should) be goog.requireType'd instead of goog.require'd.
* Remove spurious eslint-disable no-unused-vars
There were a few cases where modules were being imported with
goog.require (because they are referred to in code, not just JSDoc
comments) but were prefaced by a spurious eslint suppress.
Remove these, restoring the invariant that an import gets an eslint
if and only if it is a requireType.
* Remove obsolete Closure Compiler error group
stricterMissingRequire has been superceded by missingRequire, and now
causes a Java null pointer exception if supplied.
* Migrate core/events/events_block_drag.js to ES6 const/let
* Migrate core/events/events_block_drag.js to goog.module
* Migrate core/events/events_block_drag.js to named requires
* clang-format core/events/events_block_drag.js
* Migrate core/events/events_theme_change.js to ES6 const/let
* Migrate core/events/events_theme_change.js to goog.module
* Migrate core/events/events_theme_change.js to named requires
* clang-format core/events/events_theme_change.js
