Move search-related files from chrome/common to chrome/common/search

Move search-related files from chrome/common to chrome/common/search

This allows us to have a more specific OWNERS file than the one in chrome/ or chrome/common/

Committed: https://crrev.com/4dff7cfe023ae5086c4c2feca75d5738617b6493
Cr-Commit-Position: refs/heads/master@{#403869}

[Marc Treib, 2016-07-05 13:42:19]

Description was changed from

==========
Let search owners also own ntp_logging_events.h
==========

to

==========
Let search owners also own ntp_logging_events.h

That file only contains an enum that is used for NTP-related UMA metrics. No
point in going to a top-level owner every time that is touched.
==========

[Marc Treib, 2016-07-05 13:42:34]

treib@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-07-05 13:44:20]

what about moving all that stuff to a sub directory?

[Marc Treib, 2016-07-05 13:51:34]

On 2016/07/05 13:44:20, jochen wrote:
> what about moving all that stuff to a sub directory?

"all that stuff" is a single file :)  And it's used from render_messages.h,
which I assume is the reason it's here in the first place.

[jochen (gone - plz use gerrit), 2016-07-05 14:05:10]

On 2016/07/05 at 13:51:34, treib wrote:
> On 2016/07/05 13:44:20, jochen wrote:
> > what about moving all that stuff to a sub directory?
> 
> "all that stuff" is a single file :)  And it's used from render_messages.h,
which I assume is the reason it's here in the first place.

I meant all files in chrome/common that are related to instant/search (seems to
be 7)

[Marc Treib, 2016-07-05 15:53:32]

Description was changed from

==========
Let search owners also own ntp_logging_events.h

That file only contains an enum that is used for NTP-related UMA metrics. No
point in going to a top-level owner every time that is touched.
==========

to

==========
Move search-related files from chrome/common to chrome/common/search

This allows us to have a more specific OWNERS file than the one in chrome/ or
chrome/common/
==========

[Marc Treib, 2016-07-05 15:54:03]

On 2016/07/05 14:05:10, jochen wrote:
> On 2016/07/05 at 13:51:34, treib wrote:
> > On 2016/07/05 13:44:20, jochen wrote:
> > > what about moving all that stuff to a sub directory?
> > 
> > "all that stuff" is a single file :)  And it's used from render_messages.h,
> which I assume is the reason it's here in the first place.
> 
> I meant all files in chrome/common that are related to instant/search (seems
to
> be 7)

Ah alright, that makes sense. And so a single-line CL becomes a 30+ files CL  :D

[jochen (gone - plz use gerrit), 2016-07-05 15:56:57]

lgtm

[Marc Treib, 2016-07-05 16:01:24]

treib@chromium.org changed reviewers:
+ dcheng@chromium.org

[Marc Treib, 2016-07-05 16:01:24]

+dcheng - can I get a rubber-stamp for the include changes in render_messages.h
please? Thanks!

[dcheng, 2016-07-06 01:57:31]

rs lgtm for updating include paths in render_messages.h

[Marc Treib, 2016-07-06 08:03:31]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2016-07-06 08:03:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-06 08:53:44]

Description was changed from

==========
Move search-related files from chrome/common to chrome/common/search

This allows us to have a more specific OWNERS file than the one in chrome/ or
chrome/common/
==========

to

==========
Move search-related files from chrome/common to chrome/common/search

This allows us to have a more specific OWNERS file than the one in chrome/ or
chrome/common/
==========

[commit-bot: I haz the power, 2016-07-06 08:53:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-06 08:53:51]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-06 08:55:25]

Description was changed from

==========
Move search-related files from chrome/common to chrome/common/search

This allows us to have a more specific OWNERS file than the one in chrome/ or
chrome/common/
==========

to

==========
Move search-related files from chrome/common to chrome/common/search

This allows us to have a more specific OWNERS file than the one in chrome/ or
chrome/common/

Committed: https://crrev.com/4dff7cfe023ae5086c4c2feca75d5738617b6493
Cr-Commit-Position: refs/heads/master@{#403869}
==========

[commit-bot: I haz the power, 2016-07-06 08:55:27]

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