Flag for D3DVsync feature on chrome://flags

Flag for D3DVsync feature on chrome://flags

BUG=467617
Review-Url: https://codereview.chromium.org/2867053005
Cr-Commit-Position: refs/heads/master@{#470636}
Committed: https://chromium.googlesource.com/chromium/src/+/14bed0b13cd9a3620834c6cb6a00cfddacfb8eff

[stanisc, 2017-05-09 22:08:38]

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

[commit-bot: I haz the power, 2017-05-09 22:10:25]

Dry run: CQ is trying da patch.

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

[stanisc, 2017-05-09 23:35:03]

Description was changed from

==========
D3DVsync flag

BUG=
==========

to

==========
Flag for D3DVsync feature on chrome://flags

BUG=467617
==========

[stanisc, 2017-05-09 23:41:32]

stanisc@chromium.org changed reviewers:
+ isherman@chromium.org, thestig@chromium.org

[stanisc, 2017-05-09 23:41:32]

isherman@chromium.org: Please review changes in 
tools/metrics/histograms/enums.xml

thestig@chromium.org: Please review all other changes

[Ilya Sherman, 2017-05-09 23:50:23]

On 2017/05/09 23:41:32, stanisc wrote:
> mailto:isherman@chromium.org: Please review changes in 
> tools/metrics/histograms/enums.xml

For future reference, enums.xml changes are reviewable by any committer =)

That said, enums.xml changes in this CL LGTM.

[Lei Zhang, 2017-05-09 23:54:25]

lgtm

https://codereview.chromium.org/2867053005/diff/1/chrome/browser/about_flags.cc
File chrome/browser/about_flags.cc (right):

https://codereview.chromium.org/2867053005/diff/1/chrome/browser/about_flags....
chrome/browser/about_flags.cc:2888: #endif  // OS_WIN
defined(OS_WIN), or just omit the comment since the #if block is so short.

[stanisc, 2017-05-10 00:06:05]

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

[commit-bot: I haz the power, 2017-05-10 00:07:30]

Dry run: 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-05-10 02:09:41]

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

[commit-bot: I haz the power, 2017-05-10 02:09:42]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, build has
not started yet; builder either lacks capacity or does not exist (misspelled?))
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, build has not started yet;
builder either lacks capacity or does not exist (misspelled?))
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, build has not started yet; builder either lacks capacity or does
not exist (misspelled?))
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, build
has not started yet; builder either lacks capacity or does not exist
(misspelled?))
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
build has not started yet; builder either lacks capacity or does not exist
(misspelled?))
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, build has not started yet; builder either lacks capacity or does
not exist (misspelled?))
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, build has not started yet; builder either lacks capacity or does
not exist (misspelled?))
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, build has not started yet; builder either lacks capacity or does
not exist (misspelled?))
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, build
has not started yet; builder either lacks capacity or does not exist
(misspelled?))
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
build has not started yet; builder either lacks capacity or does not exist
(misspelled?))

[stanisc, 2017-05-10 02:14:56]

The CQ bit was checked by stanisc@chromium.org

[stanisc, 2017-05-10 02:14:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/2867053005/#ps20001
(title: "CR feedback")

[commit-bot: I haz the power, 2017-05-10 02:16: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, 2017-05-10 04:19:03]

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

[commit-bot: I haz the power, 2017-05-10 04:19:04]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, build
has not started yet; builder either lacks capacity or does not exist
(misspelled?))

[stanisc, 2017-05-10 17:35:12]

The CQ bit was checked by stanisc@chromium.org

[commit-bot: I haz the power, 2017-05-10 17:36:51]

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-05-10 17:46:57]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494437712748310,
"parent_rev": "cf4bac9c57a560d4502295194eed12a1df04ccce", "commit_rev":
"14bed0b13cd9a3620834c6cb6a00cfddacfb8eff"}

[commit-bot: I haz the power, 2017-05-10 17:47:08]

Description was changed from

==========
Flag for D3DVsync feature on chrome://flags

BUG=467617
==========

to

==========
Flag for D3DVsync feature on chrome://flags

BUG=467617

Review-Url: https://codereview.chromium.org/2867053005
Cr-Commit-Position: refs/heads/master@{#470636}
Committed:
https://chromium.googlesource.com/chromium/src/+/14bed0b13cd9a3620834c6cb6a00...
==========

[commit-bot: I haz the power, 2017-05-10 17:47:09]

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