Bug 96369 – Recognize the "--chromium-android" argument in run-webkit-tests

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Peter Beverloo]

Recognize the "--chromium-android" argument in run-webkit-tests

[Peter Beverloo]

Created attachment 163312 [details]
Patch

Comment on attachment 163312 [details]
Patch

Can you add a test to factory_unittest.py (see FactoryTest.assert_port)

[Peter Beverloo]

Created attachment 163320 [details]
Patch for landing

[Peter Beverloo]

(In reply to comment #2)
> (From update of attachment 163312 [details])
> Can you add a test to factory_unittest.py (see FactoryTest.assert_port)

Done, thank you!

[Peter Beverloo]

Committed r128164: <http://trac.webkit.org/changeset/128164>

[Dirk Pranke]

Is this really much of an improvement over '--platform chromium-android', which already worked? The reason we have the other shortcuts (--qt, --chromium) etc. is not primarily because they're shorter, but rather to indicate "pick the appropriate variant of X for the platform I'm running on".

[Peter Beverloo]

(In reply to comment #6)
> Is this really much of an improvement over '--platform chromium-android', which already worked? The reason we have the other shortcuts (--qt, --chromium) etc. is not primarily because they're shorter, but rather to indicate "pick the appropriate variant of X for the platform I'm running on".

Jochen asked me the same question. I chose to add the argument because it's consistent with the other flags, and with the input of the run-webkit-tests script which had to pass this along.

I'd be happy to change this.

[Dirk Pranke]

(In reply to comment #7)
> (In reply to comment #6)
> > Is this really much of an improvement over '--platform chromium-android', which already worked? The reason we have the other shortcuts (--qt, --chromium) etc. is not primarily because they're shorter, but rather to indicate "pick the appropriate variant of X for the platform I'm running on".
> 
> Jochen asked me the same question. I chose to add the argument because it's consistent with the other flags, and with the input of the run-webkit-tests script which had to pass this along.
> 
> I'd be happy to change this.

I don't feel strongly enough about this to want to revert it, but I'm not sure where this slope stops slipping ...

[Peter Beverloo]

(In reply to comment #8)
> (In reply to comment #7)
> > (In reply to comment #6)
> > > Is this really much of an improvement over '--platform chromium-android', which already worked? The reason we have the other shortcuts (--qt, --chromium) etc. is not primarily because they're shorter, but rather to indicate "pick the appropriate variant of X for the platform I'm running on".
> > 
> > Jochen asked me the same question. I chose to add the argument because it's consistent with the other flags, and with the input of the run-webkit-tests script which had to pass this along.
> > 
> > I'd be happy to change this.
> 
> I don't feel strongly enough about this to want to revert it, but I'm not sure where this slope stops slipping ...

If there are issues you feel we should be addressing then we should be talking about that. I value your input and concerns, so they'd be most welcome. Since you're comparing Android to a slipping slope it seems you have plenty.

I explained my rationale for making this change. I'll upload a patch to change this to --platform=chromium-android momentarily.

[Dirk Pranke]

(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > (In reply to comment #6)
> > > > Is this really much of an improvement over '--platform chromium-android', which already worked? The reason we have the other shortcuts (--qt, --chromium) etc. is not primarily because they're shorter, but rather to indicate "pick the appropriate variant of X for the platform I'm running on".
> > > 
> > > Jochen asked me the same question. I chose to add the argument because it's consistent with the other flags, and with the input of the run-webkit-tests script which had to pass this along.
> > > 
> > > I'd be happy to change this.
> > 
> > I don't feel strongly enough about this to want to revert it, but I'm not sure where this slope stops slipping ...
> 
> If there are issues you feel we should be addressing then we should be talking about that. I value your input and concerns, so they'd be most welcome. Since you're comparing Android to a slipping slope it seems you have plenty.
> 
> I explained my rationale for making this change. I'll upload a patch to change this to --platform=chromium-android momentarily.

Don't read too much into my comment, I'm not upset about the change, and I wouldn't bother uploading another patch.

My point was, I'm not sure how we decide to add --chromium-android and not add --chromium-mac or --chromium-mac-lion, but adding a flag for every possible port seems like the wrong solution.

Generally, I only like to add command line flags either when required to support new functionality or when they are shortcuts to really commonly used sets of other options (e.g., supporting --debug rather than requiring --configuration debug) that will make things easier for a lot of people. --chromium-android doesn't fall into the first category and it's not clear to me that enough people are going to use it that it'll fall into the latter.