[LoaderTest] Add the gtest pretty printer for ResourceError

Add the gtest pretty printer for ResourceError

This will be used by https://codereview.chromium.org/1257343003/.

This CL ignores a presubmit error:
  "Streams are highly discouraged.  [readability/streams]"
  in PlatformTestPrinters.cpp.
Because:
  std::ostream is required for gtest pretty printing.
BUG=532364
Committed: https://crrev.com/f09dff4405b70b9b27fba049dd7cf59c2e585515
Cr-Commit-Position: refs/heads/master@{#381034}

[hiroshige, 2016-02-23 02:12:47]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-23 02:13:48]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/1

[hiroshige, 2016-02-23 02:20:22]

Description was changed from

==========
Add the gtest pretty printer for ResourceError

BUG=
==========

to

==========
Add the gtest pretty printer for ResourceError

This will be used by https://codereview.chromium.org/1257343003/.

This CL ignores a presubmit error:
  "Streams are highly discouraged.  [readability/streams]"
  in PlatformTestPrinters.cpp.
Because:
  std::ostream is required for gtest pretty printing.
BUG=532364
==========

[hiroshige, 2016-02-23 02:23:40]

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

[commit-bot: I haz the power, 2016-02-23 03:35:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-23 03:35:58]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-02-25 19:14:57]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-25 19:16:58]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/20001

[commit-bot: I haz the power, 2016-02-25 21:20:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-25 21:20:11]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[hiroshige, 2016-02-26 23:25:59]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-26 23:29:51]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/20001

[hiroshige, 2016-02-29 16:31:52]

PTAL.

[hiroshige, 2016-02-29 16:34:31]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-29 16:34:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/40001

[commit-bot: I haz the power, 2016-02-29 17:49:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-29 17:49:19]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-03-04 01:55:00]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-04 01:55:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/60001

[commit-bot: I haz the power, 2016-03-04 05:14:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-04 05:14:48]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[tyoshino (SeeGerritForStatus), 2016-03-09 05:36:21]

lgtm

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp
(right):

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp:1: //
Copyright 2015 The Chromium Authors. All rights reserved.
2016

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp:5: #include
"config.h"
remove

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp:23: << ",
localizedDescription = " << error.localizedDescription()
isNull() is not included intentionally?

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp:31: }
// namespace blink

[hiroshige, 2016-03-11 19:01:50]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-11 19:02:36]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/80001

[hiroshige, 2016-03-11 20:36:14]

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp
(right):

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp:1: //
Copyright 2015 The Chromium Authors. All rights reserved.
On 2016/03/09 05:36:20, tyoshino wrote:
> 2016

Done.

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp:5: #include
"config.h"
On 2016/03/09 05:36:21, tyoshino wrote:
> remove

Done.

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp:23: << ",
localizedDescription = " << error.localizedDescription()
On 2016/03/09 05:36:21, tyoshino wrote:
> isNull() is not included intentionally?

Added.

https://codereview.chromium.org/1722893002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/testing/PlatformTestPrinters.cpp:31: }
On 2016/03/09 05:36:21, tyoshino wrote:
> // namespace blink

Done.

[hiroshige, 2016-03-11 20:36:37]

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

[hiroshige, 2016-03-11 20:36:37]

mkwst@, could you take a look as a platform/ OWNER?

[commit-bot: I haz the power, 2016-03-11 22:57:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-11 22:57:49]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[hiroshige, 2016-03-11 23:11:29]

The CQ bit was checked by hiroshige@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-11 23:12:06]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/100001

[commit-bot: I haz the power, 2016-03-12 04:39:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-12 04:39:07]

Dry run: This issue passed the CQ dry run.

[Mike West, 2016-03-14 10:37:52]

LGTM.

[hiroshige, 2016-03-14 15:08:15]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2016-03-14 15:08:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
tyoshino@chromium.org
Link to the patchset: https://codereview.chromium.org/1722893002/#ps100001
(title: "Rebase")

[commit-bot: I haz the power, 2016-03-14 15:08:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/100001

[commit-bot: I haz the power, 2016-03-14 18:07:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-14 18:07:25]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[hiroshige, 2016-03-14 18:08:02]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2016-03-14 18:08:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1722893002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1722893002/100001

[commit-bot: I haz the power, 2016-03-14 19:10:31]

Description was changed from

==========
Add the gtest pretty printer for ResourceError

This will be used by https://codereview.chromium.org/1257343003/.

This CL ignores a presubmit error:
  "Streams are highly discouraged.  [readability/streams]"
  in PlatformTestPrinters.cpp.
Because:
  std::ostream is required for gtest pretty printing.
BUG=532364
==========

to

==========
Add the gtest pretty printer for ResourceError

This will be used by https://codereview.chromium.org/1257343003/.

This CL ignores a presubmit error:
  "Streams are highly discouraged.  [readability/streams]"
  in PlatformTestPrinters.cpp.
Because:
  std::ostream is required for gtest pretty printing.
BUG=532364
==========

[commit-bot: I haz the power, 2016-03-14 19:10:32]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-03-14 19:12:04]

Description was changed from

==========
Add the gtest pretty printer for ResourceError

This will be used by https://codereview.chromium.org/1257343003/.

This CL ignores a presubmit error:
  "Streams are highly discouraged.  [readability/streams]"
  in PlatformTestPrinters.cpp.
Because:
  std::ostream is required for gtest pretty printing.
BUG=532364
==========

to

==========
Add the gtest pretty printer for ResourceError

This will be used by https://codereview.chromium.org/1257343003/.

This CL ignores a presubmit error:
  "Streams are highly discouraged.  [readability/streams]"
  in PlatformTestPrinters.cpp.
Because:
  std::ostream is required for gtest pretty printing.
BUG=532364

Committed: https://crrev.com/f09dff4405b70b9b27fba049dd7cf59c2e585515
Cr-Commit-Position: refs/heads/master@{#381034}
==========

[commit-bot: I haz the power, 2016-03-14 19:12:06]

Patchset 6 (id:??) landed as
https://crrev.com/f09dff4405b70b9b27fba049dd7cf59c2e585515
Cr-Commit-Position: refs/heads/master@{#381034}