Files.app: Split background.js

Files.app: Split background.js

This patch splits the background.js into the 3 files (background_base.js, app_window_wrapper.js, background.js), extracting the common classes.

This patch itself doesn't change any functionality, but the extracted files will be used in the separated audio player app in near future.

BUG=375039
TEST=browser_test passes

Committed: https://crrev.com/479878c12c436c0bec667483df29fe1f23b710a2
Cr-Commit-Position: refs/heads/master@{#294998}

[yoshiki, 2014-09-11 09:33:39]

yoshiki@chromium.org changed reviewers:
+ hirono@chromium.org

[yoshiki, 2014-09-11 09:34:51]

@hirono, PTAL. Thanks.

[hirono, 2014-09-11 12:52:12]

lgtm with a nit.

https://codereview.chromium.org/562103002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/background/js/background.js (right):

https://codereview.chromium.org/562103002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/background/js/background.js:477:
audioPlayer.eaunch(playlist, false, function(appWindow) {
launch?

[yoshiki, 2014-09-12 02:04:34]

Thanks!

https://codereview.chromium.org/562103002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/background/js/background.js (right):

https://codereview.chromium.org/562103002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/background/js/background.js:477:
audioPlayer.eaunch(playlist, false, function(appWindow) {
On 2014/09/11 12:52:11, hirono wrote:
> launch?

Done.

[yoshiki, 2014-09-12 02:04:37]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-09-12 02:09:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/562103002/20001

[commit-bot: I haz the power, 2014-09-12 02:31:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-12 02:31:46]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yoshiki, 2014-09-12 08:26:01]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-09-12 08:27:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/562103002/40001

[commit-bot: I haz the power, 2014-09-12 09:50:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-12 09:50:33]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yoshiki, 2014-09-14 03:37:47]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-09-14 03:38:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/562103002/40001

[commit-bot: I haz the power, 2014-09-14 03:51:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-14 03:51:55]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yoshiki, 2014-09-16 01:25:04]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-09-16 01:26:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/562103002/60001

[commit-bot: I haz the power, 2014-09-16 03:30:30]

Committed patchset #4 (id:60001) as dd0d7a64b28b379c69f2830e1c6b02a39c4eefab

[commit-bot: I haz the power, 2014-09-16 03:32:05]

Patchset 4 (id:??) landed as
https://crrev.com/479878c12c436c0bec667483df29fe1f23b710a2
Cr-Commit-Position: refs/heads/master@{#294998}

[yoshiki, 2014-09-16 06:23:35]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/563743004/ by yoshiki@chromium.org.

The reason for reverting is: This patch breaks the test..