My favorites ▼ | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 42301: TaskManagerBrowserTest.PopulateWebCacheFields is crashy
2 people starred this issue and may be notified of changes. Back to list
 
Project Member Reported by phajdan.jr@chromium.org, Apr 22, 2010 
Xlib:  extension "RANDR" missing on display ":9.0".
Note: Google Test filter = TaskManagerBrowserTest.PopulateWebCacheFields
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from TaskManagerBrowserTest
[ RUN      ] TaskManagerBrowserTest.PopulateWebCacheFields
[15698:15698:0422/044117:4439885896:WARNING:chrome/installer/util/master_preferences.cc(163)] Master preferences file not 
found: /b/slave/linux/build/src/out/Debug/master_preferences
[15698:15707:0422/044117:4439887243:INFO:net/base/host_resolver_impl.cc(1108)] IPv6Probe forced AddressFamily setting to 
ADDRESS_FAMILY_IPV4
[15698:15698:0422/044117:4439889949:INFO:chrome/browser/extensions/extensions_service.cc(608)] Sending 
EXTENSION_LOADED
[15698:15707:0422/044117:4439892872:INFO:chrome/browser/net/dns_global.cc(430)] DNS Prefetch service started
[15698:15698:0422/044117:4439955990:INFO:chrome/browser/history/history.cc(723)] History backend finished loading
[15698:15704:0422/044117:4439958721:INFO:chrome/browser/important_file_writer.cc(71)] successfully saved 
/tmp/org.chromium.EHEKLs/test_user_data/Default/Bookmarks
[15698:15714:0422/044118:4440200260:INFO:chrome/browser/history/history_backend.cc(340)] Adding page 
file:///b/slave/linux/build/src/chrome/test/data/title1.html
[15698:15698:0422/044118:4440205006:FATAL:base/string_util.cc(879)] Check failed: false. Negative bytes value
Backtrace:
	StackTrace::StackTrace() [0xead282]
	logging::LogMessage::~LogMessage() [0xec10e7]
	FormatBytesInternal() [0xee6e94]
	FormatBytes() [0xee70bc]
	(anonymous namespace)::FormatStatsSize() [0x952369]
	TaskManagerModel::GetResourceWebCoreImageCacheSize() [0x95354b]
	TaskManagerBrowserTest_PopulateWebCacheFields_Test::RunTestOnMainThread() [0x4ac783]
	InProcessBrowserTest::RunTestOnMainThreadLoop() [0x401b5d]
	DispatchToMethod<>() [0x40a551]
	RunnableMethod<>::Run() [0x40a33e]
	BrowserMain() [0x5221c5]
	InProcessBrowserTest::SetUp() [0x40162a]
	testing::Test::Run() [0xfdb16b]
	testing::internal::TestInfoImpl::Run() [0xfdb73b]
	testing::TestCase::Run() [0xfdbd1c]
	testing::internal::UnitTestImpl::RunAllTests() [0xfe0303]
	testing::UnitTest::Run() [0xfdefe4]
	TestSuite::Run() [0x40b701]
	main [0x40b013]
	0x2ad8deb3fabd
	0x4002e9

./base/test/test_suite.h:162: Failure
Failed
[15698:15698:0422/044118:4440205006:FATAL:base/string_util.cc(879)] Check failed: false. Negative bytes value
Backtrace:
	StackTrace::StackTrace() [0xead282]
	logging::LogMessage::~LogMessage() [0xec10e7]
	FormatBytesInternal() [0xee6e94]
	FormatBytes() [0xee70bc]
	(anonymous namespace)::FormatStatsSize() [0x952369]
	TaskManagerModel::GetResourceWebCoreImageCacheSize() [0x95354b]
	TaskManagerBrowserTest_PopulateWebCacheFields_Test::RunTestOnMainThread() [0x4ac783]
	InProcessBrowserTest::RunTestOnMainThreadLoop() [0x401b5d]
	DispatchToMethod<>() [0x40a551]
	RunnableMethod<>::Run() [0x40a33e]
	BrowserMain() [0x5221c5]
	InProcessBrowserTest::SetUp() [0x40162a]
	testing::Test::Run() [0xfdb16b]
	testing::internal::TestInfoImpl::Run() [0xfdb73b]
	testing::TestCase::Run() [0xfdbd1c]
	testing::internal::UnitTestImpl::RunAllTests() [0xfe0303]
	testing::UnitTest::Run() [0xfdefe4]
	TestSuite::Run() [0x40b701]
	main [0x40b013]
	0x2ad8deb3fabd
	0x4002e9



