Help: Implement the initial version of the cross-platform help/about page.

chrome/browser/autocomplete/autocomplete_edit.cc

Index: chrome/browser/autocomplete/autocomplete_edit.cc
diff --git a/chrome/browser/autocomplete/autocomplete_edit.cc b/chrome/browser/autocomplete/autocomplete_edit.cc
index 04e3c7b359c89ad841798e6b2a90c9aaa8c105e3..5e905335b93f0debe74ed126b1ba74dcd48b6af0 100644
--- a/chrome/browser/autocomplete/autocomplete_edit.cc
+++ b/chrome/browser/autocomplete/autocomplete_edit.cc
@@ -527,11 +527,13 @@ void AutocompleteEditModel::OpenMatch(const AutocompleteMatch& match,
     else if (!has_temporary_text_)
       log.inline_autocompleted_length = inline_autocomplete_text_.length();
     if (disposition == CURRENT_TAB) {
-      // If we know the destination is being opened in the curren tab,
+      // If we know the destination is being opened in the current tab,
       // we can easily get the tab ID.  (If it's being opened in a new
       // tab, we don't know the tab ID yet.)
-      log.tab_id = controller_->GetTabContentsWrapper()->
-                       restore_tab_helper()->session_id().id();
+      if (controller_->GetTabContentsWrapper()->restore_tab_helper()) {
+        log.tab_id = controller_->GetTabContentsWrapper()->
+            restore_tab_helper()->session_id().id();
+      }

[csilv, 2012/02/03 19:43:52]

Did you intend to include this change as part of this CL?  Looks fine, just
curious.

[James Hawkins, 2012/02/03 22:48:40]

Sorta, not really.  I need to file a bug for it.  I'll take it out of this CL.

     }
     if (user_input_in_progress_) {
       // This case should happen every time except possibly in unit tests.