Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(103)

Issue 1398433002: Remove deprecated dart:profiler library (Closed)

Created:
5 years, 2 months ago by Cutch
Modified:
5 years, 2 months ago
Reviewers:
turnidge
CC:
reviews_dartlang.org, ricow1, vm-dev_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -106 lines) Patch
D runtime/lib/profiler_sources.gypi View 1 chunk +0 lines, -12 lines 0 comments Download
M runtime/vm/BUILD.gn View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/vm/bootstrap.h View 2 chunks +0 lines, -2 lines 0 comments Download
M runtime/vm/bootstrap.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M runtime/vm/bootstrap_natives.cc View 1 chunk +0 lines, -5 lines 0 comments Download
M runtime/vm/object.h View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/vm/object.cc View 1 1 chunk +0 lines, -5 lines 0 comments Download
M runtime/vm/object_store.h View 4 chunks +0 lines, -6 lines 0 comments Download
M runtime/vm/object_store.cc View 1 chunk +0 lines, -1 line 0 comments Download
M runtime/vm/vm.gypi View 4 chunks +0 lines, -43 lines 0 comments Download
D sdk/lib/profiler/profiler.dart View 1 chunk +0 lines, -14 lines 0 comments Download
D sdk/lib/profiler/profiler_sources.gypi View 1 chunk +0 lines, -10 lines 0 comments Download
M tools/create_sdk.py View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
Cutch
5 years, 2 months ago (2015-10-07 16:45:04 UTC) #2
turnidge
lgtm
5 years, 2 months ago (2015-10-07 16:46:56 UTC) #3
Cutch
Committed patchset #2 (id:20001) manually as 535a9715ac9e8506e8fc6e9b274a1e0955738060 (presubmit successful).
5 years, 2 months ago (2015-10-07 17:01:38 UTC) #4
Kevin Millikin (Google)
On 2015/10/07 17:01:38, Cutch wrote: > Committed patchset #2 (id:20001) manually as > 535a9715ac9e8506e8fc6e9b274a1e0955738060 (presubmit ...
5 years, 2 months ago (2015-10-09 10:24:48 UTC) #5
Cutch
5 years, 2 months ago (2015-10-09 13:09:10 UTC) #6
Message was sent while issue was closed.
> I think that this was removed too quickly, since Dart itself was using
> dart:profiler in the analyzer up until 30 minutes before this change.  A ton
of
> stuff depends on the analyzer, sometimes transitively, and that will all be
> broken in builds of the VM after this change.
> 

This library has been deprecated since 1.11. All teams have had two release
cycles to stop using a deprecated library. It was supposed to be removed in 1.12
but I forgot to do it then. The alternative (dart:developer) has also been
around since 1.11 so unless someone is taking a 1.13 version of the analyzer and
running it on a 1.10 release of the VM they will be fine.

> We also didn't bump the analyzer version when removing import 'dart:profiler',
> so it's not trivial to figure out which version to ask for to make sure you
get
> a version that works with a modern build of the VM.

This is a solvable problem but bumping the sdk requirement when the next version
of the analyzer package is released.

Powered by Google App Engine
This is Rietveld 408576698