pkg/http: updated unittest dependency, removed unused imports

pkg/http: updated unittest dependency, removed unused imports

R=nweiz@google.com

Committed: https://code.google.com/p/dart/source/detail?r=37289

[kevmoo, 2014-06-12 20:25:34]

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/html/streamed_...
File pkg/http/test/html/streamed_request_test.dart (right):

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/html/streamed_...
pkg/http/test/html/streamed_request_test.dart:38: }
end with newline

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/io/streamed_re...
File pkg/http/test/io/streamed_request_test.dart (right):

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/io/streamed_re...
pkg/http/test/io/streamed_request_test.dart:58: }
end with newline

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/streamed_reque...
File pkg/http/test/streamed_request_test.dart (right):

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/streamed_reque...
pkg/http/test/streamed_request_test.dart:30: }
end w/ newline

[nweiz, 2014-06-12 20:41:18]

lgtm

https://codereview.chromium.org/334703004/diff/1/pkg/http/CHANGELOG.md
File pkg/http/CHANGELOG.md (right):

https://codereview.chromium.org/334703004/diff/1/pkg/http/CHANGELOG.md#newcode5
pkg/http/CHANGELOG.md:5: * Remove unused imports in tests.
This isn't a user-visible change. There's no need to include it in the
changelog.

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/html/streamed_...
File pkg/http/test/html/streamed_request_test.dart (right):

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/html/streamed_...
pkg/http/test/html/streamed_request_test.dart:38: }
On 2014/06/12 20:25:33, kevmoo wrote:
> end with newline

There's no need to annotate your own CL.

[kevmoo, 2014-06-12 20:56:47]

https://codereview.chromium.org/334703004/diff/1/pkg/http/CHANGELOG.md
File pkg/http/CHANGELOG.md (right):

https://codereview.chromium.org/334703004/diff/1/pkg/http/CHANGELOG.md#newcode5
pkg/http/CHANGELOG.md:5: * Remove unused imports in tests.
On 2014/06/12 20:41:18, nweiz wrote:
> This isn't a user-visible change. There's no need to include it in the
> changelog.

Done.

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/html/streamed_...
File pkg/http/test/html/streamed_request_test.dart (right):

https://codereview.chromium.org/334703004/diff/1/pkg/http/test/html/streamed_...
pkg/http/test/html/streamed_request_test.dart:38: }
On 2014/06/12 20:41:18, nweiz wrote:
> On 2014/06/12 20:25:33, kevmoo wrote:
> > end with newline
> 
> There's no need to annotate your own CL.

These deltas don't show up clearly in the CL tool. I've confused folks in the
past so figured I'd add some clarity.

[kevmoo, 2014-06-12 20:57:16]

Committed patchset #2 manually as r37289 (presubmit successful).