Add base support for camera user feedback

By default it does nothing. Implementations need to provide their own
FeedbackHelper class that handles submitting feedback for the app.

Bug: 8155939
Change-Id: I33536267fcc4c3f9aecd957c6064963f5e9d1b6c

1 files changed, 82 insertions, 76 deletions

diff --git a/res/values/strings.xml b/res/values/strings.xml
index 8bcaec830..a52a4b15c 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -511,144 +511,144 @@
     <!-- Title of category for background replacement effects. [CHAR LIMIT=26] -->
     <string name="effect_background">BACKGROUND</string>
 
-    <!-- The shutter button. [CHAR LIMIT = NONE] -->
+    <!-- The shutter button. [CHAR LIMIT=NONE] -->
     <string name="accessibility_shutter_button">Shutter</string>
-    <!-- The menu button. [CHAR LIMIT = NONE] -->
+    <!-- The menu button. [CHAR LIMIT=NONE] -->
     <string name="accessibility_menu_button">Menu button</string>
-    <!-- The mode picker to switch between camera, video and panorama. [CHAR LIMIT = NONE] -->
+    <!-- The mode picker to switch between camera, video and panorama. [CHAR LIMIT=NONE] -->
     <string name="accessibility_mode_picker">Camera, video, or panorama selector</string>
-    <!-- The check box in camera settings, such as store location. [CHAR LIMIT = NONE] -->
+    <!-- The check box in camera settings, such as store location. [CHAR LIMIT=NONE] -->
     <string name="accessibility_check_box">%1$s check box</string>
-    <!-- The button to switch to Camera mode. [CHAR LIMIT = NONE] -->
+    <!-- The button to switch to Camera mode. [CHAR LIMIT=NONE] -->
     <string name="accessibility_switch_to_camera">Switch to photo</string>
-    <!-- The button to switch to Video mode. [CHAR LIMIT = NONE] -->
+    <!-- The button to switch to Video mode. [CHAR LIMIT=NONE] -->
     <string name="accessibility_switch_to_video">Switch to video</string>
-    <!-- The button to switch to Panorama mode. [CHAR LIMIT = NONE] -->
+    <!-- The button to switch to Panorama mode. [CHAR LIMIT=NONE] -->
     <string name="accessibility_switch_to_panorama">Switch to panorama</string>
-    <!-- The button to switch to the Photo Sphere mode. [CHAR LIMIT = NONE] -->
+    <!-- The button to switch to the Photo Sphere mode. [CHAR LIMIT=NONE] -->
     <string name="accessibility_switch_to_photo_sphere">Switch to Photo Sphere</string>
-    <!-- The button to switch to the high quality mode. [CHAR LIMIT = NONE] -->
+    <!-- The button to switch to the high quality mode. [CHAR LIMIT=NONE] -->
     <string name="accessibility_switch_to_gcam">Switch to high quality</string>
-    <!-- The button to switch to the Re-Focus mode. [CHAR LIMIT = NONE] -->
+    <!-- The button to switch to the Re-Focus mode. [CHAR LIMIT=NONE] -->
     <string name="accessibility_switch_to_refocus">Switch to Refocus</string>
-    <!-- The button in review mode indicating that the photo taking, video recording, and panorama saving session should be canceled [CHAR LIMIT = NONE] -->
+    <!-- The button in review mode indicating that the photo taking, video recording, and panorama saving session should be canceled [CHAR LIMIT=NONE] -->
     <string name="accessibility_review_cancel">Review cancel</string>
-    <!-- The button in review mode indicating that the taken photo/video is OK to be attached/uploaded [CHAR LIMIT = NONE] -->
+    <!-- The button in review mode indicating that the taken photo/video is OK to be attached/uploaded [CHAR LIMIT=NONE] -->
     <string name="accessibility_review_ok">Review done</string>
     <!-- button in review mode indicate the user want to retake another photo/video for attachment [
-CHAR LIMIT = NONE] -->
+CHAR LIMIT=NONE] -->
     <string name="accessibility_review_retake">Review retake</string>
     <!-- Default text for a button that can be toggled on and off. -->
     <string name="capital_on">ON</string>
     <!-- Default text for a button that can be toggled on and off. -->
     <string name="capital_off">OFF</string>
 
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_off">Off</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_500">0.5 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_1000">1 second</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_1500">1.5 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_2000">2 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_2500">2.5 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_3000">3 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_4000">4 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_5000">5 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_6000">6 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_10000">10 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_12000">12 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_15000">15 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_24000">24 seconds</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_30000">0.5 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_60000">1 minute</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_90000">1.5 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_120000">2 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_150000">2.5 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_180000">3 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_240000">4 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_300000">5 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_360000">6 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_600000">10 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_720000">12 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_900000">15 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_1440000">24 minutes</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_1800000">0.5 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_3600000">1 hour</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_5400000">1.5 hour</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_7200000">2 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_9000000">2.5 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_10800000">3 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_14400000">4 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_18000000">5 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_21600000">6 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_36000000">10 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_43200000">12 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_54000000">15 hours</string>
-    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT = 30] -->
+    <!-- Text to indicate time lapse recording frame interval [CHAR LIMIT=30] -->
     <string name="pref_video_time_lapse_frame_interval_86400000">24 hours</string>
 
