New issue
Advanced search Search tips

Issue 718206 link

Starred by 1 user
Status: Closed
Owner:
ryansturm@chromium.org
Closed: Oct 23
Cc:
ryansturm@chromium.org
bengr@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Add unittest coverage for ChromeResourceDispatcherHostDelegate::GetPreviewsState

Project Member Reported by ryansturm@chromium.org, May 3 2017 
This function is tested in browser tests, but should get unit test coverage.

Comment 1 by bengr@chromium.org, Nov 8 2017

Cc: ryansturm@chromium.org
Labels: -Pri-2 Pri-3
Owner: ----
Refreshed during triage.

Comment 2 by efoo@chromium.org, Dec 5 2017

Components: Blink>Previews

Comment 3 by efoo@chromium.org, Dec 5 2017

Components: -UI>Browser>Previews

Comment 4 by sclit...@chromium.org, Jan 24 2018 

Refreshed during triage.

Comment 5 by bengr@chromium.org, Mar 21 2018 

Refreshed during triage.

Comment 6 by bengr@chromium.org, Jun 22 2018

Labels: -Pri-3 M-69 Pri-2
Owner: tbansal@chromium.org
Status: Assigned (was: Available)

Comment 7 by tbansal@chromium.org, Jul 19 

ryansturm: I can't find this method call anymore:
https://cs.chromium.org/search/?q=Dispatcher.*Host.*Delegate+GetPreviewsState&sq=package:chromium&type=cs

Should this be marked as WontFix?

Comment 8 by tbansal@chromium.org, Jul 19

Labels: Needs-Feedback

Comment 9 by tbansal@chromium.org, Sep 5

Owner: ryansturm@chromium.org
Assigning to Ryan since Ryan is most familiar with the code and is also going to work on it as part of previews servicification.

Comment 10 by tbansal@chromium.org, Sep 5

Labels: -Needs-Feedback

Comment 11 by dougarnett@chromium.org, Oct 23

Status: Closed (was: Assigned)
This is now obsolete
►

Sign in to add a comment