Allow undefined data param to <appview>.connect

Allow undefined data param to <appview>.connect

BUG=239656
TBR=asargent@chromium.org for app_view_browsertest.cc, app_view.js. I'm going to add myself to OWNERs in next patch.

Committed: https://crrev.com/e33d7ceba082adcb83e22a3e5f0c4ce45a022903
Cr-Commit-Position: refs/heads/master@{#291697}

[Xi Han, 2014-08-14 18:30:17]

lgtm with nit.

https://codereview.chromium.org/464213005/diff/1/chrome/test/data/extensions/...
File chrome/test/data/extensions/platform_apps/app_view/shim/main.js (right):

https://codereview.chromium.org/464213005/diff/1/chrome/test/data/extensions/...
chrome/test/data/extensions/platform_apps/app_view/shim/main.js:71: function
testAppViewBasic(appToEmbed) {
Maybe we can change the name of this test to be more meaningful, like
testAppViewWithUndefinedDataShouldSucceed.

[Fady Samuel, 2014-08-25 14:49:03]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2014-08-25 14:50:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fsamuel@chromium.org/464213005/20001

[commit-bot: I haz the power, 2014-08-25 16:11:06]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_triggered...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-25 16:47:21]

Committed patchset #2 (20001) as 93003594bcdf4394342654d6fab521b63ccf7778

[commit-bot: I haz the power, 2014-09-10 02:35:03]

Patchset 2 (id:??) landed as
https://crrev.com/e33d7ceba082adcb83e22a3e5f0c4ce45a022903
Cr-Commit-Position: refs/heads/master@{#291697}