Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c9a71474 authored by Sundong Ahn's avatar Sundong Ahn
Browse files

Change the scope to system 

The ro.surface_flinger properties are defined in the vendor and can be
used in the vendor. If the scope is internal, the vendor must read the
value using the getproperty function, not the sysprop interface. It is
not the intent of sysprop design. so change the scope to system so that
it can be used by the vendor as well.

Bug: 112386364
Test: m -j && make checkapi
Change-Id: I7a543877eb5c63bab02fa25ef3954456c144ce6b
parent ea17f777

services/surfaceflinger/sysprop/SurfaceFlingerProperties.sysprop

+25 −25
Original line number Diff line number Diff line
@@ -36,7 +36,7 @@ owner: Platform 
prop {

    api_name: "vsync_event_phase_offset_ns"

    type: Long

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.vsync_event_phase_offset_ns"

}
@@ -44,7 +44,7 @@ prop { 
prop {

    api_name: "vsync_sf_event_phase_offset_ns"

    type: Long

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.vsync_sf_event_phase_offset_ns"

}
@@ -53,7 +53,7 @@ prop { 
prop {

    api_name: "use_context_priority"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.use_context_priority"

}
@@ -62,7 +62,7 @@ prop { 
prop {

    api_name: "max_frame_buffer_acquired_buffers"

    type: Long

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.max_frame_buffer_acquired_buffers"

}
@@ -80,7 +80,7 @@ prop { 
prop {

    api_name: "has_wide_color_display"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.has_wide_color_display"

}
@@ -90,7 +90,7 @@ prop { 
prop {

    api_name: "running_without_sync_framework"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.running_without_sync_framework"

}
@@ -108,7 +108,7 @@ prop { 
prop {

    api_name: "has_HDR_display"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.has_HDR_display"

}
@@ -117,7 +117,7 @@ prop { 
prop {

    api_name: "present_time_offset_from_vsync_ns"

    type: Long

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.present_time_offset_from_vsync_ns"

}
@@ -129,7 +129,7 @@ prop { 
prop {

    api_name: "force_hwc_copy_for_virtual_displays"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.force_hwc_copy_for_virtual_displays"

}
@@ -139,7 +139,7 @@ prop { 
prop {

    api_name: "max_virtual_display_dimension"

    type: Long

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.max_virtual_display_dimension"

}
@@ -151,7 +151,7 @@ prop { 
prop {

    api_name: "use_vr_flinger"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.use_vr_flinger"

}
@@ -161,7 +161,7 @@ prop { 
prop {

    api_name: "start_graphics_allocator_service"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.start_graphics_allocator_service"

}
@@ -171,7 +171,7 @@ prop { 
    api_name: "primary_display_orientation"

    type: Enum

    enum_values: "ORIENTATION_0|ORIENTATION_90|ORIENTATION_180|ORIENTATION_270"

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.primary_display_orientation"

}
@@ -182,7 +182,7 @@ prop { 
prop {

    api_name: "use_color_management"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.use_color_management"

}
@@ -209,7 +209,7 @@ prop { 
prop {

    api_name: "default_composition_dataspace"

    type: Long

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.default_composition_dataspace"

}
@@ -220,7 +220,7 @@ prop { 
prop {

    api_name: "default_composition_pixel_format"

    type: Integer

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.default_composition_pixel_format"

}
@@ -235,7 +235,7 @@ prop { 
prop {

    api_name: "wcg_composition_dataspace"

    type: Long

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.wcg_composition_dataspace"

}
@@ -246,7 +246,7 @@ prop { 
prop {

    api_name: "wcg_composition_pixel_format"

    type: Integer

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.wcg_composition_pixel_format"

}
@@ -258,7 +258,7 @@ prop { 
prop {

    api_name: "display_primary_red"

    type: DoubleList

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.display_primary_red"

}
@@ -266,7 +266,7 @@ prop { 
prop {

    api_name: "display_primary_green"

    type: DoubleList

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.display_primary_green"

}
@@ -274,7 +274,7 @@ prop { 
prop {

    api_name: "display_primary_blue"

    type: DoubleList

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.display_primary_blue"

}
@@ -282,7 +282,7 @@ prop { 
prop {

    api_name: "display_primary_white"

    type: DoubleList

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.display_primary_white"

}
@@ -293,7 +293,7 @@ prop { 
prop {

    api_name: "set_idle_timer_ms"

    type: Integer

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.set_idle_timer_ms"

}
@@ -303,7 +303,7 @@ prop { 
prop {

    api_name: "use_smart_90_for_video"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.use_smart_90_for_video"

}
@@ -311,7 +311,7 @@ prop { 
prop {

    api_name: "enable_protected_contents"

    type: Boolean

    scope: Internal

    scope: System

    access: Readonly

    prop_name: "ro.surface_flinger.protected_contents"

}

services/surfaceflinger/sysprop/api/system-current.txt

+40 −0
Original line number Diff line number Diff line 
// Signature format: 2.0

package android.sysprop {



  public final class SurfaceFlingerProperties {

    method public static java.util.Optional<java.lang.Long> default_composition_dataspace();

    method public static java.util.Optional<java.lang.Integer> default_composition_pixel_format();

    method public static java.util.List<java.lang.Double> display_primary_blue();

    method public static java.util.List<java.lang.Double> display_primary_green();

    method public static java.util.List<java.lang.Double> display_primary_red();

    method public static java.util.List<java.lang.Double> display_primary_white();

    method public static java.util.Optional<java.lang.Boolean> enable_protected_contents();

    method public static java.util.Optional<java.lang.Boolean> force_hwc_copy_for_virtual_displays();

    method public static java.util.Optional<java.lang.Boolean> has_HDR_display();

    method public static java.util.Optional<java.lang.Boolean> has_wide_color_display();

    method public static java.util.Optional<java.lang.Long> max_frame_buffer_acquired_buffers();

    method public static java.util.Optional<java.lang.Long> max_virtual_display_dimension();

    method public static java.util.Optional<java.lang.Long> present_time_offset_from_vsync_ns();

    method public static java.util.Optional<android.sysprop.SurfaceFlingerProperties.primary_display_orientation_values> primary_display_orientation();

    method public static java.util.Optional<java.lang.Boolean> running_without_sync_framework();

    method public static java.util.Optional<java.lang.Integer> set_idle_timer_ms();

    method public static java.util.Optional<java.lang.Boolean> start_graphics_allocator_service();

    method public static java.util.Optional<java.lang.Boolean> use_color_management();

    method public static java.util.Optional<java.lang.Boolean> use_context_priority();

    method public static java.util.Optional<java.lang.Boolean> use_smart_90_for_video();

    method public static java.util.Optional<java.lang.Boolean> use_vr_flinger();

    method public static java.util.Optional<java.lang.Long> vsync_event_phase_offset_ns();

    method public static java.util.Optional<java.lang.Long> vsync_sf_event_phase_offset_ns();

    method public static java.util.Optional<java.lang.Long> wcg_composition_dataspace();

    method public static java.util.Optional<java.lang.Integer> wcg_composition_pixel_format();

  }



  public enum SurfaceFlingerProperties.primary_display_orientation_values {

    method public String getPropValue();

    enum_constant public static final android.sysprop.SurfaceFlingerProperties.primary_display_orientation_values ORIENTATION_0;

    enum_constant public static final android.sysprop.SurfaceFlingerProperties.primary_display_orientation_values ORIENTATION_180;

    enum_constant public static final android.sysprop.SurfaceFlingerProperties.primary_display_orientation_values ORIENTATION_270;

    enum_constant public static final android.sysprop.SurfaceFlingerProperties.primary_display_orientation_values ORIENTATION_90;

  }



}