doc parser breaks if module has "function" in any of the deps

[dante]

The regexp which matches deps and unwraps to AMD format is brittle, and breaks when it encounters the word "function".

$text = preg_replace_callback('/(define\((.*?)function(.*?)\{(.*)\}\);)/s', "_amd_pseudodoc_fix", $text, 1);

will accidentally stop on the first occurance of "function" in the list of deps:

define(["dojo", "dojox/lang/functional"], function(dojo, df){ 

});

becomes

;(function_al

in the unwrapp_fix callback. Need to fix regexp to match only the stuff in []'s.

[dante]

(In [25678]) refs #13345 - adding a test showing how this is broken, and attempting to fix

[dante]

(In [25679]) refs #13345 - whitespace from last ci

[dante]

(In [25688]) fixes #13345 - thanks snover for a quickfix and some regexp foo!

[dante]

(In [25696]) refs #13345 - better regexp from snover