Move browser_components_metrics.py into a subdirectory of tools/metrics/

Move //tools/metrics into //components/ directory.

BUG=223859
R=joi@chromium.org

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

[Ilya Sherman, 2013-03-26 07:45:12]

Jói, are these files still used?  I tried running the script in a clean branch,
and got the following error output:

$ python tools/metrics/browser_components_metrics.py 
All metrics are for chrome/browser.

OS ifdefs, all:                   2713
OS ifdefs, -tests:                1876
Traceback (most recent call last):
  File "tools/metrics/browser_components_metrics.py", line 64, in <module>
    sys.exit(main())
  File "tools/metrics/browser_components_metrics.py", line 58, in main
    generator.CountViolations(False))
  File "tools/metrics/browser_components_metrics.py", line 44, in
CountViolations
    skip_tests=skip_tests)
  File "tools/metrics/../checkdeps/checkdeps.py", line 157, in __init__
    self._ApplyDirectoryRulesAndSkipSubdirs(Rules(), self.base_directory)
  File "tools/metrics/../checkdeps/checkdeps.py", line 313, in
_ApplyDirectoryRulesAndSkipSubdirs
    dir_path)
  File "tools/metrics/../checkdeps/checkdeps.py", line 303, in
_ApplyDirectoryRules
    specific_include_rules, norm_dir_name),
  File "tools/metrics/../checkdeps/checkdeps.py", line 195, in _ApplyRules
    (cur_dir, self.base_directory))
Exception: Internal error: base directory is not at the beginning for
  tools/metrics/../.. and base dir
  tools/metrics/../..

[tfarina, 2013-03-27 13:21:58]

On 2013/03/26 07:45:12, Ilya Sherman wrote:
> Jói, are these files still used?
Yeah, I'd say yes, at least I am.

> I tried running the script in a clean branch,
> and got the following error output:
> 
> $ python tools/metrics/browser_components_metrics.py 
I used it on Monday without problems.

I just run:

$ ./tools/metrics/browser_components_metrics.py 

Maybe a better place would be //components/tools/metrics? I don't have strong
feelings though, and the decision is not mine :)

[Ilya Sherman, 2013-03-28 00:13:37]

On 2013/03/27 13:21:58, tfarina wrote:
> On 2013/03/26 07:45:12, Ilya Sherman wrote:
> > I tried running the script in a clean branch,
> > and got the following error output:
> > 
> > $ python tools/metrics/browser_components_metrics.py 
> I used it on Monday without problems.
> 
> I just run:
> 
> $ ./tools/metrics/browser_components_metrics.py 

Ah, yes, that invocation runs without errors.  Thanks!

> Maybe a better place would be //components/tools/metrics? I don't have strong
> feelings though, and the decision is not mine :)

That sounds fine to me.  Jói, WDYT?

[Jói, 2013-04-03 14:00:38]

//components/tools/metrics SGTM.

On Thu, Mar 28, 2013 at 12:13 AM,  <isherman@chromium.org> wrote:
> On 2013/03/27 13:21:58, tfarina wrote:
>>
>> On 2013/03/26 07:45:12, Ilya Sherman wrote:
>> > I tried running the script in a clean branch,
>> > and got the following error output:
>> >
>> > $ python tools/metrics/browser_components_metrics.py
>> I used it on Monday without problems.
>
>
>> I just run:
>
>
>> $ ./tools/metrics/browser_components_metrics.py
>
>
> Ah, yes, that invocation runs without errors.  Thanks!
>
>
>> Maybe a better place would be //components/tools/metrics? I don't have
>> strong
>> feelings though, and the decision is not mine :)
>
>
> That sounds fine to me.  Jói, WDYT?
>
> https://codereview.chromium.org/12782019/

[Ilya Sherman, 2013-04-04 05:33:23]

On 2013/04/03 14:00:38, Jói wrote:
> //components/tools/metrics SGTM.

Done.

[Jói, 2013-04-04 09:59:55]

LGTM, thanks.

[commit-bot: I haz the power, 2013-04-04 10:20:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/12782019/5001

[commit-bot: I haz the power, 2013-04-04 13:15:03]

Retried try job too often on win_rel for step(s) sync_integration_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-04-04 21:57:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/12782019/5001

[commit-bot: I haz the power, 2013-04-05 02:05:22]

Change committed as 192443