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

Issue 622530 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Out until 24 Jan
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Move Aura-only tests out of site_per_process_browsertest.cc

Project Member Reported by creis@chromium.org, Jun 23 2016

Issue description

There's a huge section of Aura-only tests in the middle of site_per_process_browsertest.cc, which is a multi-thousand line file.  This makes it confusing to tell which tests run on which platforms.  (Just noticed it as new Aura-only tests were being added in https://codereview.chromium.org/2034213002/.)

We should move these out into an Aura specific test file, to make it clear which test coverage we're missing on other platforms like Mac (and to make the file a bit smaller).

Alex, would you be willing to split this out when you get a chance?  (I'm mainly picking you because so much of the team is OOO at the moment, so feel free to find another owner.)  :)
 

Comment 1 by nasko@chromium.org, Jun 23 2016

Feel free to punt it to me.
Cc: alex...@chromium.org
Owner: nasko@chromium.org
Status: Assigned (was: Untriaged)
Happy to do it, but I probably won't get to it for another week or two.  nasko@: sure, I'll assign to you for now if you can get to it sooner, otherwise I'll plan on stealing it back in a couple of weeks. :)

Sign in to add a comment