New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 592900 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Apr 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

A bunch of tests are timing out on DrMemory

Project Member Reported by benwells@chromium.org, Mar 8 2016

Issue description

These seem to have been timing out for weeks.

CrExtensionsBrowserTestWithInstalledExtension.ExtensionServiceProfileSettingsTest
PDFExtensionTest.BasicPlugin
CrSettingsRtlTest.DrawerPanelFlips
HostedAppTest.ShouldShowLocationBarForHTTPBookmarkApp
HostedAppTest.ShouldShowLocationBarForHTTPSBookmarkApp
MimeHandlerViewTest.EmbeddedDataUrlObject
CrSettingsRtlTest.DrawerPanelFlips
PasswordsPrivateApiTest.RequestPlaintextPassword
BasicExtensionSettingsWebUITest.testNonEmptyExtensionList
SyncSetupWebUITestAsync.RestoreSyncDataTypes
CrExtensionsBrowserTest.ExtensionItemWarningsTest
CrashRecoveryBrowserTest.DoubleReloadWithError
MultilanguageOptionsWebUIBrowserTest.NotAcceptLanguage
PDFExtensionTest.ParamsParser
ZoomControllerBrowserTest.NavigationResetsManualMode
CrExtensionsBrowserTest.ExtensionItemDeveloperStateTest
InlineInstallPrivateApiTestApp.SuccessfulInstall
MaterialHistoryBrowserTest.HistoryItemTest
PushMessagingBrowserTest.SubscribeWithoutKeySuccessNotificationsGranted
CrExtensionsBrowserTestWithInstalledExtension.ExtensionServiceToggleIncognitoTest
SubframeTaskBrowserTest.TaskManagerShowsSubframeTasks
DurableStorageBrowserTest.Incognito
ImageWriterPrivateApiTest.TestWriteFromFile
CrExtensionsBrowserTest.ExtensionItemList
CrashRecoveryBrowserTest.ReloadCacheRevalidate
InstallGoodExtensionSettingsWebUITest.showOptions
CrExtensionsBrowserTestWithInstalledExtension.ExtensionServiceProfileSettingsTest
PDFExtensionTest.BasicPlugin
CrSettingsRtlTest.DrawerPanelFlips
HostedAppTest.ShouldShowLocationBarForHTTPBookmarkApp
HostedAppTest.ShouldShowLocationBarForHTTPSBookmarkApp
MimeHandlerViewTest.EmbeddedDataUrlObject
CrSettingsRtlTest.DrawerPanelFlips
PasswordsPrivateApiTest.RequestPlaintextPassword
BasicExtensionSettingsWebUITest.testNonEmptyExtensionList
SyncSetupWebUITestAsync.RestoreSyncDataTypes
CrExtensionsBrowserTest.ExtensionItemWarningsTest
CrashRecoveryBrowserTest.DoubleReloadWithError
MultilanguageOptionsWebUIBrowserTest.NotAcceptLanguage
PDFExtensionTest.ParamsParser
ZoomControllerBrowserTest.NavigationResetsManualMode
CrExtensionsBrowserTest.ExtensionItemDeveloperStateTest
InlineInstallPrivateApiTestApp.SuccessfulInstall
MaterialHistoryBrowserTest.HistoryItemTest
PushMessagingBrowserTest.SubscribeWithoutKeySuccessNotificationsGranted
CrExtensionsBrowserTestWithInstalledExtension.ExtensionServiceToggleIncognitoTest
SubframeTaskBrowserTest.TaskManagerShowsSubframeTasks
DurableStorageBrowserTest.Incognito
ImageWriterPrivateApiTest.TestWriteFromFile
CrExtensionsBrowserTest.ExtensionItemList
CrashRecoveryBrowserTest.ReloadCacheRevalidate
InstallGoodExtensionSettingsWebUITest.showOptions
PDFExtensionTest.TabTitleWithEmbeddedPdf
BrowserOptionsOverlayWebUITest.testNavigationInBackground
DownloadTestWithShelf.IncognitoDownload
InlineInstallPrivateApiTestApp.BackgroundInstall
MimeHandlerViewTest.Abort
BookmarkBubbleSignInDelegateTest.OnSignInLinkClickedReusesBlank
CrExtensionsBrowserTestWithInstalledExtension.ExtensionServiceUninstallTest
PDFExtensionTest.TabTitleWithTitle
AppViewTest.TestAppViewWithUndefinedDataShouldSucceed
WebUIWebViewBrowserTest.ExecuteScriptCode
 
Sorted unique version of that list:

