Exposed ChangeVisibility in native interface

(resolving merge error)

Fix instructions for installing protobuf-compiler

On Debian/Ubuntu need to sudo apt-get commands.

Updated grpc/proto to expose ChangeVisibility

Author: IDisposable

internal/native/empty.go

@@ -108,4 +108,6 @@ func (e *EmptyNativeInterface) SwitchToScreenIf(screenName string, shouldSwitch
 
 func (e *EmptyNativeInterface) SwitchToScreenIfDifferent(screenName string) {}
 
+func (e *EmptyNativeInterface) ChangeVisibility(objName string, show bool) {}
+
 func (e *EmptyNativeInterface) DoNotUseThisIsForCrashTestingOnly() {}

internal/native/grpc_clientmethods.go

@@ -207,6 +207,10 @@ func (c *GRPCClient) SwitchToScreenIfDifferent(screenName string) {
 	_, _ = c.client.SwitchToScreenIfDifferent(context.Background(), &pb.SwitchToScreenIfDifferentRequest{ScreenName: screenName})
 }
 
+func (c *GRPCClient) ChangeVisibility(objName string, show bool) {
+	_, _ = c.client.ChangeVisibility(context.Background(), &pb.ChangeVisibilityRequest{ObjName: objName, Show: show})
+}
+
 func (c *GRPCClient) DoNotUseThisIsForCrashTestingOnly() {
 	_, _ = c.client.DoNotUseThisIsForCrashTestingOnly(context.Background(), &pb.Empty{})
 }

internal/native/grpc_servermethods.go

@@ -224,6 +224,11 @@ func (s *grpcServer) SwitchToScreenIfDifferent(ctx context.Context, req *pb.Swit
 	return &pb.Empty{}, nil
 }
 
+func (s *grpcServer) ChangeVisibility(ctx context.Context, req *pb.ChangeVisibilityRequest) (*pb.Empty, error) {
+	s.native.ChangeVisibility(req.ObjName, req.Show)
+	return &pb.Empty{}, nil
+}
+
 func (s *grpcServer) DoNotUseThisIsForCrashTestingOnly(ctx context.Context, req *pb.Empty) (*pb.Empty, error) {
 	s.native.DoNotUseThisIsForCrashTestingOnly()
 	return &pb.Empty{}, nil

internal/native/interface.go

@@ -30,6 +30,7 @@ type NativeInterface interface {
 	DisplaySetRotation(rotation uint16) (bool, error)
 	UpdateLabelIfChanged(objName string, newText string)
 	UpdateLabelAndChangeVisibility(objName string, newText string)
+	ChangeVisibility(objName string, show bool)
 	SwitchToScreenIf(screenName string, shouldSwitch []string)
 	SwitchToScreenIfDifferent(screenName string)
 	DoNotUseThisIsForCrashTestingOnly()

internal/native/proto/native.pb.go

@@ -41,6 +41,7 @@ service NativeService {
   rpc UpdateLabelAndChangeVisibility(UpdateLabelAndChangeVisibilityRequest) returns (Empty);
   rpc SwitchToScreenIf(SwitchToScreenIfRequest) returns (Empty);
   rpc SwitchToScreenIfDifferent(SwitchToScreenIfDifferentRequest) returns (Empty);
+  rpc ChangeVisibility(ChangeVisibilityRequest) returns (Empty);
 
   // Testing
   rpc DoNotUseThisIsForCrashTestingOnly(Empty) returns (Empty);
@@ -256,3 +257,7 @@ message VideoFrame {
   int64 duration_ns = 2;
 }
 
+message ChangeVisibilityRequest {
+  string obj_name = 1;
+  bool show = 2;
+}

internal/native/proto/native.proto

@@ -679,6 +679,13 @@ func (p *NativeProxy) SwitchToScreenIfDifferent(screenName string) {
 	})
 }
 
+func (p *NativeProxy) ChangeVisibility(objName string, show bool) {
+	_ = nativeProxyClientExecWithoutArgument(p, func(client *GRPCClient) error {
+		client.ChangeVisibility(objName, show)
+		return nil
+	})
+}
+
 func (p *NativeProxy) DoNotUseThisIsForCrashTestingOnly() {
 	_ = nativeProxyClientExecWithoutArgument(p, func(client *GRPCClient) error {
 		client.DoNotUseThisIsForCrashTestingOnly()

internal/native/proto/native_grpc.pb.go

@@ -12,7 +12,7 @@ cd "$PROJECT_ROOT"
 if ! command -v protoc &> /dev/null; then
     echo "Error: protoc is not installed"
     echo "Install it with:"
-    echo "  apt-get install protobuf-compiler  # Debian/Ubuntu"
+    echo "  sudo apt-get install protobuf-compiler  # Debian/Ubuntu"
     echo "  brew install protobuf               # macOS"
     exit 1
 fi