DevTools: remove release mode from runtime.

DevTools: remove release mode from runtime.

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

[pfeldman, 2016-07-27 21:55:14]

pfeldman@chromium.org changed reviewers:
+ dgozman@chromium.org

[dgozman, 2016-07-27 22:10:46]

lgtm

https://codereview.chromium.org/2191473005/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/Runtime.js (right):

https://codereview.chromium.org/2191473005/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/Runtime.js:811: return
loadScriptsPromise(this._descriptor.scripts.map(this._modularizeURL, this),
this._remoteBase());
This makes it hard to develop remote modules - they will be loaded from the
cloud. Let's remove "remote" bit from descriptor in debug build?

[pfeldman, 2016-07-27 22:12:33]

On 2016/07/27 22:10:46, dgozman wrote:
> lgtm
> 
>
https://codereview.chromium.org/2191473005/diff/1/third_party/WebKit/Source/d...
> File third_party/WebKit/Source/devtools/front_end/Runtime.js (right):
> 
>
https://codereview.chromium.org/2191473005/diff/1/third_party/WebKit/Source/d...
> third_party/WebKit/Source/devtools/front_end/Runtime.js:811: return
> loadScriptsPromise(this._descriptor.scripts.map(this._modularizeURL, this),
> this._remoteBase());
> This makes it hard to develop remote modules - they will be loaded from the
> cloud. Let's remove "remote" bit from descriptor in debug build?

Done.

[pfeldman, 2016-07-27 22:13:19]

On 2016/07/27 22:12:33, pfeldman wrote:
> On 2016/07/27 22:10:46, dgozman wrote:
> > lgtm
> > 
> >
>
https://codereview.chromium.org/2191473005/diff/1/third_party/WebKit/Source/d...
> > File third_party/WebKit/Source/devtools/front_end/Runtime.js (right):
> > 
> >
>
https://codereview.chromium.org/2191473005/diff/1/third_party/WebKit/Source/d...
> > third_party/WebKit/Source/devtools/front_end/Runtime.js:811: return
> > loadScriptsPromise(this._descriptor.scripts.map(this._modularizeURL, this),
> > this._remoteBase());
> > This makes it hard to develop remote modules - they will be loaded from the
> > cloud. Let's remove "remote" bit from descriptor in debug build?
> 
> Done.

No, wait, not done. How can I do that? :)

[pfeldman, 2016-07-27 22:53:21]

The CQ bit was checked by pfeldman@chromium.org

[pfeldman, 2016-07-27 22:53:22]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2191473005/#ps20001
(title: "lcean")

[commit-bot: I haz the power, 2016-07-27 22:53:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-28 00:09:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-28 00:14:07]

Description was changed from

==========
DevTools: remove release mode from runtime.
==========

to

==========
DevTools: remove release mode from runtime.

Committed: https://crrev.com/c7499c30537bdd50baeb177aeff3e85e13f3ea0f
Cr-Commit-Position: refs/heads/master@{#408287}
==========

[commit-bot: I haz the power, 2016-07-28 00:14:08]

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