-    <!-- Seconds: a unit of time for time lapse intervals. [CHAR LIMIT = 20] -->
+    <!-- Seconds: a unit of time for time lapse intervals. [CHAR LIMIT=20] -->
     <string name="time_lapse_seconds">seconds</string>
-    <!-- Minutes: a unit of time for time lapse intervals. [CHAR LIMIT = 20] -->
+    <!-- Minutes: a unit of time for time lapse intervals. [CHAR LIMIT=20] -->
     <string name="time_lapse_minutes">minutes</string>
-    <!-- Hours: a unit of time for time lapse intervals. [CHAR LIMIT = 20] -->
+    <!-- Hours: a unit of time for time lapse intervals. [CHAR LIMIT=20] -->
     <string name="time_lapse_hours">hours</string>
 
-    <!-- The button to confirm time-lapse setting changes. [CHAR LIMIT = 20] -->
+    <!-- The button to confirm time-lapse setting changes. [CHAR LIMIT=20] -->
     <string name="time_lapse_interval_set">Done</string>
-    <!-- Title in time interval picker for setting time interval. [CHAR LIMIT = 30]-->
+    <!-- Title in time interval picker for setting time interval. [CHAR LIMIT=30]-->
     <string name="set_time_interval">Set Time Interval</string>
-    <!-- Help text that is shown when the time lapse feature is turned off. [CHAR LIMIT = 180]-->
+    <!-- Help text that is shown when the time lapse feature is turned off. [CHAR LIMIT=180]-->
     <string name="set_time_interval_help">Time lapse feature is off. Turn it on to set time interval.</string>
-    <!-- Title in timer setting for setting the duration for the countdown timer. [CHAR LIMIT = 50]-->
+    <!-- Title in timer setting for setting the duration for the countdown timer. [CHAR LIMIT=50]-->
     <string name="set_duration">Set duration in seconds</string>
-    <!-- On-screen hint during timer countdown for taking a photo. [CHAR LIMIT = 60]-->
+    <!-- On-screen hint during timer countdown for taking a photo. [CHAR LIMIT=60]-->
     <string name="count_down_title_text">Counting down to take a photo</string>
 
-    <!-- Title for first run dialog asking if the user wants to remember photo locations [CHAR LIMIT = 50] -->
+    <!-- Title for first run dialog asking if the user wants to remember photo locations [CHAR LIMIT=50] -->
     <string name="remember_location_title">Remember photo locations?</string>
-    <!-- Message for first run dialog asking if the user wants to remember photo locations [CHAR LIMIT = None] -->
+    <!-- Message for first run dialog asking if the user wants to remember photo locations [CHAR LIMIT=None] -->
     <string name="remember_location_prompt">Tag your photos and videos with the locations where they are taken.\n\nOther apps can access this information along with your saved images.</string>
-    <!-- Negative answer for first run dialog asking if the user wants to remember photo locations [CHAR LIMIT = 20] -->
+    <!-- Negative answer for first run dialog asking if the user wants to remember photo locations [CHAR LIMIT=20] -->
     <string name="remember_location_no">No thanks</string>
-    <!-- Positive answer for first run dialog asking if the user wants to remember photo locations [CHAR LIMIT = 20] -->
+    <!-- Positive answer for first run dialog asking if the user wants to remember photo locations [CHAR LIMIT=20] -->
     <string name="remember_location_yes">Yes</string>
 
     <!-- Camera menu labels -->
@@ -658,13 +658,13 @@ CHAR LIMIT = NONE] -->
     <string name="camera_menu_settings_label">SETTINGS</string>
 
     <!--  Tiny Planet -->
