Issue 3404 - Add platform directory to devenv path

Issue 3404 - Add platform directory to devenv path

[kzar, 2015-12-10 20:07:05]

Patch Set 1

[Sebastian Noack, 2015-12-12 00:42:25]

I'm undecided on this one. As discussed internally, I prefer to just get rid of
the Opera-specific builds, which would also help other cases.

But I see that the general problem here persists for Chrome vs. Safari and
wouldn't get any better if we add more platforms.

But I'd like to here Wladimir's opinion and rationale for doing it like it is.

[Wladimir Palant, 2016-01-25 14:25:38]

In general, I agree that this is useful. As to the implementation, this looks
like it will fail if the devenv directory doesn't exist already. Also, I'd
probably prefer devenv.chrome and devenv.safari rather than more directory
nesting.

[kzar, 2016-01-27 13:53:29]

On 2016/01/25 14:25:38, Wladimir Palant wrote:
> In general, I agree that this is useful. As to the implementation, this looks
> like it will fail if the devenv directory doesn't exist already. Also, I'd
> probably prefer devenv.chrome and devenv.safari rather than more directory
> nesting.

Sure, makes sense. Sorry I need to close this review and open a new one as
Rietveld is getting very confused since I've rebased.