WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
113341
Web Inspector: [Mac] Support dock-to-right
https://bugs.webkit.org/show_bug.cgi?id=113341
Summary
Web Inspector: [Mac] Support dock-to-right
Timothy Hatcher
Reported
2013-03-26 14:29:41 PDT
It is time to support dock to right on Mac.
Attachments
Proposed Change
(86.65 KB, patch)
2013-03-26 14:40 PDT
,
Timothy Hatcher
timothy
: commit-queue-
Details
Formatted Diff
Diff
Proposed Change (Take 2)
(93.19 KB, patch)
2013-03-26 15:47 PDT
,
Timothy Hatcher
timothy
: commit-queue-
Details
Formatted Diff
Diff
Archive of layout-test-results from gce-cr-linux-03 for chromium-linux-x86_64
(1008.69 KB, application/zip)
2013-03-26 17:07 PDT
,
WebKit Review Bot
no flags
Details
Proposed Change (Take 3)
(96.11 KB, patch)
2013-03-27 06:34 PDT
,
Timothy Hatcher
eflews.bot
: commit-queue-
Details
Formatted Diff
Diff
Proposed Change (Take 4)
(98.06 KB, patch)
2013-03-27 07:35 PDT
,
Timothy Hatcher
gtk-ews
: commit-queue-
Details
Formatted Diff
Diff
Proposed Change (Take 5)
(98.35 KB, patch)
2013-03-27 10:57 PDT
,
Timothy Hatcher
no flags
Details
Formatted Diff
Diff
Proposed Change (Take 6)
(99.14 KB, patch)
2013-03-27 12:13 PDT
,
Timothy Hatcher
joepeck
: review+
Details
Formatted Diff
Diff
Show Obsolete
(6)
View All
Add attachment
proposed patch, testcase, etc.
Timothy Hatcher
Comment 1
2013-03-26 14:30:01 PDT
<
rdar://problem/10368152
>
Timothy Hatcher
Comment 2
2013-03-26 14:40:28 PDT
Created
attachment 195166
[details]
Proposed Change This is likely going to brake some builds. Throwing to the EWS to find out what needs tweaked. The changes should be all ready to review still.
WebKit Review Bot
Comment 3
2013-03-26 14:45:25 PDT
Attachment 195166
[details]
did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/WebCore.exp.in', u'Source/WebCore/inspector/InspectorFrontendClient.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorFrontendClientLocal.h', u'Source/WebCore/inspector/InspectorFrontendHost.cpp', u'Source/WebCore/inspector/InspectorFrontendHost.h', u'Source/WebCore/inspector/InspectorFrontendHost.idl', u'Source/WebCore/inspector/front-end/InspectorFrontendHostStub.js', u'Source/WebCore/inspector/front-end/externs.js', u'Source/WebCore/testing/Internals.cpp', u'Source/WebKit/mac/ChangeLog', u'Source/WebKit/mac/Resources/Dock.pdf', u'Source/WebKit/mac/WebCoreSupport/WebInspectorClient.h', u'Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm', u'Source/WebKit/mac/WebInspector/WebInspectorFrontend.mm', u'Source/WebKit2/ChangeLog', u'Source/WebKit2/Resources/Dock.pdf', u'Source/WebKit2/Resources/DockBottom.pdf', u'Source/WebKit2/Resources/DockRight.pdf', u'Source/WebKit2/Shared/WebPreferencesStore.h', u'Source/WebKit2/UIProcess/WebInspectorProxy.cpp', u'Source/WebKit2/UIProcess/WebInspectorProxy.h', u'Source/WebKit2/UIProcess/WebInspectorProxy.messages.in', u'Source/WebKit2/UIProcess/mac/WebInspectorProxyMac.mm', u'Source/WebKit2/WebKit2.xcodeproj/project.pbxproj', u'Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp', u'Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.h', u'Source/WebKit2/WebProcess/WebPage/WebInspector.cpp', u'Source/WebKit2/WebProcess/WebPage/WebInspector.h', u'Source/WebKit2/WebProcess/WebPage/WebInspector.messages.in']" exit_code: 1 Source/WebCore/inspector/InspectorFrontendHost.h:68: The parameter name "width" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 1 in 27 files If any of these errors are false positives, please file a bug against check-webkit-style.
Early Warning System Bot
Comment 4
2013-03-26 14:53:51 PDT
Comment on
attachment 195166
[details]
Proposed Change
Attachment 195166
[details]
did not pass qt-ews (qt): Output:
http://webkit-commit-queue.appspot.com/results/17251643
WebKit Review Bot
Comment 5
2013-03-26 15:00:40 PDT
Comment on
attachment 195166
[details]
Proposed Change
Attachment 195166
[details]
did not pass cr-linux-debug-ews (chromium-xvfb): Output:
http://webkit-commit-queue.appspot.com/results/17296304
WebKit Review Bot
Comment 6
2013-03-26 15:01:28 PDT
Comment on
attachment 195166
[details]
Proposed Change
Attachment 195166
[details]
did not pass chromium-ews (chromium-xvfb): Output:
http://webkit-commit-queue.appspot.com/results/17290734
Early Warning System Bot
Comment 7
2013-03-26 15:06:34 PDT
Comment on
attachment 195166
[details]
Proposed Change
Attachment 195166
[details]
did not pass qt-wk2-ews (qt): Output:
http://webkit-commit-queue.appspot.com/results/17334092
kov's GTK+ EWS bot
Comment 8
2013-03-26 15:08:37 PDT
Comment on
attachment 195166
[details]
Proposed Change
Attachment 195166
[details]
did not pass gtk-ews (gtk): Output:
http://webkit-commit-queue.appspot.com/results/17132987
Peter Beverloo (cr-android ews)
Comment 9
2013-03-26 15:20:29 PDT
Comment on
attachment 195166
[details]
Proposed Change
Attachment 195166
[details]
did not pass cr-android-ews (chromium-android): Output:
http://webkit-commit-queue.appspot.com/results/17246582
Joseph Pecoraro
Comment 10
2013-03-26 15:39:09 PDT
Comment on
attachment 195166
[details]
Proposed Change View in context:
https://bugs.webkit.org/attachment.cgi?id=195166&action=review
Looks good to me assuming other builds get fixed. There will need to be a WK2 Owner to r+ the WK2 parts.
> Source/WebCore/inspector/InspectorFrontendClientLocal.h:108 > + void setAttachedWindow(DockSide);
This will break Qt WK1, which also extends InspectorFrontendClientLocal: WebKit/qt/WebCoreSupport/InspectorClientQt.cpp The changes would be minor.
Timothy Hatcher
Comment 11
2013-03-26 15:47:45 PDT
Created
attachment 195176
[details]
Proposed Change (Take 2) With some build fixes.
WebKit Review Bot
Comment 12
2013-03-26 15:50:28 PDT
Attachment 195176
[details]
did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/WebCore.exp.in', u'Source/WebCore/inspector/InspectorFrontendClient.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorFrontendClientLocal.h', u'Source/WebCore/inspector/InspectorFrontendHost.cpp', u'Source/WebCore/inspector/InspectorFrontendHost.h', u'Source/WebCore/inspector/InspectorFrontendHost.idl', u'Source/WebCore/inspector/front-end/InspectorFrontendHostStub.js', u'Source/WebCore/inspector/front-end/externs.js', u'Source/WebCore/testing/Internals.cpp', u'Source/WebKit/chromium/src/InspectorFrontendClientImpl.cpp', u'Source/WebKit/chromium/src/InspectorFrontendClientImpl.h', u'Source/WebKit/efl/WebCoreSupport/InspectorClientEfl.cpp', u'Source/WebKit/efl/WebCoreSupport/InspectorClientEfl.h', u'Source/WebKit/gtk/WebCoreSupport/InspectorClientGtk.cpp', u'Source/WebKit/gtk/WebCoreSupport/InspectorClientGtk.h', u'Source/WebKit/mac/ChangeLog', u'Source/WebKit/mac/Resources/Dock.pdf', u'Source/WebKit/mac/WebCoreSupport/WebInspectorClient.h', u'Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm', u'Source/WebKit/mac/WebInspector/WebInspectorFrontend.mm', u'Source/WebKit/qt/WebCoreSupport/InspectorClientQt.cpp', u'Source/WebKit/qt/WebCoreSupport/InspectorClientQt.h', u'Source/WebKit2/ChangeLog', u'Source/WebKit2/Resources/Dock.pdf', u'Source/WebKit2/Resources/DockBottom.pdf', u'Source/WebKit2/Resources/DockRight.pdf', u'Source/WebKit2/Shared/WebPreferencesStore.h', u'Source/WebKit2/UIProcess/WebInspectorProxy.cpp', u'Source/WebKit2/UIProcess/WebInspectorProxy.h', u'Source/WebKit2/UIProcess/WebInspectorProxy.messages.in', u'Source/WebKit2/UIProcess/mac/WebInspectorProxyMac.mm', u'Source/WebKit2/WebKit2.xcodeproj/project.pbxproj', u'Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp', u'Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.h', u'Source/WebKit2/WebProcess/WebPage/WebInspector.cpp', u'Source/WebKit2/WebProcess/WebPage/WebInspector.h', u'Source/WebKit2/WebProcess/WebPage/WebInspector.messages.in']" exit_code: 1 Source/WebCore/inspector/InspectorFrontendHost.h:68: The parameter name "width" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 1 in 35 files If any of these errors are false positives, please file a bug against check-webkit-style.
Early Warning System Bot
Comment 13
2013-03-26 16:00:04 PDT
Comment on
attachment 195176
[details]
Proposed Change (Take 2)
Attachment 195176
[details]
did not pass qt-wk2-ews (qt): Output:
http://webkit-commit-queue.appspot.com/results/17329187
Build Bot
Comment 14
2013-03-26 16:12:56 PDT
Comment on
attachment 195176
[details]
Proposed Change (Take 2)
Attachment 195176
[details]
did not pass win-ews (win): Output:
http://webkit-commit-queue.appspot.com/results/17232829
EFL EWS Bot
Comment 15
2013-03-26 16:25:22 PDT
Comment on
attachment 195176
[details]
Proposed Change (Take 2)
Attachment 195176
[details]
did not pass efl-ews (efl): Output:
http://webkit-commit-queue.appspot.com/results/17141970
WebKit Review Bot
Comment 16
2013-03-26 17:07:31 PDT
Comment on
attachment 195176
[details]
Proposed Change (Take 2)
Attachment 195176
[details]
did not pass chromium-ews (chromium-xvfb): Output:
http://webkit-commit-queue.appspot.com/results/17258543
New failing tests: fast/js/dfg-inline-resolve.html
WebKit Review Bot
Comment 17
2013-03-26 17:07:40 PDT
Created
attachment 195192
[details]
Archive of layout-test-results from gce-cr-linux-03 for chromium-linux-x86_64 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: gce-cr-linux-03 Port: chromium-linux-x86_64 Platform: Linux-3.3.8-gcg-201212281604-x86_64-with-GCEL-10.04-gcel_10.04
Timothy Hatcher
Comment 18
2013-03-27 06:34:55 PDT
Created
attachment 195302
[details]
Proposed Change (Take 3)
WebKit Review Bot
Comment 19
2013-03-27 06:38:48 PDT
Attachment 195302
[details]
did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/WebCore.exp.in', u'Source/WebCore/inspector/InspectorFrontendClient.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorFrontendClientLocal.h', u'Source/WebCore/inspector/InspectorFrontendHost.cpp', u'Source/WebCore/inspector/InspectorFrontendHost.h', u'Source/WebCore/inspector/InspectorFrontendHost.idl', u'Source/WebCore/inspector/front-end/InspectorFrontendHostStub.js', u'Source/WebCore/inspector/front-end/externs.js', u'Source/WebCore/testing/Internals.cpp', u'Source/WebKit/chromium/src/InspectorFrontendClientImpl.cpp', u'Source/WebKit/chromium/src/InspectorFrontendClientImpl.h', u'Source/WebKit/efl/WebCoreSupport/InspectorClientEfl.cpp', u'Source/WebKit/efl/WebCoreSupport/InspectorClientEfl.h', u'Source/WebKit/gtk/WebCoreSupport/InspectorClientGtk.cpp', u'Source/WebKit/gtk/WebCoreSupport/InspectorClientGtk.h', u'Source/WebKit/mac/ChangeLog', u'Source/WebKit/mac/Resources/Dock.pdf', u'Source/WebKit/mac/WebCoreSupport/WebInspectorClient.h', u'Source/WebKit/mac/WebCoreSupport/WebInspectorClient.mm', u'Source/WebKit/mac/WebInspector/WebInspectorFrontend.mm', u'Source/WebKit/qt/WebCoreSupport/InspectorClientQt.cpp', u'Source/WebKit/qt/WebCoreSupport/InspectorClientQt.h', u'Source/WebKit/win/WebCoreSupport/WebInspectorClient.cpp', u'Source/WebKit/win/WebCoreSupport/WebInspectorClient.h', u'Source/WebKit2/ChangeLog', u'Source/WebKit2/Resources/Dock.pdf', u'Source/WebKit2/Resources/DockBottom.pdf', u'Source/WebKit2/Resources/DockRight.pdf', u'Source/WebKit2/Shared/WebPreferencesStore.h', u'Source/WebKit2/UIProcess/WebInspectorProxy.cpp', u'Source/WebKit2/UIProcess/WebInspectorProxy.h', u'Source/WebKit2/UIProcess/WebInspectorProxy.messages.in', u'Source/WebKit2/UIProcess/mac/WebInspectorProxyMac.mm', u'Source/WebKit2/UIProcess/qt/WebInspectorProxyQt.cpp', u'Source/WebKit2/WebKit2.xcodeproj/project.pbxproj', u'Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp', u'Source/WebKit2/WebProcess/WebCoreSupport/WebInspectorFrontendClient.h', u'Source/WebKit2/WebProcess/WebPage/WebInspector.cpp', u'Source/WebKit2/WebProcess/WebPage/WebInspector.h', u'Source/WebKit2/WebProcess/WebPage/WebInspector.messages.in']" exit_code: 1 Source/WebKit/win/WebCoreSupport/WebInspectorClient.cpp:47: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 1 in 38 files If any of these errors are false positives, please file a bug against check-webkit-style.
EFL EWS Bot
Comment 20
2013-03-27 06:48:32 PDT
Comment on
attachment 195302
[details]
Proposed Change (Take 3)
Attachment 195302
[details]
did not pass efl-ews (efl): Output:
http://webkit-commit-queue.appspot.com/results/17288664
kov's GTK+ EWS bot
Comment 21
2013-03-27 06:51:20 PDT
Comment on
attachment 195302
[details]
Proposed Change (Take 3)
Attachment 195302
[details]
did not pass gtk-ews (gtk): Output:
http://webkit-commit-queue.appspot.com/results/17327165
Timothy Hatcher
Comment 22
2013-03-27 07:35:59 PDT
Created
attachment 195319
[details]
Proposed Change (Take 4)
kov's GTK+ EWS bot
Comment 23
2013-03-27 08:07:56 PDT
Comment on
attachment 195319
[details]
Proposed Change (Take 4)
Attachment 195319
[details]
did not pass gtk-ews (gtk): Output:
http://webkit-commit-queue.appspot.com/results/17340034
Timothy Hatcher
Comment 24
2013-03-27 08:25:21 PDT
Can someone from GTK and EFL ports explain the build failures? I'm exporting __ZN7WebCore28InspectorFrontendClientLocal25changeAttachedWindowWidthEj and it works fine on other ports.
Timothy Hatcher
Comment 25
2013-03-27 09:28:37 PDT
Looks like GTK is the only issue now. Windows TBD.
Build Bot
Comment 26
2013-03-27 10:34:04 PDT
Comment on
attachment 195319
[details]
Proposed Change (Take 4)
Attachment 195319
[details]
did not pass win-ews (win): Output:
http://webkit-commit-queue.appspot.com/results/17253575
Timothy Hatcher
Comment 27
2013-03-27 10:57:44 PDT
Created
attachment 195356
[details]
Proposed Change (Take 5) Win build fix. Still need advice on GTK's build failure.
Zan Dobersek
Comment 28
2013-03-27 12:02:44 PDT
The GTK build is failing due to the undefined InspectorFrontendClientLocal::changeAttachedWindowWidth, to fix that you should list the related symbol (__ZN7WebCore28InspectorFrontendClientLocal25changeAttachedWindowWidthEj) in Source/autotools/symbols.filter. Thanks for not breaking the GTK build!
Timothy Hatcher
Comment 29
2013-03-27 12:13:05 PDT
Created
attachment 195375
[details]
Proposed Change (Take 6)
Anders Carlsson
Comment 30
2013-03-27 13:15:19 PDT
Comment on
attachment 195375
[details]
Proposed Change (Take 6) WebKit 2 parts look good.
Joseph Pecoraro
Comment 31
2013-03-27 13:17:40 PDT
Comment on
attachment 195375
[details]
Proposed Change (Take 6) r=me, looks like you might be missing a few ChangeLog entires for the recent port build fixes.
Ryosuke Niwa
Comment 32
2013-03-27 14:04:55 PDT
Landed in
http://trac.webkit.org/changeset/147004
. Build fixes landed in
http://trac.webkit.org/changeset/147005
,
http://trac.webkit.org/changeset/147007
, and
http://trac.webkit.org/changeset/147009
.
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