Skip to content

Commit

Permalink
- Remove call to DartDebugger::InitDebugger in InitDartVM (flutter#4167)
Browse files Browse the repository at this point in the history
- Remove linking of dart_debugger.cc as this is not used
(all debugging is done using the service API, this debugger implementation uses the deprecated dart debugger API)
  • Loading branch information
a-siva authored Oct 3, 2017
1 parent dd65308 commit 79cb2ea
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 17 deletions.
4 changes: 0 additions & 4 deletions runtime/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -95,10 +95,6 @@ source_set("runtime") {
defines += [ "EMBED_TEST_FONT_DATA=1" ]
}

if (flutter_runtime_mode != "release") {
deps += [ "//topaz/lib/tonic/debugger" ]
}

# In AOT mode, precompiled snapshots contain the instruction buffer.
# Generation of the same requires all application specific script code to be
# specified up front. In such cases, there can be no generic snapshot.
Expand Down
1 change: 0 additions & 1 deletion runtime/dart_controller.cc
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
#include "lib/tonic/dart_message_handler.h"
#include "lib/tonic/dart_state.h"
#include "lib/tonic/dart_wrappable.h"
#include "lib/tonic/debugger/dart_debugger.h"
#include "lib/tonic/file_loader/file_loader.h"
#include "lib/tonic/logging/dart_error.h"
#include "lib/tonic/logging/dart_invoke.h"
Expand Down
9 changes: 0 additions & 9 deletions runtime/dart_init.cc
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@
#include "lib/tonic/dart_state.h"
#include "lib/tonic/dart_sticky_error.h"
#include "lib/tonic/dart_wrappable.h"
#include "lib/tonic/debugger/dart_debugger.h"
#include "lib/tonic/file_loader/file_loader.h"
#include "lib/tonic/logging/dart_error.h"
#include "lib/tonic/logging/dart_invoke.h"
Expand Down Expand Up @@ -538,14 +537,6 @@ void InitDartVM(const uint8_t* vm_snapshot_data,

FXL_CHECK(Dart_SetVMFlags(args.size(), args.data()));

#if FLUTTER_RUNTIME_MODE != FLUTTER_RUNTIME_MODE_RELEASE
if (!IsRunningPrecompiledCode()) {
TRACE_EVENT0("flutter", "DartDebugger::InitDebugger");
// This should be called before calling Dart_Initialize.
tonic::DartDebugger::InitDebugger();
}
#endif

DartUI::InitForGlobal();

// Setup embedder tracing hooks. To avoid data races, it is recommended that
Expand Down
3 changes: 0 additions & 3 deletions sky/engine/core/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,4 @@ static_library("core") {
"//topaz/lib/tonic",
]

if (flutter_runtime_mode != "release") {
public_deps += [ "//topaz/lib/tonic/debugger" ]
}
}

0 comments on commit 79cb2ea

Please sign in to comment.