Bug 73392 - REGRESSION:Deeply cloned nodes' style don't seem to listen to changes during drags
Summary: REGRESSION:Deeply cloned nodes' style don't seem to listen to changes during ...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: chrome://newtab
Keywords:
Depends on: 73227
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-29 20:53 PST by Dan Beam
Modified: 2011-11-30 13:54 PST (History)
4 users (show)

See Also:


Attachments
a screenshot of the manifestation of this issue (47.68 KB, image/png)
2011-11-29 20:55 PST, Dan Beam
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Beam 2011-11-29 20:53:15 PST
A recent WebKit roll in Chromium (http://crrev.com/111843) introduced an issue (http://crbug.com/105810) that causes clones of the source element to ignore style changes (i.e. changing draggingNode.cloneNode(true).style.left doesn't work).  One of these revs (http://goo.gl/FZnkK) introduced this.

Here are some simple test cases:
- using cloneNode - http://jsfiddle.net/wz384/9/
- using separate node - http://jsfiddle.net/wz384/13/

The manifestation of this bug is that dragging on the New Tab Page on Chrome 17.0.954.0 (today's Canary on MacOS) is broken as we're using a clone of the node a user originally dragged as a drag representation.  I've attached a screenshot to show what this looks like.  Notice that the mouse has started dragging but the clone isn't able to move like it used to (http://goo.gl/TveP5).

[Enclosure: chrome_new_tab_page_drag_representation_issue.png]
Comment 1 Dan Beam 2011-11-29 20:55:07 PST
Created attachment 117112 [details]
a screenshot of the manifestation of this issue
Comment 2 Dan Beam 2011-11-29 21:20:32 PST
This smells highly like a dupe of bug 73227.
Comment 3 Andreas Kling 2011-11-30 13:48:56 PST
(In reply to comment #2)
> This smells highly like a dupe of bug 73227.

I certainly agree with this assessment. Did you have a chance to verify that it got fixed?
Comment 4 Dan Beam 2011-11-30 13:54:47 PST
> I certainly agree with this assessment. Did you have a chance to verify that it got fixed?

We haven't rolled to/past the fixing rev (r101502).  We're still on r101316 at the moment (http://src.chromium.org/viewvc/chrome/trunk/src/DEPS?view=markup).