Bug 88043 - Author Shadow DOM should be disabled in contenteditable temporarily.
Summary: Author Shadow DOM should be disabled in contenteditable temporarily.
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-31 21:33 PDT by Shinya Kawanaka
Modified: 2012-06-21 10:33 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Shinya Kawanaka 2012-05-31 21:33:21 PDT
When Shadow DOM is attached in contenteditable, it causes a lot of crashes while editing.

To make Shadow DOM go further, we should consider disabling Author Shadow DOM temporarily.
Adding User Agent Shadow DOM, which is used for <input>, <textarea>, <meter>, etc, should be OK though.

Note that we should be able to enable Author Shadow DOM for debugging in DRT!
Comment 1 Ryosuke Niwa 2012-05-31 21:42:30 PDT
(In reply to comment #0)
> When Shadow DOM is attached in contenteditable, it causes a lot of crashes while editing.

I don't think that's an acceptable solution. We can't disable a bunch of features that don't work well inside shadow DOM arbitrarily. We need to fix editing code to work well with shadow DOM.
Comment 2 Shinya Kawanaka 2012-05-31 22:03:33 PDT
(In reply to comment #1)
> (In reply to comment #0)
> > When Shadow DOM is attached in contenteditable, it causes a lot of crashes while editing.
> 
> I don't think that's an acceptable solution. We can't disable a bunch of features that don't work well inside shadow DOM arbitrarily. We need to fix editing code to work well with shadow DOM.

I understand it, but...

We're just thinking a kind of alternative plan.
Comment 3 Dimitri Glazkov (Google) 2012-06-01 09:33:26 PDT
(In reply to comment #2)
> (In reply to comment #1)
> > (In reply to comment #0)
> > > When Shadow DOM is attached in contenteditable, it causes a lot of crashes while editing.
> > 
> > I don't think that's an acceptable solution. We can't disable a bunch of features that don't work well inside shadow DOM arbitrarily. We need to fix editing code to work well with shadow DOM.
> 
> I understand it, but...
> 
> We're just thinking a kind of alternative plan.

I think we should discuss this idea first. I am kind of ambivalent about what's the right thing to do here. We did this with disabling author shadows on inputs, and now we nearly trained the few peeps who experiment with shadow DOM that inputs are not usable with shadow DOM, which seems like a bad thing.  On the other hand, we don't want to be crashing like drunk monkeys in ferraris.

Let's put it this way: I am confident in your abilities to climb HTML editing learning curve and identify the root issues (I am sure there are only a few of those) and fix this in a reasonable timeframe :)
Comment 4 Ryosuke Niwa 2012-06-01 11:15:35 PDT
(In reply to comment #3)
> I think we should discuss this idea first. I am kind of ambivalent about what's the right thing to do here. We did this with disabling author shadows on inputs, and now we nearly trained the few peeps who experiment with shadow DOM that inputs are not usable with shadow DOM, which seems like a bad thing.  On the other hand, we don't want to be crashing like drunk monkeys in ferraris.

Right, this is not a desirable solution if we want a good consistent API for the Web and want other browser vendors to implement it :)  For example, why would Microsoft or Mozilla should disable contenteditable in their shadow DOM if their editing code was much more sound than ours?

> Let's put it this way: I am confident in your abilities to climb HTML editing learning curve and identify the root issues (I am sure there are only a few of those) and fix this in a reasonable timeframe :)

I'm more than happy to look at those crashes if you guys can send a list of crashes with stack trace & reduction.
Comment 5 Ryosuke Niwa 2012-06-21 10:33:30 PDT
Probably not.