[DevTools] Remove unnecessary checks for project types.

[DevTools] Remove unnecessary checks for project types.

This moves us closer to separating users of file system projects from network ones,
and just makes code cleaner.

Eventual goal is to remove project types completely.

BUG=670180
Committed: https://crrev.com/1ac525d4bdbec3318eae77ded7a1ff83f52f1b5f
Cr-Commit-Position: refs/heads/master@{#435856}

[dgozman, 2016-12-01 07:06:07]

The CQ bit was checked by dgozman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-01 07:06:20]

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

[dgozman, 2016-12-01 07:06:22]

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

[dgozman, 2016-12-01 07:06:22]

Could you please take a look?

[commit-bot: I haz the power, 2016-12-01 08:21:20]

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

[commit-bot: I haz the power, 2016-12-01 08:21:21]

Dry run: This issue passed the CQ dry run.

[lushnikov, 2016-12-02 01:31:32]

lgtm

https://codereview.chromium.org/2546473005/diff/1/third_party/WebKit/Source/d...
File
third_party/WebKit/Source/devtools/front_end/sources/CallStackSidebarPane.js
(left):

https://codereview.chromium.org/2546473005/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/sources/CallStackSidebarPane.js:245:
if (uiSourceCode.project().type() === Workspace.projectTypes.FileSystem)
let's bring this back for now

[dgozman, 2016-12-02 02:33:00]

https://codereview.chromium.org/2546473005/diff/1/third_party/WebKit/Source/d...
File
third_party/WebKit/Source/devtools/front_end/sources/CallStackSidebarPane.js
(left):

https://codereview.chromium.org/2546473005/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/sources/CallStackSidebarPane.js:245:
if (uiSourceCode.project().type() === Workspace.projectTypes.FileSystem)
On 2016/12/02 01:31:32, lushnikov wrote:
> let's bring this back for now

Done.

[dgozman, 2016-12-02 02:33:03]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-12-02 02:33:04]

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

[commit-bot: I haz the power, 2016-12-02 02:33:47]

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-12-02 04:28:35]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480645983987440,
"parent_rev": "760bec212621c49218b0cf9dc471981bbf39f32a", "commit_rev":
"76882ed2924ec187424a4102cd0f1909da3a1002"}

[commit-bot: I haz the power, 2016-12-02 04:29:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-02 04:31:59]

Description was changed from

==========
[DevTools] Remove unnecessary checks for project types.

This moves us closer to separating users of file system projects from network
ones,
and just makes code cleaner.

Eventual goal is to remove project types completely.

BUG=670180
==========

to

==========
[DevTools] Remove unnecessary checks for project types.

This moves us closer to separating users of file system projects from network
ones,
and just makes code cleaner.

Eventual goal is to remove project types completely.

BUG=670180

Committed: https://crrev.com/1ac525d4bdbec3318eae77ded7a1ff83f52f1b5f
Cr-Commit-Position: refs/heads/master@{#435856}
==========

[commit-bot: I haz the power, 2016-12-02 04:32:00]

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