Fix index out of bounds bug which causes memory leak or browser crash in extension sessions API.

chrome/browser/extensions/api/sessions/sessions_api.cc

Index: chrome/browser/extensions/api/sessions/sessions_api.cc
===================================================================
--- chrome/browser/extensions/api/sessions/sessions_api.cc	(revision 254429)
+++ chrome/browser/extensions/api/sessions/sessions_api.cc	(working copy)
@@ -228,8 +228,10 @@
     int tab_index,
     int selected_index) {
   std::string session_id = SessionId(session_tag, tab.tab_id.id()).ToString();
+  int index_corrected = std::min(tab.current_navigation_index,
+                                 static_cast<int>(tab.navigations.size() - 1));
   return CreateTabModelHelper(GetProfile(),
-                              tab.navigations[tab.current_navigation_index],
+                              tab.navigations[index_corrected],

[not at google - send to devlin, 2014/03/07 21:21:45]

it looks like you can use tab.normalized_navigation_index() here:

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ses...

                               session_id,
                               tab_index,
                               tab.pinned,