Fix reference to potentially NULL TabContentsWrapper.

Fix reference to potentially NULL TabContentsWrapper.

BUG=105689
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112705

[Peter Kasting, 2011-12-01 21:03:37]

LGTM

http://codereview.chromium.org/8745016/diff/1/chrome/browser/autocomplete/aut...
File chrome/browser/autocomplete/autocomplete_edit.cc (right):

http://codereview.chromium.org/8745016/diff/1/chrome/browser/autocomplete/aut...
chrome/browser/autocomplete/autocomplete_edit.cc:1039: } else {
Nit: While here... no else after return.

http://codereview.chromium.org/8745016/diff/1/chrome/browser/autocomplete/aut...
chrome/browser/autocomplete/autocomplete_edit.cc:1051: return;
Nit: It would be nice to write a comment here explaining when this can be NULL. 
(Also for DoInstant() above.)

[dominich, 2011-12-02 00:30:41]

http://codereview.chromium.org/8745016/diff/1/chrome/browser/autocomplete/aut...
File chrome/browser/autocomplete/autocomplete_edit.cc (right):

http://codereview.chromium.org/8745016/diff/1/chrome/browser/autocomplete/aut...
chrome/browser/autocomplete/autocomplete_edit.cc:1039: } else {
On 2011/12/01 21:03:38, Peter Kasting wrote:
> Nit: While here... no else after return.

Done.

http://codereview.chromium.org/8745016/diff/1/chrome/browser/autocomplete/aut...
chrome/browser/autocomplete/autocomplete_edit.cc:1051: return;
On 2011/12/01 21:03:38, Peter Kasting wrote:
> Nit: It would be nice to write a comment here explaining when this can be
NULL. 
> (Also for DoInstant() above.)

Done.

[commit-bot: I haz the power, 2011-12-02 14:35:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dominich@chromium.org/8745016/5001

[commit-bot: I haz the power, 2011-12-02 15:46:50]

Change committed as 112705