I tried running the browser in Purify and the browser crashed on me.  Basically, the animations were really slow and at one point I got  a NULL pointer read because TabContents was NULL while updating  the animations. 
I tried running the browser in Purify and the browser crashed on me. 
Basically, the animations were really slow and at one point I got 
a NULL pointer read because TabContents was NULL while updating 
the animations. 

Call stack:

    TabContents::is_loading(void)const [browser\tab_contents.h:214]
    Browser::IsCurrentPageLoading(void)const [browser\browser.cc:400]
    BrowserView::LoadingAnimationCallback(void) [browser\views\frame\browser_view.cc:1325]
    ... removed Timer calls for clarity...

We already check the return value of GetSelectedTabContents in many 
places. It makes sense to do it here also, I think.

Review URL: http://codereview.chromium.org/13647

git-svn-id: dd90618784b6a4b323ea0c23a071cb1c9e6f2ac7@6587 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser,wistoch/meego-app-browser