[ FATAL ] testing/gtest/include/gtest/internal/gtest-port.h:1029:: pthread_mutex_destroy(&mutex_)failed with error 16
[0422/044118:ERROR:chrome/test/test_launcher/out_of_proc_test_runner.cc(64)] Test timeout exceeded!
[0422/044118:ERROR:base/process_util_posix.cc(171)] Unable to terminate process 15698: No such process

Setting priority to Pri-1 because the test is going to be disabled.
Apr 22, 2010
#1 bugdroid1@gmail.com 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=45308 

------------------------------------------------------------------------
r45308 | phajdan.jr@chromium.org | 2010-04-22 05:59:25 -0700 (Thu, 22 Apr 2010) | 8 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager_browsertest.cc?r1=45308&r2=45307

Disable crashy TaskManagerBrowserTest.PopulateWebCacheFields

TBR=jamesr

TEST=still crashy
BUG=42301

Review URL: http://codereview.chromium.org/1722005
------------------------------------------------------------------------
Jul 1, 2010
#2 erik...@chromium.org 
(No comment was entered for this change.)
Labels: not-extensions
Jul 9, 2010
#3 lafo...@chromium.org 
If we see this in the wild, we can bump it up to a P1 and the current mstone.
Status: Assigned
Owner: rvar...@chromium.org
Cc: jam...@chromium.org
Labels: -Pri-1 -Area-Undefined Pri-2 Area-Internals Mstone-7
Jul 9, 2010
#4 rvargas@chromium.org 
Note that every single test on that file (TaskManagerBrowserTest) is disabled at least on Windows because it crashes.
Owner: phajdan...@chromium.org
Jul 30, 2010
#5 mal@google.com 
(No comment was entered for this change.)
Labels: -FlakyTest Tests-Flaky
Aug 2, 2010
#6 phajdan.jr@chromium.org 
(No comment was entered for this change.)
Labels: -Tests-Flaky Tests-Disabled
Aug 4, 2010
#7 phajdan.jr@chromium.org 
The values we receive on the browser side are suspiciously big:

[19664:19664:0804/172643:201240508892:ERROR:chrome/browser/task_manager.cc(61)] stat.size = 0
[19664:19664:0804/172643:201240508914:ERROR:chrome/browser/task_manager.cc(62)] stat.liveSize = 7309956029606479468
[19664:19664:0804/172643:201240508960:ERROR:chrome/browser/task_manager.cc(61)] stat.size = 0
[19664:19664:0804/172643:201240508973:ERROR:chrome/browser/task_manager.cc(62)] stat.liveSize = 3275914009855805300
[19664:19664:0804/172643:201240509007:ERROR:chrome/browser/task_manager.cc(61)] stat.size = 0
[19664:19664:0804/172643:201240509026:ERROR:chrome/browser/task_manager.cc(62)] stat.liveSize = 7959113080249478772

I suspect some signed/unsigned confusion issues, or something like that. At some point we start treating the values as ints on the browser side, which may make them negative.

Any suggestions?
Status: Started
Aug 9, 2010
#8 bugdroid1@gmail.com 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=55449 

------------------------------------------------------------------------
r55449 | phajdan.jr@chromium.org | 2010-08-09 13:03:23 -0700 (Mon, 09 Aug 2010) | 11 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager.cc?r1=55449&r2=55448
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager.h?r1=55449&r2=55448
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager_browsertest.cc?r1=55449&r2=55448
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager_resource_providers.cc?r1=55449&r2=55448
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager_resource_providers.h?r1=55449&r2=55448
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/notification_type.h?r1=55449&r2=55448

Fix some problems with TaskManagerBrowserTest.PopulateWebCacheFields:

- wait for an actual update to occur; otherwise we're not testing the real thing
- change DCHECKs to EXPECT_EQs; that's what we should use in tests

This change should also fix the crashiness of this test.

TEST=browser_tests
BUG=42301

Review URL: http://codereview.chromium.org/2856091
------------------------------------------------------------------------
Aug 10, 2010
#9 bugdroid1@gmail.com 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=55530 

