Bug 67896 - Web Inspector: Profiler: Overlapping data in function names column
Summary: Web Inspector: Profiler: Overlapping data in function names column
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Mikhail Naganov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-10 15:03 PDT by Mikhail Naganov
Modified: 2011-09-13 08:32 PDT (History)
10 users (show)

See Also:


Attachments
Patch (2.27 KB, patch)
2011-09-10 15:06 PDT, Mikhail Naganov
pfeldman: review+
mnaganov: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mikhail Naganov 2011-09-10 15:03:55 PDT
Two problems, but of a similar kind.

From http://code.google.com/p/chromium/issues/detail?id=92526:

Google Chrome	15.0.849.0 (Official Build 96325) dev
OS tested: Gunbutu lucid (10.4)

What steps will reproduce the problem?
1.launch chrome
2.navigate to amazon.com
3.ctrl+shfit+i to bring up devtool
4. go to profile, click start profiling
5. navigate to some items in amazon
6. observe the profile saved

What is the expected output? What do you see instead?
all URL links should be properly list under function dropdown menu in Function column.

currently those very long URl link will overlap the function name, especially when this function menu dropdown has sub-menu. URL link will overlap the function name pretty badly.


From http://code.google.com/p/chromium/issues/detail?id=95904:

What steps will reproduce the problem?
1.Open the browser.
2.Open the mentioned URL.
3.Open DevTools (Ctrl+Shift+I on Win/Linux, Command+Option+I on Mac)
4.Go to 'Profiles' page
5.Press 'Take heap snapshot' button (eye icon on the left side of status bar)
6.Now Click on the Snapshot 1
7.Now in the right side column "Constructor" keep expanding the items.Now we can see the
Overlapping of Column data to next column area.

What is the expected result?
Should not be overlapped to next column data area.

What happens instead?
Overlapping column data of Snapshot to next column data area in the Profiles.
Comment 1 Mikhail Naganov 2011-09-10 15:06:27 PDT
Created attachment 106985 [details]
Patch
Comment 2 Pavel Feldman 2011-09-13 02:38:04 PDT
Comment on attachment 106985 [details]
Patch

Could you please make sure that it does not break the network panel and cookies grids.
Comment 3 Mikhail Naganov 2011-09-13 08:18:37 PDT
(In reply to comment #2)
> (From update of attachment 106985 [details])
> Could you please make sure that it does not break the network panel and cookies grids.

Thanks! Both grids look OK.

table-layout:fixed prescribes to use column width specified in style, not calculate them based on contents length (otherwise, if you have oversized contents, columns widths can be adjusted and the resulting table can be wider than specified in style rules);
and overflow:hidden cuts off contents that overflow columns.

I guess, both network panel and cookies don't requre an opposite behaviour.
Comment 4 Mikhail Naganov 2011-09-13 08:32:50 PDT
Manually committed: http://trac.webkit.org/changeset/95027

2011-09-10  Mikhail Naganov  <mnaganov@chromium.org>

        Web Inspector: Profiler: Fix overlapping data in function names column.
        https://bugs.webkit.org/show_bug.cgi?id=67896

        Reviewed by Pavel Feldman.

        * inspector/front-end/ProfileDataGridTree.js:
        (WebInspector.ProfileDataGridNode.prototype.createCell):
        * inspector/front-end/dataGrid.css:
        (.data-grid table.data):
        (.data-grid td):