New issue
Advanced search Search tips

Issue 680273 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

[Missing Tests]: Add flash plugin path to chrome://version

Project Member Reported by ligim...@chromium.org, Jan 11 2017

Issue description

Automated tests for the below commit have been missing.

Greg, would you mind adding them.

CL:
https://chromium.googlesource.com/chromium/src/+/6916e37b2aefca3f6b7c804c1cdc9e65b32b7bc7

Reference bug:
https://bugs.chromium.org/p/chromium/issues/detail?id=667826
 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d3dcfef1ef6dd498d0c77412742e654814c5a210

commit d3dcfef1ef6dd498d0c77412742e654814c5a210
Author: kerrnel <kerrnel@chromium.org>
Date: Thu Jan 12 20:06:04 2017

Add test for Flash information on chrome://version page.

This adds a test for the Flash information displayed on the
chrome://version page. The test cannot check the actual version or
location as it varies per build, or is disabled in Chromium builds.

BUG= 680273 

Review-Url: https://codereview.chromium.org/2629033002
Cr-Commit-Position: refs/heads/master@{#443334}

[modify] https://crrev.com/d3dcfef1ef6dd498d0c77412742e654814c5a210/chrome/browser/ui/browser_browsertest.cc

Cc: ligim...@chromium.org
Are we merging the test to M56? I'm not sure that I'd consider it critical.
Status: Fixed (was: Started)

Sign in to add a comment