This changelist introduces persistent storage for retaining settings
accessed through the SetUI service. The functionality is exposed
through two main traits. The Store trait defines an API for writing
and retrieving values, while the SettingCodec trait defines an
encoding/decoding protocol. Concrete implementations for both
have been provided, using JSON for the latter.

Note that the codecs must be updated to support new SettingData
types. The JSON codec currently only supports StringValue and
AccountSettings.

Bug: SU-167
Test: fx run-test setui_service_tests -- --test
Change-Id: Iae55c256bdeaeec957a62836abe342793320d2c4

This reverts commit 426805c5.

Reason for revert: ViewTokenPair is published in the Fuchsia SDK, so this change was a hard API transition for SDK consumers (e.g. Chromium & ANGLE).

Original change's description:
> [scenic] Convert ViewTokenPair to object
>
> This unifies the API for C++, Dart, and Rust better.  All 3 language
> bindings are pretty idiomatic now.
>
> Tested: view_token_pair_test
> SCN-1214 #comment
>
> Change-Id: I00bcbde5ff64c747ce427764cdfffdf6530e1295

TBR=abarth@google.com,jjosh@google.com,dworsham@google.com,jsankey@google.com,adamgousetis@google.com

Change-Id: Ie9219897b794e5adc38ad5e84a79ee99c1491bca
No-Presubmit: true
No-Tree-Checks: true
No-Try: true

This changelist adds adapters to the SetUI setting service. Adapters
allow us to extend the service to handle different types of settings. It
also implements the setting listen functionality.

A TestAdapter has also been added for the Unknown setting type as
an example.

Bug: SU-167
Test: fx run-test setui_service_tests -- --test
Change-Id: Ia105d77a18ba80d3624bb45d509c6ad6b211a5c3

This is the stub of the test, that is not joined to the build. This is
because a lot of logic for making this test to work is required and the
CL is big enough at it is. The following CL will do the actual
watchpoint logic and hook up this test to run.

TEST=No functional change.

Change-Id: Idee83c1e65aa55552242d5679e0ff178b8151fcc

Although slightly more verbose, being explicit makes the semantics of
WritebackWork easier to reason about (and easier to refactor).

Change-Id: If9b6bb6677eb3ec3713a31ef3eb03557c71685a2

This starts a component_test_runner which is an implementation of the
fuchsia.sys.Runner interface that can instantiate a component within an
environment configured for tests.  The specification of the test
environment and of the component under test is the component manifest
itself. This way a test author can configure different environments to
run a test in without having to modify the code for the component under
test.

This patch responds to incoming Runner requests and starts parsing the
test specification data, but doesn't yet actually attempt to create the
test environment or actually instantiate the component.

Change-Id: Ibc70a60202c026174268a109a376c072bd370a10

This should allow us to easier map a cause to a specific
frame.

Test=Only logging changed. Verified new logging format by
inspecting bug report

Change-Id: Ifa43161e45228ecc5ae5215c50bdf1fe6f5befb4

Implements Root Clock Generator Enable/Disable/Request Rate support
for 8x53.

Change-Id: I8c52e5dd2734e5c3a6d71bd1f67f9dfbf95b9a7d

Disable the usage of ZX_WAIT_ASYNC_REPEATING, and print a message if
some previously built binary attempts to use it, to aid in debugging.

A subsequent CL will remove the implementation code after a refactor.

Bug: ZX-3090

Test: CQ.
Change-Id: I52f7a51a8b5ddcffcfa528780ba57046fe88bd97

This is step 4 of 4 in a soft transition to migrate the Auth Provider
between repos.

AUTH-164 #comment

Change-Id: I65871f247eee4a23628a5adfcf5bd8c9464ec7be

* this will reduce the amount of code in feedback_agent_unittest
* this will allow us to shared the StubLogListener with the upcoming log_listener_unittest

TESTED=`fx run-test feedback_agent_tests -t feedback_agent_unittest`

Change-Id: I97d1312a4980053193848332ce12f01a076a0411

Also add an explicit option in environment() to enable a per-variant
libprefix in all variants of a toolchain (or none).  By default, each
variant that has the "instrumented" tag gets "$variant/" appended to its
${toolchain.libprefix} value.  Remove the "asan" variant's explicit
toolchain_vars.libprefix setting, which is now redundant.

