[DevTools] Apps implementation.

[DevTools] Apps implementation.

DevTools frontend now runs an app: screencast app, simple undocked app or advanced app with docking support.
There will be a toolbox app soon (the always-docked part of DevTools).

This patch mostly moves the code into the new WebInspector.App hierarchy.

BUG=none
R=pfeldman@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175060

[dgozman, 2014-05-28 12:34:03]

Take a look please.

[aandrey, 2014-05-28 12:36:29]

On 2014/05/28 12:34:03, dgozman wrote:
> Apps implementation.

what is this?

[dgozman, 2014-05-28 12:39:57]

On 2014/05/28 12:36:29, aandrey wrote:
> On 2014/05/28 12:34:03, dgozman wrote:
> > Apps implementation.
> 
> what is this?

See CL description.

[pfeldman, 2014-05-28 15:12:00]

lgtm w/ nits

https://codereview.chromium.org/307623003/diff/1/Source/devtools/front_end/ma...
File Source/devtools/front_end/main/App.js (right):

https://codereview.chromium.org/307623003/diff/1/Source/devtools/front_end/ma...
Source/devtools/front_end/main/App.js:79: WebInspector.AdvancedApp = function()
These all should be different files.

https://codereview.chromium.org/307623003/diff/1/Source/devtools/front_end/ma...
Source/devtools/front_end/main/App.js:108:
this._toggleResponsiveDesignButton.toggled =
WebInspector.overridesSupport.settings.overrideDeviceMetrics.get();
I'm changing this as well...

[dgozman, 2014-05-29 10:03:40]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2014-05-29 10:04:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dgozman@chromium.org/307623003/40001

[dgozman, 2014-05-29 15:44:20]

Committed patchset #3 manually as r175060 (presubmit successful).

[apavlov, 2014-05-29 15:50:23]

Can we have a bug tracking all the UI structure changes going on lately? Us
non-involved team members barely know anything about the ultimate goal of this
activity or the design decisions behind individual patches, yet it looks like it
is going to have great impact on the overall devtools frontend architecture.

[dgozman, 2014-05-29 15:51:51]

On 2014/05/29 15:50:23, apavlov wrote:
> Can we have a bug tracking all the UI structure changes going on lately? Us
> non-involved team members barely know anything about the ultimate goal of this
> activity or the design decisions behind individual patches, yet it looks like
it
> is going to have great impact on the overall devtools frontend architecture.

You are right. Sorry, this particular patch went without bug. All the rest have
BUG=327641.

[apavlov, 2014-05-29 15:54:04]

https://codereview.chromium.org/307623003/diff/40001/Source/devtools/front_en...
File Source/devtools/front_end/main/AdvancedApp.js (right):

https://codereview.chromium.org/307623003/diff/40001/Source/devtools/front_en...
Source/devtools/front_end/main/AdvancedApp.js:15: createGlobalStatusBarItems:
function()
These apps should certainly go into different modules, since they are going to
provide different sets of contributions (e.g. see my
https://codereview.chromium.org/283063003/ (pending review), which has just
rotten to death.)