Marking TopSitesTest.DeleteNotifications as DISABLED

Marking TopSitesTest.DeleteNotifications as DISABLED

Marks TopSitesTest.DeleteNotifications as DISABLED.  This test has been crashing unit_tests Mac 10.6.  E.g. http://build.chromium.org/buildbot/waterfall/builders/Mac10.6%20Tests/builds/5065/steps/unit_tests/logs/DeleteNotifications

BUG=49799
TEST=TopSitesTest.DeleteNotifications
TBR=nshkrob@chromium.org

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

[Paweł Hajdan Jr., 2010-07-21 17:36:19]

LGTM with a comment.

http://codereview.chromium.org/2832066/diff/1/2
File chrome/browser/history/top_sites_unittest.cc (right):

http://codereview.chromium.org/2832066/diff/1/2#newcode708
chrome/browser/history/top_sites_unittest.cc:708: // This test has been failing
to complete on Mac 10.6 sporadically.
nit: Just say it crashes entire unit_tests, it indicates the severity of the
problem more accurately.

[Nik Shkrob, 2010-07-21 17:38:20]

LGTM. 

I'll look into fixing this.

On 2010/07/21 17:36:19, Paweł Hajdan Jr. wrote:
> LGTM with a comment.
> 
> http://codereview.chromium.org/2832066/diff/1/2
> File chrome/browser/history/top_sites_unittest.cc (right):
> 
> http://codereview.chromium.org/2832066/diff/1/2#newcode708
> chrome/browser/history/top_sites_unittest.cc:708: // This test has been
failing
> to complete on Mac 10.6 sporadically.
> nit: Just say it crashes entire unit_tests, it indicates the severity of the
> problem more accurately.

[dhollowa, 2010-07-21 17:39:22]

http://codereview.chromium.org/2832066/diff/1/2
File chrome/browser/history/top_sites_unittest.cc (right):

http://codereview.chromium.org/2832066/diff/1/2#newcode708
chrome/browser/history/top_sites_unittest.cc:708: // This test has been failing
to complete on Mac 10.6 sporadically.
On 2010/07/21 17:36:19, Paweł Hajdan Jr. wrote:
> nit: Just say it crashes entire unit_tests, it indicates the severity of the
> problem more accurately.

Done.