--
--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-dev
---
You received this message because you are subscribed to the Google Groups "Chromium-dev" group.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/chromium-dev/23624630.1r3eYUQgxm%40twilight.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/chromium-dev/CAEoffTCQGt8AoorgGGRUssTq5Hm23Uh2yvHegf_A%2B9Rzz5_8_g%40mail.gmail.com.
473 lines of code across 84 files seems a bit much to disable a feature. Maybe there are different places we could make the cut that aren't quite as intrusive?
On Friday, August 24, 2018 at 4:42:14 PM UTC-4, Allan Sandfeld Jensen wrote:On Freitag, 24. August 2018 22:27:31 CEST Brett Wilson wrote:
> You can see the patch that removed the defines:
> https://chromium-review.googlesource.com/c/chromium/src/+/1059408
>
> There were a lot, and they were in some really hairy files where we'd want
> to reduce complexity.
>
> According to the change that removed the flag (also relatively large):
> https://chromium-review.googlesource.com/c/chromium/src/+/1044220
> it was "broken for months and no one has noticed".
>
Well, we noticed, but fixing it was very minor things like updating some
arguments to functions that now took one more argument. This is the kind of
stuff that usually just gets fixed in the next release of Chrome, but I can
see how that attitude can be a problem if all embedders think like that.
Br.
Allan
--
--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-dev
---
You received this message because you are subscribed to the Google Groups "Chromium-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to chromium-dev...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/chromium-dev/ade95923-24e5-47e3-8049-33ea0006aeaf%40chromium.org.