Modify session restore test to set restore setting via UI and verify UI settings using web driver.

functional/prefs.py

 #!/usr/bin/python
 # Copyright (c) 2011 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
 import os
 import shutil
 import sys
 
 import pyauto_functional  # Must be imported before pyauto
 import pyauto
+import test_utils
 
 
 class PrefsTest(pyauto.PyUITest):
   """TestCase for Preferences."""
 
   def Debug(self):
     """Test method for experimentation.
 
     This method will not run automatically.
     """
     while True:
       raw_input('Interact with the browser and hit <enter> to dump prefs... ')
       self.pprint(self.GetPrefsInfo().Prefs())
 
   def testSessionRestore(self):
     """Test session restore preference."""
+
+    pref_url = 'chrome://settings/browser'
     url1 = 'http://www.google.com/'

[kkania, 2011/11/22 20:42:46]

there's no reason to use public URLs for this test; it would be great to change
them; up to you

[vivianz, 2011/11/22 22:29:43]

make sense, but will leave as is this time, since all other prefs test still use
these public urls. We should have one cl to fix them all. 

     url2 = 'http://news.google.com/'
+
+    self.NavigateToURL(pref_url)
+    # Set pref to restore session on startup.
+    driver = self.NewWebDriver()
+    restore = driver.find_element_by_xpath(

[kkania, 2011/11/22 20:42:46]

I think a good convention to start is appending '_elem' on to the end for all
WebElements. change to restore_elem

[vivianz, 2011/11/22 22:29:43]

Done.

+        '//input[@metric="Options_Startup_LastSession"]')
+    restore.click()
+    self.assertTrue(restore.is_selected())
+    self.RestartBrowser(clear_profile=False)
     self.NavigateToURL(url1)
     self.AppendTab(pyauto.GURL(url2))
     num_tabs = self.GetTabCount()
-    # Set pref to restore session on startup
-    self.SetPrefs(pyauto.kRestoreOnStartup, 1)
-    logging.debug('Setting %s to 1' % pyauto.kRestoreOnStartup)
     self.RestartBrowser(clear_profile=False)
-    # Verify
+    # Verify tabs are properly restored.
     self.assertEqual(self.GetPrefsInfo().Prefs(pyauto.kRestoreOnStartup), 1)
     self.assertEqual(num_tabs, self.GetTabCount())
     self.ActivateTab(0)
     self.assertEqual(url1, self.GetActiveTabURL().spec())
     self.ActivateTab(1)
     self.assertEqual(url2, self.GetActiveTabURL().spec())
+    # Verify session restore option is still selected.
+    self.NavigateToURL(pref_url, 0, 0)
+    driver = self.NewWebDriver()
+    self.assertTrue(driver.find_element_by_xpath(
+        '//input[@metric="Options_Startup_LastSession"]').is_selected())
 
   def testNavigationStateOnSessionRestore(self):
     """Verify navigation state is preserved on session restore."""
     urls = ('http://www.google.com/',
             'http://news.google.com/',
             'http://dev.chromium.org/',)
     for url in urls:
       self.NavigateToURL(url)
     tab = self.GetBrowserWindow(0).GetTab(0)
     tab.GoBack()
@@ skipping 134 matching lines @@
     self.assertEqual(title, self.GetActiveTabTitle())
 
   def testHaveLocalStatePrefs(self):
     """Verify that we have some Local State prefs."""
     self.assertTrue(self.GetLocalStatePrefsInfo())
 
 
 if __name__ == '__main__':
   pyauto_functional.Main()