Move Handle in TeeTest and MockFetchDataLoaderClient to TestUtil

Move Handle in TeeTest and MockFetchDataLoaderClient to TestUtil

Preparation for using them in ResponseTest.cpp.

BUG=480746
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197811

[hiroshige, 2015-06-24 21:18:40]

hiroshige@chromium.org changed reviewers:
+ yhirano@chromium.org

[hiroshige, 2015-06-24 21:18:49]

PTAL.

[yhirano, 2015-06-25 04:31:03]

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
File Source/modules/fetch/DataConsumerHandleTestUtil.h (right):

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
Source/modules/fetch/DataConsumerHandleTestUtil.h:22: #include "wtf/Locker.h"
+Deque
+Vector
+ThreadingPrimitives
+ThreadSafeRefCounted

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
Source/modules/fetch/DataConsumerHandleTestUtil.h:282: void add(const Command&
command)
Please move function definitions to the cpp file.

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
File Source/modules/fetch/DataConsumerTeeTest.cpp (right):

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
Source/modules/fetch/DataConsumerTeeTest.cpp:23: #include "wtf/Vector.h"
-Deque
-Vector
-ThreadingPrimitives
-ThreadSafeRefCounted

[hiroshige, 2015-06-25 05:33:57]

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
File Source/modules/fetch/DataConsumerHandleTestUtil.h (right):

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
Source/modules/fetch/DataConsumerHandleTestUtil.h:22: #include "wtf/Locker.h"
On 2015/06/25 04:31:03, yhirano wrote:
> +Deque
> +Vector
> +ThreadingPrimitives
> +ThreadSafeRefCounted

Done.

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
Source/modules/fetch/DataConsumerHandleTestUtil.h:282: void add(const Command&
command)
On 2015/06/25 04:31:03, yhirano wrote:
> Please move function definitions to the cpp file.

Done.

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
File Source/modules/fetch/DataConsumerTeeTest.cpp (right):

https://codereview.chromium.org/1210693002/diff/1/Source/modules/fetch/DataCo...
Source/modules/fetch/DataConsumerTeeTest.cpp:23: #include "wtf/Vector.h"
On 2015/06/25 04:31:03, yhirano wrote:
> -Deque
> -Vector
> -ThreadingPrimitives
> -ThreadSafeRefCounted

Done.

[yhirano, 2015-06-25 05:40:07]

lgtm

[hiroshige, 2015-06-25 05:55:24]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2015-06-25 06:26:07]

The CQ bit was unchecked by hiroshige@chromium.org

[hiroshige, 2015-06-25 06:26:15]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2015-06-25 06:28:24]

The CQ bit was unchecked by hiroshige@chromium.org

[hiroshige, 2015-06-25 06:32:20]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2015-06-25 06:45:51]

The CQ bit was unchecked by hiroshige@chromium.org

[hiroshige, 2015-06-25 06:45:56]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2015-06-25 07:37:32]

The CQ bit was unchecked by hiroshige@chromium.org

[hiroshige, 2015-06-25 07:37:37]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2015-06-25 09:01:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1210693002/20001

[commit-bot: I haz the power, 2015-06-25 10:08:17]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=197811