0
Commit Graph

9 Commits

Author SHA1 Message Date
akalin@chromium.org
caaf3ccabb Relax check for message loop in SequencedWorkerPool
Assert that the constructor message loop is non-NULL only when
the SequencedWorkerPool is about to be destroyed.

Revert now-unneccessary test changes that adds MessagePools for
SequenedWorkerPool.

BUG=117940
TEST=


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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127335 0039d316-1c4b-4281-b951-d872f2087c98
2012-03-17 03:27:53 +00:00
akalin@chromium.org
71b18fbce8 Ensure that SequencedWorkerPools in tests don't outlive their tests
This prevents strange races with other tests.

Add an OnDestroy() method to SequencedWorkerPool::TestingObserver().

Fix a bug where one test wasn't calling Shutdown() on its
WorkerPools.

Fix a deadlock if a Worker object releases the last ref to the WorkerPool.

BUG=115987
TEST=


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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125152 0039d316-1c4b-4281-b951-d872f2087c98
2012-03-06 14:17:34 +00:00
husky@chromium.org
b77226e27c Use SequencedWorkerPool for disk operations in TraceSubscriberStdio.
This class was hitting a ThreadRestrictions assert because it called
OpenFile on the UI thread. To reduce unnecessary copying, I've changed
the OnTraceDataCollected argument from std::string to RefCountedString.

Second attempt (originally committed in 123140, reverted in 123146).
Added DISALLOW_COPY_AND_ASSIGN to fix Windows compilation error.

BUG=None
TEST=content_unittests


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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123433 0039d316-1c4b-4281-b951-d872f2087c98
2012-02-24 04:17:06 +00:00
groby@chromium.org
e28403a126 Revert 123140 - Use SequencedWorkerPool for disk operations in TraceSubscriberStdio.
Reverted since it causes compile failure on Win/Aura. 
(scoped_refptr of TraceSubscriberStdioImpl which is only forward-declared)

This class was hitting a ThreadRestrictions assert because it called
OpenFile on the UI thread. To reduce unnecessary copying, I've changed
the OnTraceDataCollected argument from std::string to RefCountedString.

BUG=None
TEST=content_unittests

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

TBR=husky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/9447003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123146 0039d316-1c4b-4281-b951-d872f2087c98
2012-02-23 00:25:18 +00:00
husky@chromium.org
ecc6a5b5f2 Use SequencedWorkerPool for disk operations in TraceSubscriberStdio.
This class was hitting a ThreadRestrictions assert because it called
OpenFile on the UI thread. To reduce unnecessary copying, I've changed
the OnTraceDataCollected argument from std::string to RefCountedString.

BUG=None
TEST=content_unittests

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123140 0039d316-1c4b-4281-b951-d872f2087c98
2012-02-22 23:53:00 +00:00
jbates@chromium.org
91ea6343d7 Internalize JSON chunk merging to trace_event.h API (retry).
Original review: http://codereview.chromium.org/8355024/

BUG=100291
TEST=base_unittests, content_unittests

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106924 0039d316-1c4b-4281-b951-d872f2087c98
2011-10-24 16:34:08 +00:00
mpcomplete@chromium.org
a4ecc3d225 Revert: Internalize JSON chunk merging to trace_event.h API.
BUG=100291
TEST=base_unittests, content_unittests

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

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106812 0039d316-1c4b-4281-b951-d872f2087c98
2011-10-22 00:15:14 +00:00
jbates@chromium.org
027fc03bee Internalize JSON chunk merging to trace_event.h API.
BUG=100291
TEST=base_unittests, content_unittests

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106808 0039d316-1c4b-4281-b951-d872f2087c98
2011-10-21 23:31:51 +00:00
husky@google.com
58a4d14583 TraceSubscriber implementation that writes to a file.
Review URL: http://codereview.chromium.org/7044010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87459 0039d316-1c4b-4281-b951-d872f2087c98
2011-06-01 12:13:26 +00:00