Add note pointing to internal histograms.xml location

Add note pointing to internal histograms.xml location

This helps Googlers find where the internal histograms are located, which
makes it a little easier to migrate them out to the public file.

BUG=none
TEST=none

Committed: https://crrev.com/42e72dcee17448fcb237b2afbc1862dfc392cf83
Cr-Commit-Position: refs/heads/master@{#333205}

[James Cook, 2015-06-05 18:08:37]

Patchset #2 (id:20001) has been deleted

[James Cook, 2015-06-05 18:09:32]

jamescook@chromium.org changed reviewers:
+ isherman@chromium.org

[James Cook, 2015-06-05 18:09:32]

isherman, PTAL

tools/histograms is its own git repo, so I'm still trying to figure out how to
add a README there. This should help for now.

[Ilya Sherman, 2015-06-05 20:56:08]

LGTM

On 2015/06/05 18:09:32, James Cook wrote:
> isherman, PTAL
> 
> tools/histograms is its own git repo, so I'm still trying to figure out how to
> add a README there. This should help for now.

I believe that tools/histograms is no longer used at all.  Rather than adding a
README file to it, perhaps we should delete the git repository?  (Please do
correct me if it seems to still be in use.)

[James Cook, 2015-06-05 21:50:33]

On 2015/06/05 20:56:08, Ilya Sherman wrote:
> LGTM
> 
> On 2015/06/05 18:09:32, James Cook wrote:
> > isherman, PTAL
> > 
> > tools/histograms is its own git repo, so I'm still trying to figure out how
to
> > add a README there. This should help for now.
> 
> I believe that tools/histograms is no longer used at all.  Rather than adding
a
> README file to it, perhaps we should delete the git repository?  (Please do
> correct me if it seems to still be in use.)

The last commit to tools/histograms says:

"    Re-create empty tools/histograms directory.
    
    This way, folks won't be broken by the removal.
    
    Plan is to delete this and the line from
    src-internal/DEPS that syncs it in a follow-up
    CL later.
    
    BUG=400825"

So yeah, maybe deleting it is the right thing to do. I'm just worried about what
got broken that necessitated adding it back. :-)

[James Cook, 2015-06-05 21:50:42]

The CQ bit was checked by jamescook@chromium.org

[commit-bot: I haz the power, 2015-06-05 21:51:47]

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

[Ilya Sherman, 2015-06-05 21:55:25]

On 2015/06/05 21:50:33, James Cook wrote:
> On 2015/06/05 20:56:08, Ilya Sherman wrote:
> > LGTM
> > 
> > On 2015/06/05 18:09:32, James Cook wrote:
> > > isherman, PTAL
> > > 
> > > tools/histograms is its own git repo, so I'm still trying to figure out
how
> to
> > > add a README there. This should help for now.
> > 
> > I believe that tools/histograms is no longer used at all.  Rather than
adding
> a
> > README file to it, perhaps we should delete the git repository?  (Please do
> > correct me if it seems to still be in use.)
> 
> The last commit to tools/histograms says:
> 
> "    Re-create empty tools/histograms directory.
>     
>     This way, folks won't be broken by the removal.
>     
>     Plan is to delete this and the line from
>     src-internal/DEPS that syncs it in a follow-up
>     CL later.
>     
>     BUG=400825"
> 
> So yeah, maybe deleting it is the right thing to do. I'm just worried about
what
> got broken that necessitated adding it back. :-)

Heh, okay.  Probably worth double-checking with whoever authored that commit. 
(Hopefully it wasn't me... because I don't remember it :P)

[chromium-reviews, 2015-06-05 21:57:05]

I pinged the bug and I'm talking to pgervais on infra.

On Fri, Jun 5, 2015 at 2:55 PM, <isherman@chromium.org> wrote:

> On 2015/06/05 21:50:33, James Cook wrote:
>
>> On 2015/06/05 20:56:08, Ilya Sherman wrote:
>> > LGTM
>> >
>> > On 2015/06/05 18:09:32, James Cook wrote:
>> > > isherman, PTAL
>> > >
>> > > tools/histograms is its own git repo, so I'm still trying to figure
>> out
>>
> how
>
>> to
>> > > add a README there. This should help for now.
>> >
>> > I believe that tools/histograms is no longer used at all.  Rather than
>>
> adding
>
>> a
>> > README file to it, perhaps we should delete the git repository?
>> (Please do
>> > correct me if it seems to still be in use.)
>>
>
>  The last commit to tools/histograms says:
>>
>
>  "    Re-create empty tools/histograms directory.
>>
>
>       This way, folks won't be broken by the removal.
>>
>
>       Plan is to delete this and the line from
>>      src-internal/DEPS that syncs it in a follow-up
>>      CL later.
>>
>
>       BUG=400825"
>>
>
>  So yeah, maybe deleting it is the right thing to do. I'm just worried
>> about
>>
> what
>
>> got broken that necessitated adding it back. :-)
>>
>
> Heh, okay.  Probably worth double-checking with whoever authored that
> commit.
> (Hopefully it wasn't me... because I don't remember it :P)
>
> https://codereview.chromium.org/1157663008/
>

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

[Ilya Sherman, 2015-06-05 22:01:06]

On 2015/06/05 21:57:05, James Cook wrote:
> I pinged the bug and I'm talking to pgervais on infra.

Thanks!

[commit-bot: I haz the power, 2015-06-06 00:05:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-06 00:05:38]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[James Cook, 2015-06-06 00:10:29]

The CQ bit was checked by jamescook@chromium.org

[commit-bot: I haz the power, 2015-06-06 00:12:16]

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

[commit-bot: I haz the power, 2015-06-06 02:49:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-06-06 02:50:37]

Patchset 1 (id:??) landed as
https://crrev.com/42e72dcee17448fcb237b2afbc1862dfc392cf83
Cr-Commit-Position: refs/heads/master@{#333205}