AppViewTest.TestAppViewWithUndefinedDataShouldSucceed
BasicExtensionSettingsWebUITest.testNonEmptyExtensionList
BookmarkBubbleSignInDelegateTest.OnSignInLinkClickedReusesBlank
BrowserOptionsOverlayWebUITest.testNavigationInBackground
CrashRecoveryBrowserTest.DoubleReloadWithError
CrashRecoveryBrowserTest.ReloadCacheRevalidate
CrExtensionsBrowserTest.ExtensionItemDeveloperStateTest
CrExtensionsBrowserTest.ExtensionItemList
CrExtensionsBrowserTest.ExtensionItemWarningsTest
CrExtensionsBrowserTestWithInstalledExtension.ExtensionServiceProfileSettingsTest
CrExtensionsBrowserTestWithInstalledExtension.ExtensionServiceToggleIncognitoTest
CrExtensionsBrowserTestWithInstalledExtension.ExtensionServiceUninstallTest
CrSettingsRtlTest.DrawerPanelFlips
DownloadTestWithShelf.IncognitoDownload
DurableStorageBrowserTest.Incognito
HostedAppTest.ShouldShowLocationBarForHTTPBookmarkApp
HostedAppTest.ShouldShowLocationBarForHTTPSBookmarkApp
ImageWriterPrivateApiTest.TestWriteFromFile
InlineInstallPrivateApiTestApp.BackgroundInstall
InlineInstallPrivateApiTestApp.SuccessfulInstall
InstallGoodExtensionSettingsWebUITest.showOptions
MaterialHistoryBrowserTest.HistoryItemTest
MimeHandlerViewTest.Abort
MimeHandlerViewTest.EmbeddedDataUrlObject
MultilanguageOptionsWebUIBrowserTest.NotAcceptLanguage
PasswordsPrivateApiTest.RequestPlaintextPassword
PDFExtensionTest.BasicPlugin
PDFExtensionTest.ParamsParser
PDFExtensionTest.TabTitleWithEmbeddedPdf
PDFExtensionTest.TabTitleWithTitle
PushMessagingBrowserTest.SubscribeWithoutKeySuccessNotificationsGranted
SubframeTaskBrowserTest.TaskManagerShowsSubframeTasks
SyncSetupWebUITestAsync.RestoreSyncDataTypes
WebUIWebViewBrowserTest.ExecuteScriptCode
ZoomControllerBrowserTest.NavigationResetsManualMode

(so much better!)
Cc: thestig@chromium.org
I'm going to exclude all these tests, but it feels like a hack.

Adding the next sheriff who might have better ideas....
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f2a796fea172c30e8ca69d59f36120c53c8b2acd

commit f2a796fea172c30e8ca69d59f36120c53c8b2acd
Author: benwells <benwells@chromium.org>
Date: Tue Mar 08 07:57:51 2016

Exclude lots of tests from memory bots

- a Braille test which is flaky on ChromeOS MSAN
- loads of tests which are timing out on DrMemory.

TBR=thestig@chromium.org
BUG= 592900 ,  592893 

Review URL: https://codereview.chromium.org/1777463002

Cr-Commit-Position: refs/heads/master@{#379779}

[modify] https://crrev.com/f2a796fea172c30e8ca69d59f36120c53c8b2acd/chrome/browser/extensions/api/braille_display_private/braille_display_private_apitest.cc
[modify] https://crrev.com/f2a796fea172c30e8ca69d59f36120c53c8b2acd/tools/valgrind/gtest_exclude/browser_tests.gtest-drmemory.txt

Comment 5 by osh...@chromium.org, Mar 21 2016

Cc: tommycli@chromium.org bruening@chromium.org
FYI: +today's sheriff and DrMemory expert. 
Cc: zhaoqin@chromium.org
Be aware that the gtest filter is limited in size.  The latest exclusions may have pushed it beyond the limit as I'm seeing:

https://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Browser%20%28DrMemory%20full%29%20%281%29/builds/2877/steps/memory%20test%3A%20browser_tests/logs/stdio

[4472:2024:0318/151214:WARNING:chrome_browser_main_win.cc(454)] Command line too long for RegisterApplicationRestart

Project Member

Comment 7 by bugdroid1@chromium.org, Apr 5 2016

Project Member

Comment 9 by bugdroid1@chromium.org, May 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/334e252faede9a5b0f694b4ce9077c89f024d189

commit 334e252faede9a5b0f694b4ce9077c89f024d189
Author: jyasskin <jyasskin@chromium.org>
Date: Mon May 16 21:51:04 2016

Exclude more tests that time out under DrMemory

BUG= 592900 
TBR=csharrison@chromium.org

Review-Url: https://codereview.chromium.org/1978413002
Cr-Commit-Position: refs/heads/master@{#393944}

[modify] https://crrev.com/334e252faede9a5b0f694b4ce9077c89f024d189/tools/valgrind/gtest_exclude/browser_tests.gtest-drmemory.txt

Project Member

Comment 10 by bugdroid1@chromium.org, Jun 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/146ec8e2625a55eadd2ed79e4ef5d43adcf03d36

commit 146ec8e2625a55eadd2ed79e4ef5d43adcf03d36
Author: Reilly Grant <reillyg@chromium.org>
Date: Tue Jun 21 22:06:52 2016

Disable more tests that time out under Dr. Memory.

ChromeUIOverridesBrowserTest.BookmarkShortcutOverrides
TracingBrowserTest.TestBackgroundMemoryInfra

BUG= 581368 , 592900 
TBR=oshima@chromium.org

Review URL: https://codereview.chromium.org/2089843002 .

Cr-Commit-Position: refs/heads/master@{#401121}

[modify] https://crrev.com/146ec8e2625a55eadd2ed79e4ef5d43adcf03d36/tools/valgrind/gtest_exclude/browser_tests.gtest-drmemory.txt

Status: WontFix (was: Untriaged)
DrMemory is gone.

Sign in to add a comment