The dynamic linker is compiled to request the libprefix from the loader
service, and all executables have the prefixed name in PT_INTERP so they
get the same variant's dynamic linker.

Bug: BLD-429 #done
Bug: ZX-3624 #done
Change-Id: Ia74bc8d449aa6ce3ded00df8940b057e790486f4

There are situation in which we're being delayed in the 150-200ms range.
Until that situation is resolved bump the ringbuffer size 'higher than
should ever be needed'.

Test: Built+ran end to end test. CI repeatedly passing will be the final
      test.
Bug: ZX-3973
Change-Id: Ic240384192a7c418de8b37249a7379045d8e6c27

Additionally, some minor related changes:
- Remove unused "FormatMinfs" function.
- Plumb the "Should we check filesystems?" boolean through explicit
arguments instead of environment variables.
- Make minfs superblock validation more strict. This validates that
clients can't format a filesystem with N blocks, and either mount
of check the consistency of a filesystem with M blocks, for M < N.
This change, along with the fshost tests, acts as a regression for
a bug where the consistency checker only inspected a subsection of the
filesystem.

Change-Id: I15470e9529553526011c8fba2d8a6f5987b8e1a6

This reflects FTL library code refactors to:
- map page cache improvements
- Change FTL-NDM interface to direct function calls

Change-Id: If3668b3fb079cc18c6d4b28b122c74789fff0e78

These were only used by ViewManager, which is no longer used.

SCN-1318 #comment

Change-Id: Ie9778af954992b9bef96079cdc67281d4da3c1e5

This required to extract a bit the "breakpoint type" from the settings.
This change is a no-op but requires to touch quite a bit of files because
of our "use the ipc type as our internal storage" strategy.

TEST=host + target tests.

Change-Id: Ia24efea188224a330e38ff1c5a44585f0021da8b

When realm's job dies, that realm becomes useless because it cannot
create any more components aor realms inside of it, so better to kill it
and lets realm owner know that realm was killed so that they can exit
gracefully or create a new one.

This will also complete bug CF-706 because now we have checks in place
which ensure that appmgr doesn't crashdue to closed handles  when there
is a oom.

CF-706 #done

TEST=fx run-test appmgr_integration_tests

Change-Id: I4a06bc73d76fe44a075d63fc3f0f762a94a19a14

This code was all written from scratch for Fuchsia.
Move it to system/ulib/c to make its provenance more clear.

Test: no effect on the build
Change-Id: I6b4105157c9233ad3b7ec41124c35cbf144c2ae0

This unifies the API for C++, Dart, and Rust better.  All 3 language
bindings are pretty idiomatic now.

Tested: view_token_pair_test
SCN-1214 #comment

Change-Id: I00bcbde5ff64c747ce427764cdfffdf6530e1295

This is the start of fdio conversion from C to LLCPP.

TEST: Compiles
Change-Id: Ia546d5199d9b73677cb30ae38b445f45225fea34

Implements fx build tools/fidlgen_llcpp_zircon:validate.
In addition, adds the validation pass to the fuchsia build.

Test: Use fuchsia-mem:llcpp somewhere, then change it. Observe that the
build fails with an error prompting the user to re-run bindings
generation.

Change-Id: I376516c794120fa5faaaf4e2afaffc0283604d09

Change-Id: I24a609e37a42c956514076b330b924464e73ed41

ES-198 #done

Change-Id: I5dcdec7f10067c90d0e8fc18d17069f26bbfeb59

TEST: tried importing the generated fuchsia.mem in some test and it
builds.

Bug: ZX-3874

Change-Id: Ibb963f4581e0f9fc1bfca862d36b6fb0a76335c0

This support is no longer being used.
All drivers are now binding to the generic I2C either directly
or as a composite device.

TEST: Touch screen works on astro and cleo. runtests -t platform-bus-test.
Change-Id: If189943cdabfebc2fab2df1010881daf458ada6b

When the number of interfaces grows, the vector containing the interfaces
grows with it.  This involves the interface being moved.

When you generate an interface for the first time, each interface method is
given a pointer to the enclosing interface.

When the interface vector is resized, the interface method's pointer is
dangling.

This CL fixes the issue in two ways:

- The move constructor for the interface now creates an updated interface
method instead of reusing the old one.

- Interface vector initialization now reserves enough space for all of the
interfaces, so that moves will no longer be necessary.

Change-Id: Ifa4c51f21bb3dc194695d5af5029a46b862ae907

