New issue
Advanced search Search tips

Issue 912129 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug

Blocking:
issue 912155



Sign in to add a comment

flags: default & remove enable-canvas-2d-image-chromium

Project Member Reported by ellyjo...@chromium.org, Dec 5

Issue description

This flag should be enabled by default and removed.
 
Blocking: 912155
Cc: ccameron@chromium.org
Components: Blink>Canvas
Might be a good idea to change it to a base::Feature -- or is that not really helping?
A base::Feature is fine for now, but features are in my sights in the future :) the real goal here is to reduce the amount of runtime configurations chromium has, so if you reduced this to:

bool ShouldDoTheThing() {
#if defined(OS_MACOSX)
  return true;
#else
  return false;
#endif
}

that would be a-ok in my book.
It can be helpful to be able to disable this feature for debugging. So I'm OK with removing the "enable" flag.
#6: But do you then want to add a corresponding "disable" flag? If so, how long would we retain the flag for?
Cc: ellyjo...@chromium.org mcasas@chromium.org
Owner: ccameron@chromium.org

Comment 9 by ellyjo...@chromium.org, Today (15 hours ago)

ccameron: what should we do here? (open question in #7)

Sign in to add a comment