New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 625850 link

Starred by 2 users
Status: Fixed
Owner:
█ andersoncss@google.com
Email to this user bounced
Closed: Aug 2016
Cc:
georgesak@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Enable [TabManager] on TestingBrowserProcess

Project Member Reported by andersoncss@google.com, Jul 5 2016 
Enabling TabManager on testing BrowserProcess will allow us to have unittests that use TabManager.

https://cs.chromium.org/chromium/src/chrome/test/base/testing_browser_process.cc?l=371
Project Member

Comment 1 by sheriffbot@chromium.org, Jul 6 2016

Labels: Hotlist-Google
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 6 2016 

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

commit 13af30c7bfb22dae8f79ebe203d959c71c711a37
Author: andersoncss <andersoncss@google.com>
Date: Wed Jul 06 22:09:04 2016

To make it possible run all TabManager based browsertests as unittest.

BUG= 625850 

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

[modify] https://crrev.com/13af30c7bfb22dae8f79ebe203d959c71c711a37/chrome/test/base/testing_browser_process.cc
[modify] https://crrev.com/13af30c7bfb22dae8f79ebe203d959c71c711a37/chrome/test/base/testing_browser_process.h

Comment 3 by andersoncss@google.com, Aug 12 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment