Kill unnecessary HasSadTab() function.

Kill unnecessary HasSadTab() function.

BUG=125846
[email protected]


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=172913

[tfarina, 2012-12-12 23:54:26]

Hi Scott, could you review this to me?

Thanks,

[sky, 2012-12-13 00:52:38]

https://codereview.chromium.org/11519047/diff/2001/chrome/browser/ui/sad_tab_...
File chrome/browser/ui/sad_tab_helper.cc (right):

https://codereview.chromium.org/11519047/diff/2001/chrome/browser/ui/sad_tab_...
chrome/browser/ui/sad_tab_helper.cc:40: if (sad_tab_.get())
I believe you don't need the get() anymore here.

[tfarina, 2012-12-13 00:58:16]

https://codereview.chromium.org/11519047/diff/2001/chrome/browser/ui/sad_tab_...
File chrome/browser/ui/sad_tab_helper.cc (right):

https://codereview.chromium.org/11519047/diff/2001/chrome/browser/ui/sad_tab_...
chrome/browser/ui/sad_tab_helper.cc:40: if (sad_tab_.get())
On 2012/12/13 00:52:38, sky wrote:
> I believe you don't need the get() anymore here.
Do'h I thought I have checked that scoped_ptr_malloc didn't have this ability.
But it HAS :)

Done.

[sky, 2012-12-13 17:23:42]

LGTM

[commit-bot: I haz the power, 2012-12-13 17:27:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/11519047/8001

[commit-bot: I haz the power, 2012-12-13 19:15:33]

Change committed as 172913