diff --git a/shell/common/engine.cc b/shell/common/engine.cc index d42b8e1a40f8a9b44d2bccb01da8a78a408bd3f6..7345ac2055da893228fcb05082399a2e0ab2ff23 100644 --- a/shell/common/engine.cc +++ b/shell/common/engine.cc @@ -128,9 +128,6 @@ Engine::RunStatus Engine::Run(RunConfiguration configuration) { return RunStatus::Failure; } - last_entry_point_ = configuration.GetEntrypoint(); - last_entry_point_library_ = configuration.GetEntrypointLibrary(); - auto isolate_launch_status = PrepareAndLaunchIsolate(std::move(configuration)); if (isolate_launch_status == Engine::RunStatus::Failure) { @@ -506,12 +503,4 @@ void Engine::HandleAssetPlatformMessage(fml::RefPtr message) { response->CompleteEmpty(); } -const std::string& Engine::GetLastEntrypoint() const { - return last_entry_point_; -} - -const std::string& Engine::GetLastEntrypointLibrary() const { - return last_entry_point_library_; -} - } // namespace flutter diff --git a/shell/common/engine.h b/shell/common/engine.h index 1f219c053b3f1fc4267fdb2f2cd994a5c98ece29..cc9d1bcf68761467bfb581318fa27a99036172a2 100644 --- a/shell/common/engine.h +++ b/shell/common/engine.h @@ -717,18 +717,6 @@ class Engine final : public RuntimeDelegate, PointerDataDispatcher::Delegate { // |PointerDataDispatcher::Delegate| void ScheduleSecondaryVsyncCallback(fml::closure callback) override; - //---------------------------------------------------------------------------- - /// @brief Get the last Entrypoint that was used in the RunConfiguration - /// when |Engine::Run| was called. - /// - const std::string& GetLastEntrypoint() const; - - //---------------------------------------------------------------------------- - /// @brief Get the last Entrypoint Library that was used in the - /// RunConfiguration when |Engine::Run| was called. - /// - const std::string& GetLastEntrypointLibrary() const; - private: Engine::Delegate& delegate_; const Settings settings_; @@ -740,8 +728,6 @@ class Engine final : public RuntimeDelegate, PointerDataDispatcher::Delegate { // is destructed first. std::unique_ptr pointer_data_dispatcher_; - std::string last_entry_point_; - std::string last_entry_point_library_; std::string initial_route_; ViewportMetrics viewport_metrics_; std::shared_ptr asset_manager_; diff --git a/shell/common/shell.cc b/shell/common/shell.cc index 2b1b0e07915c14387d4c1583bf0a98c4d973c219..cc88bb9095ddf3a7652602a25da8f3a0e6accfa3 100644 --- a/shell/common/shell.cc +++ b/shell/common/shell.cc @@ -545,10 +545,14 @@ fml::WeakPtr Shell::GetRasterizer() { return weak_rasterizer_; } +// TODO(dnfield): Remove this when either Topaz is up to date or flutter_runner +// is built out of this repo. +#ifdef OS_FUCHSIA fml::WeakPtr Shell::GetEngine() { FML_DCHECK(is_setup_); return weak_engine_; } +#endif // OS_FUCHSIA fml::WeakPtr Shell::GetPlatformView() { FML_DCHECK(is_setup_); @@ -1267,9 +1271,6 @@ bool Shell::OnServiceProtocolRunInView( RunConfiguration configuration(std::move(isolate_configuration)); - configuration.SetEntrypointAndLibrary(engine_->GetLastEntrypoint(), - engine_->GetLastEntrypointLibrary()); - configuration.AddAssetResolver( std::make_unique(fml::OpenDirectory( asset_directory_path.c_str(), false, fml::FilePermission::kRead))); diff --git a/shell/common/shell.h b/shell/common/shell.h index 2f5a45923eea039da31783b40eda630779d5b72b..e4ee6960389ad06a5f6f170582e5487a53bed502 100644 --- a/shell/common/shell.h +++ b/shell/common/shell.h @@ -216,6 +216,9 @@ class Shell final : public PlatformView::Delegate, /// fml::WeakPtr GetRasterizer(); +// TODO(dnfield): Remove this when either Topaz is up to date or flutter_runner +// is built out of this repo. +#ifdef OS_FUCHSIA //------------------------------------------------------------------------------ /// @brief Engines may only be accessed on the UI thread. This method is /// deprecated, and implementers should instead use other API @@ -224,6 +227,7 @@ class Shell final : public PlatformView::Delegate, /// @return A weak pointer to the engine. /// fml::WeakPtr GetEngine(); +#endif // OS_FUCHSIA //---------------------------------------------------------------------------- /// @brief Platform views may only be accessed on the platform task diff --git a/shell/common/shell_unittests.cc b/shell/common/shell_unittests.cc index e497e4609ba267186a76facbe96cf7e557f16654..942db37c943c67eb6231e56e03d2949b1df0d43f 100644 --- a/shell/common/shell_unittests.cc +++ b/shell/common/shell_unittests.cc @@ -202,27 +202,6 @@ TEST_F(ShellTest, SecondaryIsolateBindingsAreSetupViaShellSettings) { ASSERT_FALSE(DartVMRef::IsInstanceRunning()); } -TEST_F(ShellTest, LastEntrypoint) { - ASSERT_FALSE(DartVMRef::IsInstanceRunning()); - auto settings = CreateSettingsForFixture(); - auto shell = CreateShell(settings); - ASSERT_TRUE(ValidateShell(shell.get())); - - auto configuration = RunConfiguration::InferFromSettings(settings); - ASSERT_TRUE(configuration.IsValid()); - std::string entry_point = "fixturesAreFunctionalMain"; - configuration.SetEntrypoint(entry_point); - - fml::AutoResetWaitableEvent main_latch; - AddNativeCallback( - "SayHiFromFixturesAreFunctionalMain", - CREATE_NATIVE_ENTRY([&main_latch](auto args) { main_latch.Signal(); })); - - RunEngine(shell.get(), std::move(configuration)); - main_latch.Wait(); - EXPECT_EQ(entry_point, shell->GetEngine()->GetLastEntrypoint()); -} - TEST(ShellTestNoFixture, EnableMirrorsIsWhitelisted) { if (DartVM::IsRunningPrecompiledCode()) { // This covers profile and release modes which use AOT (where this flag does