Porting changes in chrome/{browser,common}, also enabling more unit tests....

chrome/browser/session_startup_pref.cc

Index: chrome/browser/session_startup_pref.cc
===================================================================
--- chrome/browser/session_startup_pref.cc	(revision 2402)
+++ chrome/browser/session_startup_pref.cc	(working copy)
@@ -37,6 +37,9 @@
     case URLS:
       type = 4;
       break;
+
+    default:

[M-A Ruel, 2008/09/19 12:51:18]

add 'NOTREACHED();' here.

[Dean McNamee, 2008/09/19 12:58:10]

That's a lie, it is reached.  It's the case of DEFAULT.

On 2008/09/19 12:51:18, M-A wrote:

[M-A Ruel, 2008/09/19 12:59:49]

Ok, don't listen to me.

On 2008/09/19 12:58:10, deanm wrote:

+      break;
   }
   prefs->SetInteger(prefs::kRestoreOnStartup, type);
 
@@ -86,7 +89,7 @@
     if (url_pref_list->Get(i, &value)) {
       std::wstring url_text;
       if (value->GetAsString(&url_text))
-        pref.urls.push_back(GURL(url_text));
+        pref.urls.push_back(GURL(WideToUTF8(url_text)));
     }
   }