[Oilpan] Split sweep phase in tracing into two categories

Split sweep into "Lazy sweep" and "Complete sweep" phases in
about:tracing.

This change is following http://crrev.com/758163004

BUG=None
TEST=./tools/perf/run_tests help OilpanGCTimesTest --browser=release

Committed: https://crrev.com/35f317c778c0c36d1d5df412a7f0a97cc3dfdb78
Cr-Commit-Position: refs/heads/master@{#312572}

[peria, 2015-01-21 05:23:49]

peria@chromium.org changed reviewers:
+ kouhei@chromium.org, oilpan-reviews@chromium.org, skyostil@chromium.org

[peria, 2015-01-21 05:23:49]

PTL.

[haraken, 2015-01-21 05:25:17]

LGTM

[kouhei (in TOK), 2015-01-21 05:29:13]

Sorry. Not lgtm yet.

I think we should keep sweep metric to measure total time spent in sweep.

Also I'm not sure if oilpan_conservative_lazy_sweep ever happens.

[haraken, 2015-01-21 05:58:25]

> Also I'm not sure if oilpan_conservative_lazy_sweep ever happens.

What do you mean? Lazy sweeping happens in both conservative GC and precise GC.

[kouhei (in TOK), 2015-01-21 06:03:57]

On 2015/01/21 05:58:25, haraken wrote:
> > Also I'm not sure if oilpan_conservative_lazy_sweep ever happens.
> 
> What do you mean? Lazy sweeping happens in both conservative GC and precise
GC.

You're right. Please keep oilpan_conservative_lazy_sweep

[peria, 2015-01-21 06:15:32]

On 2015/01/21 05:29:13, kouhei wrote:
> Sorry. Not lgtm yet.
> 
> I think we should keep sweep metric to measure total time spent in sweep.
> 
> Also I'm not sure if oilpan_conservative_lazy_sweep ever happens.

Now in this CL, we have following 2*2*3=12 metrics for sweeping phases.
  oilpan_(precise|conservative)_(lazy|complete)_sweep(_total|_max|)
Should I add another metric "oilpan_sweep_total"?

[kouhei (in TOK), 2015-01-21 06:18:34]

On 2015/01/21 06:15:32, peria wrote:
> On 2015/01/21 05:29:13, kouhei wrote:
> > Sorry. Not lgtm yet.
> > 
> > I think we should keep sweep metric to measure total time spent in sweep.
> > 
> > Also I'm not sure if oilpan_conservative_lazy_sweep ever happens.
> 
> Now in this CL, we have following 2*2*3=12 metrics for sweeping phases.
>   oilpan_(precise|conservative)_(lazy|complete)_sweep(_total|_max|)
> Should I add another metric "oilpan_sweep_total"?

I think so.

[peria, 2015-01-21 06:36:34]

On 2015/01/21 06:18:34, kouhei wrote:
> On 2015/01/21 06:15:32, peria wrote:
> > On 2015/01/21 05:29:13, kouhei wrote:
> > > Sorry. Not lgtm yet.
> > > 
> > > I think we should keep sweep metric to measure total time spent in sweep.
> > > 
> > > Also I'm not sure if oilpan_conservative_lazy_sweep ever happens.
> > 
> > Now in this CL, we have following 2*2*3=12 metrics for sweeping phases.
> >   oilpan_(precise|conservative)_(lazy|complete)_sweep(_total|_max|)
> > Should I add another metric "oilpan_sweep_total"?
> 
> I think so.

done.  Its name is "oilpan_sweep", as we had.

[kouhei (in TOK), 2015-01-21 06:45:23]

lgtm

[Sami, 2015-01-21 16:09:36]

lgtm. I think the crrev.com link in the description is wrong by the way.

[peria, 2015-01-22 03:58:16]

On 2015/01/21 16:09:36, Sami wrote:
> lgtm. I think the http://crrev.com link in the description is wrong by the
way.

Oops, thank you for the pointer.
Fixed.

[peria, 2015-01-22 03:58:22]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2015-01-22 04:00:05]

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

[commit-bot: I haz the power, 2015-01-22 06:12:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-22 06:13:36]

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