-    <!-- Button to press for creating a tiny planet image. [CHAR LIMIT = 30] -->
+    <!-- Button to press for creating a tiny planet image. [CHAR LIMIT=30] -->
     <string name="create_tiny_planet">Create Tiny Planet</string>
-    <!-- Message shown while a tiny planet image is being saved. [CHAR LIMIT = 30] -->
+    <!-- Message shown while a tiny planet image is being saved. [CHAR LIMIT=30] -->
     <string name="saving_tiny_planet">Saving Tiny Planet …</string>
-    <!-- Label above a slider that let's the user set the zoom of a tiny planet image. [CHAR LIMIT = 15] -->
+    <!-- Label above a slider that let's the user set the zoom of a tiny planet image. [CHAR LIMIT=15] -->
     <string name="tiny_planet_zoom">Zoom</string>
-    <!-- Label above a slider that let's the user set the rotation of a tiny planet image. [CHAR LIMIT = 15] -->
+    <!-- Label above a slider that let's the user set the rotation of a tiny planet image. [CHAR LIMIT=15] -->
     <string name="tiny_planet_rotate">Rotate</string>
 
     <!-- Crop -->
@@ -678,25 +678,31 @@ CHAR LIMIT = NONE] -->
     <!--  String shown when setting the homepage wallpaper in the background [CHAR LIMIT=NONE] -->
     <string name="setting_wallpaper">Setting wallpaper</string>
 
-    <!-- Camera settings. [CHAR LIMIT = 25] -->
+    <!-- Camera settings. [CHAR LIMIT=25] -->
     <string name="mode_settings">Settings</string>
-    <!-- Text shown in the camera mode options list for normal camera mode [CHAR LIMIT = 25] -->
+    <!-- Text shown in the camera mode options list for normal camera mode [CHAR LIMIT=25] -->
     <string name="mode_camera">Camera</string>
-    <!-- Text shown in the camera mode options list for putting camera in video recording mode [CHAR LIMIT = 25] -->
+    <!-- Text shown in the camera mode options list for putting camera in video recording mode [CHAR LIMIT=25] -->
     <string name="mode_video">Video</string>
-    <!-- Text shown in the camera mode options list for putting camera in photo sphere mode. 'Photo sphere' is a product name. [CHAR LIMIT = 25] -->
+    <!-- Text shown in the camera mode options list for putting camera in photo sphere mode. 'Photo sphere' is a product name. [CHAR LIMIT=25] -->
     <string name="mode_photosphere">Photo Sphere</string>
-    <!-- Text shown in the camera mode options list for advanced user mode. Craft mode contains more advanced features such as capability to refocus and recognize barcode/QR code. [CHAR LIMIT = 25] -->
+    <!-- Text shown in the camera mode options list for advanced user mode. Craft mode contains more advanced features such as capability to refocus and recognize barcode/QR code. [CHAR LIMIT=25] -->
     <string name="mode_craft">Craft</string>
-    <!-- Text shown in the camera mode options list for putting camera in time lapse mode [CHAR LIMIT = 25] -->
+    <!-- Text shown in the camera mode options list for putting camera in time lapse mode [CHAR LIMIT=25] -->
     <string name="mode_timelapse">Time Lapse</string>
-    <!-- Text shown in the camera mode options list for putting camera in wide angle panorama mode [CHAR LIMIT = 25] -->
+    <!-- Text shown in the camera mode options list for putting camera in wide angle panorama mode [CHAR LIMIT=25] -->
     <string name="mode_wideangle">Wide Angle</string>
 
+    <!-- Text shown in camera settings list for toggling photo location on or off [CHAR LIMIT=25] -->
     <string name="setting_location">Location</string>
+    <!-- Text shown in camera settings list for selecting the size of photos that will be captured [CHAR LIMIT=25] -->
     <string name="setting_picture_size">Picture Size</string>
+    <!-- Text shown in camera settings list for selecting the resolution of videos that will be recorded [CHAR LIMIT=25] -->
     <string name="setting_video_resolution">Video Resolution</string>
+    <!-- Text shown in camera settings list for selecting the camera mode that will be used by default when the app starts [CHAR LIMIT=25] -->
     <string name="setting_default_camera">Default Camera</string>
+    <!-- Text shown in camera settings list for sending user feedback to the app publisher [CHAR LIMIT=25] -->
+    <string name="setting_send_feedback">Send Feedback</string>
 
     <!-- TODO: don't ship with these -->
     <string name="focus_ring_toggle" translatable="false">FR</string>