Fixes regression from changes to restore tabs

Fixes regression from changes to restore tabs

The code was assuming that if IDC_RESTORE_TAB returned true that the
TabRestoreService had non-empty entries. This is no longer a valid
assumpion.

BUG=404407
TEST=see bug
R=pkotwicz@chromium.org

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

[sky, 2014-08-19 19:36:30]

Clearly I need a test, but I'm going to do that separately as this is a top
crasher.

[pkotwicz, 2014-08-19 21:12:14]

LGTM

https://codereview.chromium.org/488643005/diff/1/chrome/browser/ui/views/fram...
File chrome/browser/ui/views/frame/system_menu_model_delegate.cc (right):

https://codereview.chromium.org/488643005/diff/1/chrome/browser/ui/views/fram...
chrome/browser/ui/views/frame/system_menu_model_delegate.cc:48: // the menu.
Nit: chrome::IsCommandEnabled() returns true for IDC_RESTORE_TAB while
TabRestoreService loads.

[sky, 2014-08-19 21:35:08]

https://codereview.chromium.org/488643005/diff/1/chrome/browser/ui/views/fram...
File chrome/browser/ui/views/frame/system_menu_model_delegate.cc (right):

https://codereview.chromium.org/488643005/diff/1/chrome/browser/ui/views/fram...
chrome/browser/ui/views/frame/system_menu_model_delegate.cc:48: // the menu.
On 2014/08/19 21:12:14, pkotwicz wrote:
> Nit: chrome::IsCommandEnabled() returns true for IDC_RESTORE_TAB while
> TabRestoreService loads.

Done.

[sky, 2014-08-19 21:35:13]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-08-19 21:36:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sky@chromium.org/488643005/20001

[commit-bot: I haz the power, 2014-08-19 22:39:49]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[commit-bot: I haz the power, 2014-08-19 23:50:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-19 23:50:37]

Try jobs failed on following builders:
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[sky, 2014-08-20 01:09:27]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-08-20 01:10:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sky@chromium.org/488643005/40001

[commit-bot: I haz the power, 2014-08-20 02:34:17]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-20 03:33:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-20 03:33:23]

Try jobs failed on following builders:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)

[sky, 2014-08-20 04:46:44]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-08-20 04:47:11]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sky@chromium.org/488643005/40001

[commit-bot: I haz the power, 2014-08-20 05:56:26]

Committed patchset #3 (40001) as 290772