Fix broken chrome_proxy unit tests

Fix broken chrome_proxy unit tests

testHTTPResponseTimelineRecorder was failing due to an occasional favicon
request which would occassional cause the test to fail.  To fix the issue
it will now filter out any favicon requests before validation.

testChromeProxyMetricForBlockOnce failed to due not being updated when
the BlockOnce case was changed.  I also renamed the metrics we store in the
test to be consistent with other cases.

Last item is adding myself to OWNERS for chrome_proxy tests.

BUG=452279, 449979, 455269
Committed: https://crrev.com/2464c8bdbf7c59d12a7b8572a82a1eedc7164652
Cr-Commit-Position: refs/heads/master@{#367098}

[bustamante, 2015-12-29 19:44:01]

Description was changed from

==========
Fix broken chrome_proxy unit tests

testHTTPResponseTimelineRecorder was failing due to an occasional favicon
request which would occassional cause the test to fail.  To fix the issue
it will now filter out any favicon requests before validation.

testChromeProxyMetricForBlockOnce failed to due not being updated when
the BlockOnce case was changed.  I also renamed the metrics we store in the
test to be consistent with other cases.

Last item is adding myself to OWNERS for chrome_proxy tests.

BUG=452279,449979,455269
==========

to

==========
Fix broken chrome_proxy unit tests

testHTTPResponseTimelineRecorder was failing due to an occasional favicon
request which would occassional cause the test to fail.  To fix the issue
it will now filter out any favicon requests before validation.

testChromeProxyMetricForBlockOnce failed to due not being updated when
the BlockOnce case was changed.  I also renamed the metrics we store in the
test to be consistent with other cases.

Last item is adding myself to OWNERS for chrome_proxy tests.

BUG=452279,449979,455269
==========

[bustamante, 2015-12-29 19:44:01]

bustamante@chromium.org changed reviewers:
+ bengr@chromium.org, sclittle@chromium.org

[bustamante, 2015-12-29 19:48:37]

bustamante@chromium.org changed reviewers:
+ bustamante@chromium.org

[bustamante, 2015-12-29 19:48:37]

PTAL when you can

[sclittle, 2015-12-29 19:52:25]

LGTM! thanks for fixing these tests

[bustamante, 2015-12-29 19:54:14]

The CQ bit was checked by bustamante@chromium.org

[commit-bot: I haz the power, 2015-12-29 19:54:23]

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

[commit-bot: I haz the power, 2015-12-29 20:10:03]

Description was changed from

==========
Fix broken chrome_proxy unit tests

testHTTPResponseTimelineRecorder was failing due to an occasional favicon
request which would occassional cause the test to fail.  To fix the issue
it will now filter out any favicon requests before validation.

testChromeProxyMetricForBlockOnce failed to due not being updated when
the BlockOnce case was changed.  I also renamed the metrics we store in the
test to be consistent with other cases.

Last item is adding myself to OWNERS for chrome_proxy tests.

BUG=452279,449979,455269
==========

to

==========
Fix broken chrome_proxy unit tests

testHTTPResponseTimelineRecorder was failing due to an occasional favicon
request which would occassional cause the test to fail.  To fix the issue
it will now filter out any favicon requests before validation.

testChromeProxyMetricForBlockOnce failed to due not being updated when
the BlockOnce case was changed.  I also renamed the metrics we store in the
test to be consistent with other cases.

Last item is adding myself to OWNERS for chrome_proxy tests.

BUG=452279,449979,455269
==========

[commit-bot: I haz the power, 2015-12-29 20:10:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-29 20:10:50]

Description was changed from

==========
Fix broken chrome_proxy unit tests

testHTTPResponseTimelineRecorder was failing due to an occasional favicon
request which would occassional cause the test to fail.  To fix the issue
it will now filter out any favicon requests before validation.

testChromeProxyMetricForBlockOnce failed to due not being updated when
the BlockOnce case was changed.  I also renamed the metrics we store in the
test to be consistent with other cases.

Last item is adding myself to OWNERS for chrome_proxy tests.

BUG=452279,449979,455269
==========

to

==========
Fix broken chrome_proxy unit tests

testHTTPResponseTimelineRecorder was failing due to an occasional favicon
request which would occassional cause the test to fail.  To fix the issue
it will now filter out any favicon requests before validation.

testChromeProxyMetricForBlockOnce failed to due not being updated when
the BlockOnce case was changed.  I also renamed the metrics we store in the
test to be consistent with other cases.

Last item is adding myself to OWNERS for chrome_proxy tests.

BUG=452279,449979,455269

Committed: https://crrev.com/2464c8bdbf7c59d12a7b8572a82a1eedc7164652
Cr-Commit-Position: refs/heads/master@{#367098}
==========

[commit-bot: I haz the power, 2015-12-29 20:10:51]

Patchset 2 (id:??) landed as
https://crrev.com/2464c8bdbf7c59d12a7b8572a82a1eedc7164652
Cr-Commit-Position: refs/heads/master@{#367098}