Fix double loading of cms-modules.js.

**Ticket:** [TNL-272](https://openedx.atlassian.net/browse/TNL-272)
**Acceptance criteria:**
 * The module cms-modules.XXXXX.js should be loaded only once on the unit edit page in Studio.
 * Performance improvement should be measured before and after this story is completed. 

**sandbox:** http://studio.polesye.m.sandbox.edx.org/

Studio -> Content -> Pages
Before: 104 requests, 1.6MB transferred, load: about 6.7 - 7.2 s, DOMContentLoaded: about 1.1 s
After: 103 requests, 1.4MB transferred, load: about 6.1-6.4 s, DOMContentLoaded: about 1.1 s

@cahrens , @tymofij please review.
Should you pay attention to the return value from wait_xmodules_to_load()? And if result was false from load_requrejs_modules(javascript), is there any point to calling wait_xmodules_to_load()?

It looks like the return value from wait_for_requirejs isn't actually used, so this is probably a moot question. However, I'd expect to see something more like

```code```
result = load_requrejs_modules(javascript)
if result and 'xmodule' in dependencies:
    result = wait_xmodules_to_load()

return result
```code