Tweak apk_size_regressions.md to be more explicit about a couple things

Tweak apk_size_regressions.md to be more explicit about a couple things

1. Say "Some regressions" rather than "Most regressions" at the top.
2. Point out that since bisects don't work, sheriffs are not to spend
time finding culprit
3. Add instructions for what to change when filing bugs

BUG=none

Review-Url: https://codereview.chromium.org/2675973002
Cr-Commit-Position: refs/heads/master@{#448328}
Committed: https://chromium.googlesource.com/chromium/src/+/6b10ff77d359100c6fd3e287feccd0342211b51d

[agrieve, 2017-02-03 16:51:53]

Description was changed from

==========
Tweak apk_size_regressions.md to be more explicit about a couple things

1. Say "Some regressions" rather than "Most regressions" at the top.
2. Point out that since bisects don't work, sheriffs are not to spend
time finding culprit
3. Add instructions for what to change when filing bugs

BUG=none
==========

to

==========
Tweak apk_size_regressions.md to be more explicit about a couple things

1. Say "Some regressions" rather than "Most regressions" at the top.
2. Point out that since bisects don't work, sheriffs are not to spend
time finding culprit
3. Add instructions for what to change when filing bugs

BUG=none
==========

[agrieve, 2017-02-03 16:51:53]

agrieve@chromium.org changed reviewers:
+ tdresser@chromium.org

[agrieve, 2017-02-03 16:52:13]

Some tweaks based on your feedback :)

[agrieve, 2017-02-06 15:10:48]

The CQ bit was checked by agrieve@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-06 15:11:03]

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

[tdresser, 2017-02-06 15:44:08]

LGTM % question.

https://codereview.chromium.org/2675973002/diff/1/tools/perf/docs/apk_size_re...
File tools/perf/docs/apk_size_regressions.md (right):

https://codereview.chromium.org/2675973002/diff/1/tools/perf/docs/apk_size_re...
tools/perf/docs/apk_size_regressions.md:43: * Remove label:
`Restrict-View-Google`
Why do we RVG until here, if we always remove RVG at this point?

[agrieve, 2017-02-06 16:35:56]

https://codereview.chromium.org/2675973002/diff/1/tools/perf/docs/apk_size_re...
File tools/perf/docs/apk_size_regressions.md (right):

https://codereview.chromium.org/2675973002/diff/1/tools/perf/docs/apk_size_re...
tools/perf/docs/apk_size_regressions.md:43: * Remove label:
`Restrict-View-Google`
On 2017/02/06 15:44:07, tdresser wrote:
> Why do we RVG until here, if we always remove RVG at this point?

AFAIK, there's currently no way to change the bug template. I'm sure anyone
going through this will wonder the same thing though, so added the link to the
bug to this list.

[agrieve, 2017-02-06 16:36:02]

The CQ bit was checked by agrieve@chromium.org

[agrieve, 2017-02-06 16:36:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdresser@chromium.org
Link to the patchset: https://codereview.chromium.org/2675973002/#ps20001
(title: "add bug link for template")

[commit-bot: I haz the power, 2017-02-06 16:36:20]

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, 2017-02-06 17:39:52]

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

[commit-bot: I haz the power, 2017-02-06 17:39:53]

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

[agrieve, 2017-02-06 18:06:48]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2017-02-06 18:07:26]

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, 2017-02-06 18:50:21]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486404408675870,
"parent_rev": "6cfb9e373f08dc861e7605de924933058a94ff17", "commit_rev":
"6b10ff77d359100c6fd3e287feccd0342211b51d"}

[commit-bot: I haz the power, 2017-02-06 18:50:55]

Description was changed from

==========
Tweak apk_size_regressions.md to be more explicit about a couple things

1. Say "Some regressions" rather than "Most regressions" at the top.
2. Point out that since bisects don't work, sheriffs are not to spend
time finding culprit
3. Add instructions for what to change when filing bugs

BUG=none
==========

to

==========
Tweak apk_size_regressions.md to be more explicit about a couple things

1. Say "Some regressions" rather than "Most regressions" at the top.
2. Point out that since bisects don't work, sheriffs are not to spend
time finding culprit
3. Add instructions for what to change when filing bugs

BUG=none

Review-Url: https://codereview.chromium.org/2675973002
Cr-Commit-Position: refs/heads/master@{#448328}
Committed:
https://chromium.googlesource.com/chromium/src/+/6b10ff77d359100c6fd3e287fecc...
==========

[commit-bot: I haz the power, 2017-02-06 18:50:56]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/6b10ff77d359100c6fd3e287fecc...