Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1099)

Issue 11087028: Add croc config file for iOS code coverage. (Closed)

Created:
8 years, 2 months ago by lliabraa
Modified:
7 years, 1 month ago
CC:
chromium-reviews
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Add croc config file for iOS code coverage. This croc config file instruments all files with 'ios' in the path for test coverage. BUG=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=160860

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+71 lines, -0 lines) Patch
A build/ios/chrome_ios.croc View 1 chunk +71 lines, -0 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
lliabraa
8 years, 2 months ago (2012-10-09 13:54:27 UTC) #1
stuartmorgan
LGTM
8 years, 2 months ago (2012-10-09 14:02:26 UTC) #2
Randall Spangler
lgtm
8 years, 2 months ago (2012-10-09 15:02:34 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/lliabraa@chromium.org/11087028/1
8 years, 2 months ago (2012-10-09 15:05:22 UTC) #4
commit-bot: I haz the power
Change committed as 160860
8 years, 2 months ago (2012-10-09 17:00:59 UTC) #5
Nico
Do you use this for anything?
7 years, 1 month ago (2013-11-10 21:45:59 UTC) #6
lliabraa
On 2013/11/10 21:45:59, Nico (ooo until Nov 12) wrote: > Do you use this for ...
7 years, 1 month ago (2013-11-11 12:43:04 UTC) #7
Nico
Does anyone user that dashboard? The desktop chrome coverage dashboard had been broken for many ...
7 years, 1 month ago (2013-11-11 14:31:35 UTC) #8
stuartmorgan
On 2013/11/11 14:31:35, Nico (ooo until Nov 12) wrote: > Does anyone user that dashboard? ...
7 years, 1 month ago (2013-11-11 18:39:28 UTC) #9
Nico
7 years, 1 month ago (2013-11-11 22:00:02 UTC) #10
On Mon, Nov 11, 2013 at 10:39 AM, <stuartmorgan@chromium.org> wrote:

> On 2013/11/11 14:31:35, Nico (ooo until Nov 12) wrote:
>
>> Does anyone user that dashboard? The desktop chrome coverage dashboard had
>> been broken for many months and nobody really misses it.
>>
>
> As with desktop, it doesn't work consistently. It has been useful in the
> past
> though, and making it harder to get working again isn't ideal.
>
>
>  The coverage code makes some changes I want to make harder, so I thought
>> I'd
>>
> delete it.
>
> Maybe this should be brought up on chromium-dev? If there's really nobody
> interested in taking ownership of the general infrastructure, then I guess
> we're
> out of luck since we don't have the bandwidth to maintain all platforms,
> but
> maybe there are others interested in keeping it working even if they aren't
> regular consumers.
>

Sounds good:
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/3OSQwUXuZkg


>
> https://chromiumcodereview.appspot.com/11087028/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698