------------------------------------------------------------------------
r55530 | viettrungluu@chromium.org | 2010-08-09 22:23:33 -0700 (Mon, 09 Aug 2010) | 14 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager.cc?r1=55530&r2=55529
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager.h?r1=55530&r2=55529
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager_browsertest.cc?r1=55530&r2=55529
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager_resource_providers.cc?r1=55530&r2=55529
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/task_manager_resource_providers.h?r1=55530&r2=55529
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/notification_type.h?r1=55530&r2=55529

Revert 55449 - Fix some problems with TaskManagerBrowserTest.PopulateWebCacheFields:

- wait for an actual update to occur; otherwise we're not testing the real thing
- change DCHECKs to EXPECT_EQs; that's what we should use in tests

This change should also fix the crashiness of this test.

TEST=browser_tests
BUG=42301

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

TBR=phajdan.jr@chromium.org,glen@chromium.org
Review URL: http://codereview.chromium.org/3118003
------------------------------------------------------------------------
Aug 10, 2010
#10 phajdan.jr@chromium.org 
jamesr, my fix has failed or was incomplete. Could you take a look? The test itself seems to be fine, looks like the production code has problems.
Status: Assigned
Owner: jam...@chromium.org
Cc: -jam...@chromium.org
Aug 10, 2010
#11 jamesr@chromium.org 
I don't have bandwidth to take this as an m7 task.
Owner: ---
Cc: phajdan...@chromium.org
Aug 10, 2010
#12 phajdan.jr@chromium.org 
(No comment was entered for this change.)
Owner: phajdan...@chromium.org
Cc: -phajdan...@chromium.org jam...@chromium.org
Aug 16, 2010
#13 phajdan.jr@chromium.org 
I got Anthony's approval to mark it Mstone-8. I have tried several times to fix this test, but failed. James Robinson knows much more about WebKit and was actually working on this stats code.
Owner: jam...@chromium.org
Cc: -jam...@chromium.org
Labels: -Mstone-7 Mstone-8
Oct 19, 2010
#14 lafo...@chromium.org 
Since we are passed the branch, moving all mstone-8 issues to mstone-9 for triage/punting
Labels: -Mstone-8 Mstone-9
Oct 21, 2010
#15 lafo...@chromium.org 
Moving P2s with an owner to Mstone 10, bring back to M9 if you think it's important and you don't have higher priority work.
Labels: -Mstone-9 Mstone-10
Dec 9, 2010
#16 kerz@chromium.org 
P2 bugs with an owner that are not marked as started are being automatically moved to mstone:11.
Labels: -Mstone-10 MovedFrom-10 Mstone-11
Mar 9, 2011
#17 kar...@google.com 
rolling non releaseblocker mstone 11 bugs to mstone 12.
Labels: -Mstone-11 MovedFrom-11 Mstone-12
Apr 6, 2011
#18 jamesr@chromium.org 
I'm not working on this now and do not anticipate having time to work on it in the near future.
Status: Available
Owner: ---
Cc: jamesr@chromium.org
Apr 25, 2011
#19 kerz@google.com 
Moving out of M12.
Labels: Mstone13 MovedFrom12
Apr 25, 2011
#20 kerz@google.com 
Moving out of M12.
Labels: -Mstone-12 Mstone-13
Jun 2, 2011
#21 lafo...@chromium.org 
Moving !type=meta|regression and !releaseblocker to next mstone
Labels: -Mstone-13 Mstone-14 MovedFrom13
Jun 2, 2011
#22 lafo...@chromium.org 
(No comment was entered for this change.)
Labels: -MovedFrom12 MovedFrom-12
Jul 27, 2011
#23 kerz@google.com 
Punting out non-critical bugs.  Please move back to 14 if you believe this was done in error.
Labels: -Mstone-14 Mstone-15 MovedFrom-14
Sep 8, 2011
#24 kar...@google.com 
moving non-essential bugs from 15 to 16. Please feel free to move back if this is an error and your bug is a blocker for 15.
Labels: Mstone-16 MovedFrom15 bulkmove
Oct 24, 2011
#25 lafo...@google.com 
(No comment was entered for this change.)
Labels: -Mstone-16 MovedFrom-16 Mstone-17
Dec 19, 2011
#26 kerz@google.com 
Removing milestone on all bugs marked available and targeted at m17.  Please re-triage to a new milestone if desired.
Labels: -Mstone-17 MovedFrom-17
Mar 10, 2013
#27 bugdroid1@chromium.org 
(No comment was entered for this change.)
Labels: -Area-Internals -Tests-Disabled Cr-Internals Cr-Tests-Disabled
► Sign in to add a comment

Powered by Google Project Hosting