[patch] [cla] remove unused dependencies and avoid loading dojo/main

[bill]

See attached patch. It removes unused dependencies, and loads "dojo/json" rather than "dojo" itself. I'd like to just use the JSON package directly but unfortunately it doesn't work from rhino.

I'm having problems testing though, when I bring up test_Dialog.html it has the wrong path trying to load dojo modules such as array.js. But this error is occurring on a clean checkout too, so it's probably unrelated.

[Adam Peller]

better still, can we use native JSON routines?

[bill]

I'd like to just use the native JSON routines but unfortunately they don't work from rhino.

[Adam Peller]

They'll work if/when we upgrade to Rhino 1.7R3. Right now, we share js.jar with Shrinksafe, but we could include two different versions in our build. There's a patch pending for #9773 to upgrade Shrinksafe to 1.7R3 also, but we ran into a snag with performance problems. I guess the has() calls in dojo/json should handle cutting over to the native version when it's available.

[Rawld Gill]

In [29079]:

remove unused dependencies and avoid loading dojo/main; thanks bill; fixes #15443; !strict