Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit e6974df

Browse files
authored
[fuchsia] Remove now unnecessary diagnostics directory (#51180)
Inspect is now published using InspectSink and we can get rid of this directory. I'm unsure what "debug" and "ctrl" are for so I left them untouched. Bug: https://g-issues.fuchsia.dev/issues/320785253 ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] I added new tests to check the change I am making or feature I am adding, or the PR is [test-exempt]. See [testing the engine] for instructions on writing and running engine tests. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I signed the [CLA]. - [x] All existing and new tests are passing. If you need help, consider asking for advice on the #hackers-new channel on [Discord]. <!-- Links --> [Contributor Guide]: https://github.com/flutter/flutter/wiki/Tree-hygiene#overview [Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene [test-exempt]: https://github.com/flutter/flutter/wiki/Tree-hygiene#tests [Flutter Style Guide]: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style [testing the engine]: https://github.com/flutter/flutter/wiki/Testing-the-engine [CLA]: https://cla.developers.google.com/ [flutter/tests]: https://github.com/flutter/tests [breaking change policy]: https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes [Discord]: https://github.com/flutter/flutter/wiki/Chat
1 parent 3e9928e commit e6974df

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

shell/platform/fuchsia/dart_runner/dart_component_controller.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ bool DartComponentController::CreateAndBindNamespace() {
231231
dart_outgoing_dir_ptr_to_check_on_open_.NewRequest());
232232

233233
// Collect our standard set of directories.
234-
std::vector<std::string> other_dirs = {"debug", "ctrl", "diagnostics"};
234+
std::vector<std::string> other_dirs = {"debug", "ctrl"};
235235

236236
dart_outgoing_dir_ptr_to_check_on_open_.events().OnOpen =
237237
[this, other_dirs](zx_status_t status, auto unused) {

shell/platform/fuchsia/flutter/component_v2.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ ComponentV2::ComponentV2(
274274

275275
// Collect our standard set of directories along with directories that are
276276
// included in the cml file to expose.
277-
std::vector<std::string> other_dirs = {"debug", "ctrl", "diagnostics"};
277+
std::vector<std::string> other_dirs = {"debug", "ctrl"};
278278
for (auto dir : metadata.expose_dirs) {
279279
other_dirs.push_back(dir);
280280
}

0 commit comments

Comments
 (0)