WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
35966
Web Inspector: GraphBar elements at the bottom of Timeline are not visible.
https://bugs.webkit.org/show_bug.cgi?id=35966
Summary
Web Inspector: GraphBar elements at the bottom of Timeline are not visible.
Ilya Tikhonovsky
Reported
2010-03-10 04:11:51 PST
In case of long recording interval the latest GraphBar elements are not visible at all if they are small.
Attachments
[Patch] The fix for GraphBar's position.
(1.95 KB, patch)
2010-03-10 04:13 PST
,
Ilya Tikhonovsky
timothy
: review+
commit-queue
: commit-queue-
Details
Formatted Diff
Diff
Old screen shot with incorrect position of GraphBars.
(11.17 KB, image/png)
2010-03-10 04:18 PST
,
Ilya Tikhonovsky
no flags
Details
[Patch] The same patch but rebaselined. The fix fir Graph Bar's position.
(1.90 KB, patch)
2010-03-13 05:08 PST
,
Ilya Tikhonovsky
timothy
: review+
commit-queue
: commit-queue-
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Timothy Hatcher
Comment 1
2010-03-10 04:13:33 PST
Under the scrollbal?
Ilya Tikhonovsky
Comment 2
2010-03-10 04:13:56 PST
Created
attachment 50389
[details]
[Patch] The fix for GraphBar's position.
Ilya Tikhonovsky
Comment 3
2010-03-10 04:18:51 PST
Created
attachment 50391
[details]
Old screen shot with incorrect position of GraphBars.
Ilya Tikhonovsky
Comment 4
2010-03-10 04:24:54 PST
(In reply to
comment #1
)
> Under the scrollbal?
Yep. Usually the last elements were hidden by scroll bar. But the real reason of that was misspositioning because we are using expandOffset which for collapsing groups of elements, minimum width of the bars and the bar border size.
WebKit Commit Bot
Comment 5
2010-03-11 17:46:32 PST
Comment on
attachment 50389
[details]
[Patch] The fix for GraphBar's position. Rejecting patch 50389 from commit-queue. Failed to run "['/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', '--reviewer', 'Timothy Hatcher', '--force']" exit_code: 2 patching file WebCore/ChangeLog patch: **** malformed patch at line 18: Unreviewed build fix. Fix variable name change that somehow didn't patching file WebCore/inspector/front-end/TimelinePanel.js Hunk #1 succeeded at 570 (offset -2 lines). Full output:
http://webkit-commit-queue.appspot.com/results/602028
Ilya Tikhonovsky
Comment 6
2010-03-13 05:08:40 PST
Created
attachment 50654
[details]
[Patch] The same patch but rebaselined. The fix fir Graph Bar's position.
WebKit Commit Bot
Comment 7
2010-03-13 13:28:38 PST
Comment on
attachment 50654
[details]
[Patch] The same patch but rebaselined. The fix fir Graph Bar's position. Rejecting patch 50654 from commit-queue. Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--exit-after-n-failures=1', '--quiet']" exit_code: 1 Running build-dumprendertree Compiling Java tests make: Nothing to be done for `default'. Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests Testing 12482 test cases. fast/loader/api-test-new-window-data-load-base-url.html -> failed Exiting early after 1 failures. 7684 tests run. 128.69s total testing time 7683 test cases (99%) succeeded 1 test case (<1%) had incorrect layout 3 test cases (<1%) had stderr output Full output:
http://webkit-commit-queue.appspot.com/results/731012
Pavel Feldman
Comment 8
2010-03-15 06:02:39 PDT
Committing to
http://svn.webkit.org/repository/webkit/trunk
... M WebCore/ChangeLog M WebCore/inspector/front-end/TimelinePanel.js Committed
r55991
Eric Seidel (no email)
Comment 9
2010-03-15 12:06:07 PDT
I'm guessing this was
bug 35594
.
Ilya Tikhonovsky
Comment 10
2010-03-15 13:02:53 PDT
This change-list just change absolute positions of bar elements in the Timeline view of Web Inspector when WebInspector is opened and Timeline is recorded. I think fast/loader/api-test-new-window-data-load-base-url.html test just flaky. This test passed successfully when the patch was landed manually.
Eric Seidel (no email)
Comment 11
2010-03-15 13:04:25 PDT
Yes. Definitely appears to be flakey. You should feel encouraged to file bugs and simply re-queue (by marking cq+ again) if/when the commit-queue ever hits a flakey test.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug