Move cuttlefish/common/libs/utils:result* to cuttlefish/result:result*

The `common` naming was an artifact from a time when this code was used
for both device and host builds. The new location is more central and
shorter to include, which aligns with it being a pervasive type in the
codebase.

Creating a distinct directory gives more room to separate `result.h`
into multiple files, with the following benefits:

- Separating the `Result` type from the `EXPECT` logic allows depending
  on the `Result` type without depending on every type with special
  handling in the macros.
- Separating out `CF_EXPECTF` makes it possible to depend on `Result`
  without depending on `@fmt`.
- Separate files are easier to understand in isolation.

The original headers and build targets are left as forwarding
dependencies to the new location.

Bug: b/471037778

Author: Databean

base/cvd/cuttlefish/common/libs/utils/BUILD.bazel

@@ -275,42 +275,18 @@ cf_cc_library(
 
 cf_cc_library(
     name = "result",
-    srcs = [
-        "result.cpp",
-    ],
-    hdrs = [
-        "result.h",
-    ],
-    deps = [
-        "//libbase",
-        "@fmt",
-    ],
-)
-
-cf_cc_test(
-    name = "result_test",
-    srcs = [
-        "result_test.cpp",
-    ],
+    hdrs = ["result.h"],
     deps = [
-        ":result",
-        ":result_matchers",
-        "//libbase",
-        "@googletest//:gtest",
-        "@googletest//:gtest_main",
+        "//cuttlefish/result",
     ],
 )
 
 cf_cc_library(
     name = "result_matchers",
     testonly = 1,
-    hdrs = [
-        "result_matchers.h",
-    ],
+    hdrs = ["result_matchers.h"],
     deps = [
-        ":result",
-        "//libbase",
-        "@googletest//:gtest",
+        "//cuttlefish/result:result_matchers",
     ],
 )