Bug 20549 - Resize cursor not shown for some nested framesets.
Summary: Resize cursor not shown for some nested framesets.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Frames (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-27 12:51 PDT by Brad Garcia
Modified: 2008-08-28 10:46 PDT (History)
0 users

See Also:


Attachments
Example using framesets with columns. (397 bytes, text/html)
2008-08-27 12:54 PDT, Brad Garcia
no flags Details
Example using framesets with rows. (397 bytes, text/html)
2008-08-27 12:55 PDT, Brad Garcia
no flags Details
Fix for bug. (1.42 KB, patch)
2008-08-28 08:46 PDT, Brad Garcia
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brad Garcia 2008-08-27 12:51:30 PDT
A frameset that is 1) nested within another frameset, and 2) is not the first element of that frameset, will not show the "resize" cursor upon mouseover of the resize bar.

The frameset is still resizable, it's just that the cursor does not get updated, and remains as the default arrow cursor.
Comment 1 Brad Garcia 2008-08-27 12:54:27 PDT
Created attachment 23036 [details]
Example using framesets with columns.

Example of nested columnar framesets.  The bar on the left displays the resize cursor (outermost frame), while the bar on the right does not (the nested frame).
Comment 2 Brad Garcia 2008-08-27 12:55:21 PDT
Created attachment 23037 [details]
Example using framesets with rows.

For completeness, framesets using rows display the problem as well.
Comment 3 Brad Garcia 2008-08-28 08:46:08 PDT
Created attachment 23054 [details]
Fix for bug.

This patch does not contain a corresponding test.  I do not know how to write a test for this fix.  If there is a way, I would appreciate any help in writing one!
Comment 4 mitz 2008-08-28 09:03:34 PDT
Comment on attachment 23054 [details]
Fix for bug.

r=me. As far as I know there is currently no way to make a layout test for this.
Comment 5 mitz 2008-08-28 10:46:26 PDT
Landed in <http://trac.webkit.org/changeset/35973>.