Enable 'chrome/browser/net' and 'chrome/browser/extensions' related browser_tests on coverage bot.

Enable 'chrome/browser/net' and 'chrome/browser/extensions' related browser_tests on coverage bot.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170987

[John Grabowski, 2012-12-03 18:32:00]

https://codereview.chromium.org/11415264/diff/1/tools/code_coverage/coverage_...
File tools/code_coverage/coverage_posix.py (right):

https://codereview.chromium.org/11415264/diff/1/tools/code_coverage/coverage_...
tools/code_coverage/coverage_posix.py:255: **Below are the
src/chrome/browser/net related tests enabled.**
Seems like it would be better to place the **header** comments inline to the
variable definition.  Avoids some text dup and prevents the comments from
drifting.

[pshenoy, 2012-12-03 18:59:33]

https://codereview.chromium.org/11415264/diff/1/tools/code_coverage/coverage_...
File tools/code_coverage/coverage_posix.py (right):

https://codereview.chromium.org/11415264/diff/1/tools/code_coverage/coverage_...
tools/code_coverage/coverage_posix.py:255: **Below are the
src/chrome/browser/net related tests enabled.**
On 2012/12/03 18:32:00, John Grabowski wrote:
> Seems like it would be better to place the **header** comments inline to the
> variable definition.  Avoids some text dup and prevents the comments from
> drifting.

Done.

[pshenoy, 2012-12-03 19:07:50]

Forgot to remove the pdb statements. Removed in this patchset.

[John Grabowski, 2012-12-03 19:24:33]

LGTM with text deletion

https://codereview.chromium.org/11415264/diff/6002/tools/code_coverage/covera...
File tools/code_coverage/coverage_posix.py (right):

https://codereview.chromium.org/11415264/diff/6002/tools/code_coverage/covera...
tools/code_coverage/coverage_posix.py:253: (# Below are the downloads related
tests enabled.
"Below are" is redundant.
"enabled" is also a bit redundant since the containing variable is named
gTestInclusions.

Can you remove that text?

[pshenoy, 2012-12-03 19:29:09]

https://codereview.chromium.org/11415264/diff/6002/tools/code_coverage/covera...
File tools/code_coverage/coverage_posix.py (right):

https://codereview.chromium.org/11415264/diff/6002/tools/code_coverage/covera...
tools/code_coverage/coverage_posix.py:253: (# Below are the downloads related
tests enabled.
On 2012/12/03 19:24:33, John Grabowski wrote:
> "Below are" is redundant.
> "enabled" is also a bit redundant since the containing variable is named
> gTestInclusions.
> 
> Can you remove that text?

Done.

[commit-bot: I haz the power, 2012-12-03 20:33:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pshenoy@chromium.org/11415264/1004

[commit-bot: I haz the power, 2012-12-03 21:25:41]

Retried try job too often on win_rel for step(s) content_browsertests

[commit-bot: I haz the power, 2012-12-03 21:29:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pshenoy@chromium.org/11415264/1004

[commit-bot: I haz the power, 2012-12-04 03:32:01]

Retried try job too often on mac_rel for step(s) browser_tests

[commit-bot: I haz the power, 2012-12-04 04:14:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pshenoy@chromium.org/11415264/1004

[commit-bot: I haz the power, 2012-12-04 06:54:49]

Retried try job too often on mac_rel for step(s) browser_tests

[commit-bot: I haz the power, 2012-12-04 15:53:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pshenoy@chromium.org/11415264/1004

[commit-bot: I haz the power, 2012-12-04 17:15:59]

Change committed as 170987