New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 591865 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

DevTools: roll closure compiler

Project Member Reported by kozyatinskiy@chromium.org, Mar 4 2016

Issue description

Last compiler update was a lot of months ago.
This is meta issue for all related frontend code changes.

 
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 4 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e92b321e3356b463f671e6b8004f8c38d76cc6ad

commit e92b321e3356b463f671e6b8004f8c38d76cc6ad
Author: kozyatinskiy <kozyatinskiy@chromium.org>
Date: Fri Mar 04 16:17:18 2016

[DevTools] Fix compile errors before closure compiler roll

BUG= 591865 
R=lushnikov@chromium.org

Review URL: https://codereview.chromium.org/1761243002

Cr-Commit-Position: refs/heads/master@{#379295}

[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/accessibility/AccessibilityNodeView.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/animation/AnimationModel.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/audits/AuditRules.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/bindings/TempFile.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/devices/DevicesView.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/elements/ElementsBreadcrumbs.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/emulation/DeviceModeToolbar.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/emulation/DevicesSettingsTab.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/resources/DatabaseModel.js
[modify] https://crrev.com/e92b321e3356b463f671e6b8004f8c38d76cc6ad/third_party/WebKit/Source/devtools/front_end/sources/FilePathScoreFunction.js

Project Member

Comment 5 by bugdroid1@chromium.org, Mar 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9c6677f6d346364b23581302dbb3a2d3905bf873

commit 9c6677f6d346364b23581302dbb3a2d3905bf873
Author: kozyatinskiy <kozyatinskiy@chromium.org>
Date: Sat Mar 05 00:58:37 2016

[DevTools] Fix more frontend compiler errors

BUG= 591865 
R=lushnikov@chromium.org

Review URL: https://codereview.chromium.org/1763193002

Cr-Commit-Position: refs/heads/master@{#379420}

[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/animation/AnimationModel.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/extensions/ExtensionAPI.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/externs.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/profiler/CPUProfileFlameChart.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/profiler/HeapSnapshotProxy.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/sources/SourceMapNamesResolver.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/timeline/LayerTreeModel.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js
[modify] https://crrev.com/9c6677f6d346364b23581302dbb3a2d3905bf873/third_party/WebKit/Source/devtools/front_end/ui_lazy/DataGrid.js

Project Member

Comment 6 by bugdroid1@chromium.org, Mar 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c

commit 9441c5e62f2a8caf30b8d5709ad374bb1dc6432c
Author: Alexey Kozyatinskiy <kozyatinskiy@chromium.org>
Date: Tue Mar 08 21:53:32 2016

[DevTools] Roll closure compiler to ToT version

Closure version:
commit e89bb281f3dbf72c20f1a86e3313831f943be688
Date:   Mon Mar 7 08:27:56 2016 -0800

Array.prototype.push supports type checking.
Better arrow function support.
Removed third_party/WebKit/Source/devtools/front_end/es6.js.

BUG= 591865 
R=alph@chromium.org, caseq@chromium.org, lushnikov@chromium.org, dgozman@chromium.org
TBR=sergeyv@chromium.org

Review URL: https://codereview.chromium.org/1774503005 .

Cr-Commit-Position: refs/heads/master@{#379920}

[delete] https://crrev.com/a76a64fc18d8ee0578b7e5588d0f948f21a1fb9b/third_party/WebKit/Source/devtools/front_end/es6.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/externs.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/platform/utilities.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/profiler/HeapSnapshotDataGrids.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/sass/ASTSourceMap.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/timeline/TimelineProfileTree.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/timeline/TimelineTreeView.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/ui/InspectorView.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/front_end/ui_lazy/DataGrid.js
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/scripts/closure/compiler.jar
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/scripts/compile_frontend.py
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/scripts/compiler-runner/closure-runner.jar
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/scripts/compiler-runner/src/org/chromium/devtools/compiler/Runner.java
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/scripts/jsdoc-validator/hashes
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/scripts/jsdoc-validator/jsdoc-validator.jar
[modify] https://crrev.com/9441c5e62f2a8caf30b8d5709ad374bb1dc6432c/third_party/WebKit/Source/devtools/scripts/jsdoc-validator/src/org/chromium/devtools/jsdoc/FileCheckerCallable.java

Status: Fixed (was: Assigned)
Status: Assigned (was: Fixed)
This roll has taken compiles from 3min to 13min on some machines.

On my machine, it was stuck compiling for >40 minutes. 

Is anyone investigating these performance regressions?
Status: Fixed (was: Assigned)

Sign in to add a comment