* there is no need to run the underlying delayed task if the log collection already completed

TESTED=manually removed the complete_ok() in Done() and checked the timeout callback is no longer called

Change-Id: Ic844d6ca95ebe65d31651b080e4e26a4cece7870

This implements half the FIDL generation for LLCPP bindings.  It
can't be done fully the straightforward way until we can build the
fidlgen tool (Go).  Instead, this integrates it halfway by doing the
JSON generation part.  Then instead of running fidlgen, it produces
a build API module (JSON) describing how it would have run fidlgen,
and expects to find the fidlgen output files in the source tree.
Later integration will take that information and do the regeneration
and/or check that it doesn't need to be done, so that the
higher-level build can verify what the Zircon build alone can't yet.

Bug: BLD-442 #comment fidl_library() support for llcpp
Bug: BLD-353 #comment Switchback for FIDL LLCPP bindings before unification
Change-Id: Ied2e9972da1c1bb4957e29f98c721d96d8839d97

1. Because Google app doesn't require callback interface between the FTL and
   the NDM, removed it to increase clarity and decrease lines of code.
2. Changed some local variables from 'ftl' to 'ftl_cfg' to better differentiate
   between control block handles and temporary configuration structures.
3. Removed ENOMEM definition from 'kernel.h'.
4. Removed FsError() calls from routines in 'fsmem.c' since their callers
   invoke FsError2() if there is an error.
5. Introduced 'ftl_pri.h' (for internal use) and 'ftl_pub.h' (for driver).
   Following check-ins will consolidate other headers into those files.

Change-Id: I6c27dfd24755c5fda1e0cb5ba09e8707ce31b653

Create an abstract "BlockDeviceInterface" class, and
add unit tests for it.

Change-Id: Ie61268a11a217a27e23fabfbdd5873d01e734a8a

FLK-187
FLK-188

Change-Id: I25a5a2c72bd5a5348953e315142bc2bdc8fff610

This changelist encapsulates the SetUIService into a struct/impl.
This structure will facilitate future changes where state and logical
units will need to exist across channels.

Test: fx run-test setui_service_tests -- --test
Bug: SU-167
Change-Id: I850d75321a7d9f830bc611ba4ffc3bfcb6680534

Renamed session update function and moved CommandContext creation to
be assigned the trace ID, rather than guess the next ID.

Tested:
- updated unittests
- ran astro

Change-Id: I421ffe839c9add7440361209cc881c66c80e0843

Previously, it was possible for a hot-looping task to be
created as a result of a stale PEER_CLOSED signal arriving
for a new, still-open channel. This change prevents this
by ensuring that the `PEER_CLOSED` signal gets reset and
re-waited in the event that a read or write occurs without
resulting in PEER_CLOSED.

This change fixes the issue for zx::Fifo and zx::Channel,
and a followup change will be opened for zx::Socket.

Change-Id: I0220b084320158d20d624b75c4267b0f87e6687f

Instead of loading bundled defaults from the system_image package
start requesting the config-data feature and also looking for the
settings from the config-data path.

MTWN-262 #done

TEST=none

Change-Id: Ia1cd55f6c8bb1f8ae5546d651b50492642f867c9

Previously this library only supported connecting to Cobalt via the
older, heavier technique of passing a VMO containing the registry bytes.
Now it also supports connecting via the newer, lighter technique of
passing just the project name.

Test:
I added a new test case to cobalt_logger_unittest.cc

Change-Id: Id90ff6fc67faf73d7b19d9dc8a9691adee48de49

Update the ARP table on receiving gratuitous ARP requests and responses,
as specified in RFC 2002.

NET-2102 #done

Change-Id: I8f78200deb2f36f5cfff46bfbd9baec13f2d161f

Cpp Style (anonymous namespace for tests, and CamelCase for function
names).
Use zx type safe classes instead of raw zx_handle_t.
Prevent leak of handles by early release of a[0] on failure.

TEST=core/handle-transfer-test

Change-Id: Ic9ec5346d70afb9846f233c814ad709a3b35addd

BUG: FLK-200 #done

- Test case "TestSucceedsBeforeTimeoutFires" was causing flake due to
low timeout value (5s). Bumped value to 60s to prevent false negatives
in timeout test case.

Change-Id: Id7a999a3025291db12c7c39a3b1c20ced154e89c