OSDN Git Service

Camera2: Rework the FPS range vs. FPS single setting detection
authorEino-Ville Talvala <etalvala@google.com>
Thu, 14 Nov 2013 18:32:13 +0000 (10:32 -0800)
committerEino-Ville Talvala <etalvala@google.com>
Thu, 14 Nov 2013 18:32:13 +0000 (10:32 -0800)
commitd8a62e25ba6520c2531c7a3d32cc8066e1dab776
treedbf4856e2a0f6781f3287fed2534f7e358c6b053
parent4215e6166fca9f87a6e9e848b3dfd4ab0d25c954
Camera2: Rework the FPS range vs. FPS single setting detection

Give up on current approach of writing out consistent FPS values
into parameters that will be read back by the app.

- Preserve app's latest set parameters exactly, and compare against
  them when detecting if a new FPS range or single FPS value has been
  selected.

- Since get() returns exactly what was set(), it doesn't matter if the
  app calls getParameters() before its next setParameters(), in terms
  of retriggering FPS selection logic. Before, the behavior varied
  depending on whether the app re-read the parameters.

- As before, if app changes both range and single FPS in a single set
  call, the range set wins. Otherwise, the value that has changed more
  recently is used.

Bug: 11570973
Change-Id: I72b5e60c3f60e88d55127dd1bda87e26eaf929c6
services/camera/libcameraservice/api1/client2/Parameters.cpp
services/camera/libcameraservice/api1/client2/Parameters.h