CB-9789: Support setting the default app locale

[dpogue]

This allows adding a defaultlocale="XX" attribute to the widget tag in config.xml to set the CFBundleDevelopmentRegion to a different language. This allows native UI (like the camera plugin) to use a non-English locale.

Requires apache/cordova-lib#466
References apache/cordova-discuss#25

/cc @jasongin for the bugfix in cleanIcons and cleanSplashScreens

[codecov-io]

Current coverage is 53.50% (diff: 50.00%)

Merging #237 into master will decrease coverage by <.01%

@@             master       #237   diff @@
==========================================
  Files            12         12          
  Lines          1026       1028     +2   
  Methods         145        145          
  Messages          0          0          
  Branches        168        169     +1   
==========================================
+ Hits            549        550     +1   
- Misses          477        478     +1   
  Partials          0          0          

Powered by Codecov. Last update c245535...1976afb

[jcesarmobile]

Can you create a new issue for the language thing and link it instead of CB-9789?
And push only the code related to that.

Are there issues open for the cleanIcons and cleanSplashScreens bugs?

Tests are failing because has no method 'getAttribute'

[dpogue]

The getAttribute method is implemented in apache/cordova-lib#466 (which isn't merged yet)

[shazron]

apache/cordova-lib#466 has been merged and should be in cordova-common 1.4.1 when it is released (we need to update the bundled cordova-common in this repo)

[shazron]

cordova-common 1.4.1 has been bundled in master, and this PR now fails one unit-test (npm run unit-tests):

Failures:

  1) prepare updateProject method should write out the app id to info plist as CFBundleIdentifier with ios-CFBundleIdentifier
   Message:
     Expected 'testpkg' to equal 'testpkg_ios'.
   Stacktrace:
     Error: Expected 'testpkg' to equal 'testpkg_ios'.
  at /Users/shaz/Documents/Git/Apache/cordova-ios/tests/spec/unit/prepare.spec.js:126:79
  at _fulfilled (/Users/shaz/Documents/Git/Apache/cordova-ios/node_modules/q/q.js:834:54)
  at self.promiseDispatch.done (/Users/shaz/Documents/Git/Apache/cordova-ios/node_modules/q/q.js:863:30)
  at Promise.promise.promiseDispatch (/Users/shaz/Documents/Git/Apache/cordova-ios/node_modules/q/q.js:796:13)
  at /Users/shaz/Documents/Git/Apache/cordova-ios/node_modules/q/q.js:556:49
  at runSingle (/Users/shaz/Documents/Git/Apache/cordova-ios/node_modules/q/q.js:137:13)
  at flush (/Users/shaz/Documents/Git/Apache/cordova-ios/node_modules/q/q.js:125:13)
  at _combinedTickCallback (internal/process/next_tick.js:67:7)
  at process._tickCallback (internal/process/next_tick.js:98:9)

[dpogue]

@shazron Thanks, I've rebased on master and updated the tests

[shazron]

@dpogue Please send a PR for a docs update as well.