Set lockscreen settings in the locksettings db.

build/android/pylib/device_settings.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import logging
 
 from pylib import content_settings
+from pylib.device import device_errors
+
+_LOCK_SCREEN_SETTINGS_PATH = '/data/system/locksettings.db'
+PASSWORD_QUALITY_UNSPECIFIED = 0
 
 
 def ConfigureContentSettingsDict(device, desired_settings):
   """Configures device content setings from a dictionary.
 
   Many settings are documented at:
     http://developer.android.com/reference/android/provider/Settings.Global.html
     http://developer.android.com/reference/android/provider/Settings.Secure.html
     http://developer.android.com/reference/android/provider/Settings.System.html
 
@@ skipping 23 matching lines @@
   if device.old_interface.GetBuildType() == 'userdebug':
     for table, key_value in sorted(desired_settings.iteritems()):
       settings = content_settings.ContentSettings(table, device)
       for key, value in key_value.iteritems():
         settings[key] = value
       logging.info('\n%s %s', table, (80 - len(table)) * '-')
       for key, value in sorted(settings.iteritems()):
         logging.info('\t%s: %s', key, value)
 
 
+def SetLockScreenSettings(device):
+  """Sets lock screen settings on the device.
+
+  On certain device/Android configurations we need to disable the lock screen in
+  a different database. Additionally, the password type must be set to
+  DevicePolicyManager.PASSWORD_QUALITY_UNSPECIFIED.
+  Lock screen settings are stored in sqlite on the device in:
+      /data/system/locksettings.db
+
+  IMPORTANT: The first column is used as a primary key so that all rows with the
+  same value for that column are removed from the table prior to inserting the
+  new values.
+
+  Args:
+    device: A DeviceUtils instance for the device to configure.
+
+  Raises:
+    Exception if the setting was not properly set.
+  """
+  if (not device.old_interface.FileExistsOnDevice(_LOCK_SCREEN_SETTINGS_PATH) or
+      device.old_interface.GetBuildType() != 'userdebug'):
+    return
+
+  db = _LOCK_SCREEN_SETTINGS_PATH
+  locksettings = [('locksettings', 'lockscreen.disabled', '1'),
+                  ('locksettings', 'lockscreen.password_type',
+                   str(PASSWORD_QUALITY_UNSPECIFIED)),
+                  ('locksettings', 'lockscreen.password_type_alternate',
+                   str(PASSWORD_QUALITY_UNSPECIFIED))]
+  for locksetting in locksettings:

[jbudorick, 2014/07/01 15:10:55]

nit: These four lines could be collapsed into:

  for table, key, value in locksettings:

[navabi, 2014/07/01 22:44:43]

Done.

+    table = locksetting[0]
+    key = locksetting[1]
+    value = locksetting[2]
+    # Set the lockscreen setting for default user '0'
+    columns = ['name', 'user', 'value']
+    values = [key, '0', value]
+    print 'db: %s' % db

[jbudorick, 2014/07/01 15:10:55]

What's up with the prints? Are these left over from debugging?

[navabi, 2014/07/01 22:44:43]

Done. Yes. They are. Good catch.

+    print 'table: %s' % table
+    print 'columns: %s' % columns
+    print 'values: %s' % values
+    assert len(columns), 'must have at least one column'

[jbudorick, 2014/07/01 15:10:55]

These asserts seems somewhat excessive given that everything checked is
hard-coded in this function.

[navabi, 2014/07/01 22:44:43]

Removed the asserts. These were copied over from another provision script where
this was a separate function. You are right that we don't need them here.

+    assert len(columns) == len(values), 'need same number of columns and values'
+    assert '"' not in table, 'table cannot contain double quotes'
+    assert '"' not in ''.join(columns), 'columns cannot contain double quotes'
+    assert '"' not in ''.join(values), 'values cannot contain double quotes'
+    def escape(s):

[jbudorick, 2014/07/01 15:10:55]

What's up with this? None of the items in |columns| or |values| contain single
quotes. Are you anticipating them changing in the future?

[navabi, 2014/07/01 22:44:43]

Removed this as well. See previous comment.

+      return s.replace("'", "''")
+
+    cmd = """begin transaction;
+delete from '%(table)s' where %(primary_key)s='%(primary_value)s';
+insert into '%(table)s' (%(columns)s) values (%(values)s);
+commit transaction;""" % {
+      'table': table,
+      'primary_key': escape(columns[0]),
+      'primary_value': escape(values[0]),
+      'columns': ', '.join([escape(column) for column in columns]),
+      'values': ', '.join(["'%s'" % escape(value) for value in values])
+    }
+    try:
+      output_msg = device.RunShellCommand('\'sqlite3 %s "%s"\'' % (db, cmd))

[jbudorick, 2014/07/01 15:10:55]

RunShellCommand should quote the command for you, so I don't think you need the
single quotes here.

[navabi, 2014/07/01 22:44:43]

I don't think it adds the single quotes correctly. I added the single quotes
after it did not work without them. Here is what the error is (i.e. output_msg)
without the single quotes:

['Error: no such column: lockscreen.password_type_alternate']

I then looked at the RunShellCommand in
build_internal/scripts/slave/android/device_utils.py and noticed that
implementation put a single quote around the command.

[jbudorick, 2014/07/02 01:01:29]

In that case, this is fine with me. I'll have to look at it when I switch the
implementation of RunShellCommand away from the AndroidCommands version.

+    except device_errors.CommandFailedError:
+      raise Exception('Failed to set system setting.\n%s' % ''.join(output_msg))

[jbudorick, 2014/07/01 15:10:55]

If RunShellCommand raises an error, output_msg isn't going to be set...

[navabi, 2014/07/01 22:44:43]

Done.

+
+
 ENABLE_LOCATION_SETTING = {
   'settings/secure': {
     # Ensure Geolocation is enabled and allowed for tests.
     'location_providers_allowed': 'gps,network',
   }
 }
 
 DISABLE_LOCATION_SETTING = {
   'settings/secure': {
     # Ensure Geolocation is disabled.
@@ skipping 56 matching lines @@
   },
 }
 
 
 NETWORK_DISABLED_SETTINGS = {
   'settings/global': {
     'airplane_mode_on': 1,
     'wifi_on': 0,
   },
 }