[Findit] Update found_project and found_components.

[Findit] Update found_project and found_components.

BUG=644411
Committed: https://chromium.googlesource.com/infra/infra/+/9191cc8075241cd6f90bb5382c01999d8cbe0ed9

[Sharu Jiang, 2016-09-27 21:45:47]

Patchset #1 (id:1) has been deleted

[Sharu Jiang, 2016-09-27 21:48:55]

Patchset #1 (id:20001) has been deleted

[Sharu Jiang, 2016-09-27 21:49:23]

Description was changed from

==========
[Findit] Update found_project and found_components.

BUG=644411
==========

to

==========
[Findit] Update found_project and found_components.

BUG=644411
==========

[Sharu Jiang, 2016-09-27 21:49:24]

katesonia@chromium.org changed reviewers:
+ chanli@chromium.org, lijeffrey@chromium.org, stgao@chromium.org

[Sharu Jiang, 2016-09-27 21:49:36]

PTAL ：）

[Sharu Jiang, 2016-09-27 21:49:38]

PTAL ：）

[stgao, 2016-09-27 23:14:52]

lgtm if clients are OK with the change to "found".

https://codereview.chromium.org/2373133002/diff/40001/appengine/findit/crash/...
File appengine/findit/crash/findit_for_chromecrash.py (right):

https://codereview.chromium.org/2373133002/diff/40001/appengine/findit/crash/...
appengine/findit/crash/findit_for_chromecrash.py:39: "found": true, # Indicate
whether Findit found any suspects.
As the definition of "found" is changed, we should update this.

More importantly, you should talk to clients like Fracas and Cracas to ensure
this won't cause confusion on their end.

I'm fine with this change if no confusion for clients.

[Sharu Jiang, 2016-09-27 23:20:04]

https://codereview.chromium.org/2373133002/diff/40001/appengine/findit/crash/...
File appengine/findit/crash/findit_for_chromecrash.py (right):

https://codereview.chromium.org/2373133002/diff/40001/appengine/findit/crash/...
appengine/findit/crash/findit_for_chromecrash.py:39: "found": true, # Indicate
whether Findit found any suspects.
On 2016/09/27 23:14:52, stgao (slow) wrote:
> As the definition of "found" is changed, we should update this.
> 
> More importantly, you should talk to clients like Fracas and Cracas to ensure
> this won't cause confusion on their end.
> 
> I'm fine with this change if no confusion for clients.

I will inform clients and update this in related docs.

[Sharu Jiang, 2016-09-29 19:50:12]

Patchset #3 (id:80001) has been deleted

[Sharu Jiang, 2016-09-29 19:50:20]

The CQ bit was checked by katesonia@chromium.org

[Sharu Jiang, 2016-09-29 19:50:20]

The patchset sent to the CQ was uploaded after l-g-t-m from stgao@chromium.org
Link to the patchset: https://codereview.chromium.org/2373133002/#ps100001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-09-29 19:50:28]

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-09-29 20:04:46]

Description was changed from

==========
[Findit] Update found_project and found_components.

BUG=644411
==========

to

==========
[Findit] Update found_project and found_components.

BUG=644411

Committed:
https://chromium.googlesource.com/infra/infra/+/9191cc8075241cd6f90bb5382c019...
==========

[commit-bot: I haz the power, 2016-09-29 20:04:47]

Committed patchset #3 (id:100001) as
https://chromium.googlesource.com/infra/infra/+/9191cc8075241cd6f90bb5382c019...