Removes loadByPath from charm model

Removes loadByPath from charm model

loadByPath is an option for loading a charm in Charm.sync, but we never pass a
charmstore to the model's load function in the app. We do pass an environment,
which is handled by a different case.
-Delete the "loadByPath" case for the sync function
-Remove tests regarding load/sync and the charm store
-Remove tests regarding updating data from charm store based loads in endpoints
(other tests cover updating data from service's and the env).

https://code.launchpad.net/~jcsackett/juju-gui/remove-old-api-charm-model-2/+merge/177391

(do not edit description out of merge proposal)

[j.c.sackett, 2013-07-29 14:43:44]

Please take a look.

[bac, 2013-07-29 16:13:34]

LGTM, code removal FTW

[rharding, 2013-07-29 16:26:33]

LGTM

https://codereview.appspot.com/12031043/diff/1/test/test_endpoints.js
File test/test_endpoints.js (left):

https://codereview.appspot.com/12031043/diff/1/test/test_endpoints.js#oldcode616
test/test_endpoints.js:616: 
I've started to leave these. I've been asked by some devs because their editors
will do a better job with folding with the one space between.

[j.c.sackett, 2013-07-29 17:04:22]

*** Submitted:

Removes loadByPath from charm model

loadByPath is an option for loading a charm in Charm.sync, but we never pass a
charmstore to the model's load function in the app. We do pass an environment,
which is handled by a different case.
-Delete the "loadByPath" case for the sync function
-Remove tests regarding load/sync and the charm store
-Remove tests regarding updating data from charm store based loads in endpoints
(other tests cover updating data from service's and the env).

R=bac, rharding
CC=
https://codereview.appspot.com/12031043