Bug 173903 – Web Replay: remove some unused code

• Component: Web Inspector
• Status: RESOLVED FIXED

[Blaze Burg]

.

[Blaze Burg]

Created attachment 313978 [details]
Patch

[Build Bot]

Attachment 313978 [details] did not pass style-queue:


ERROR: Source/WebCore/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebCore/PAL/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebKit/mac/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/JavaScriptCore/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebKit2/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
Total errors found: 5 in 21 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Joseph Pecoraro]

Comment on attachment 313978 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=313978&action=review

rs=me

> Source/JavaScriptCore/ChangeLog:9
> +        * Configurations/FeatureDefines.xcconfig:

This misses:
TestWebKitAPI/Configurations/FeatureDefines.xcconfig

> Source/WebCore/PlatformMac.cmake:-91
> -    "${WEBCORE_DIR}/ForwardingHeaders/replay"

You should be removing this directory as well. it contains:

./Source/WebCore/ForwardingHeaders/replay
./Source/WebCore/ForwardingHeaders/replay/EncodedValue.h
./Source/WebCore/ForwardingHeaders/replay/NondeterministicInput.h
./Source/WebCore/ForwardingHeaders/replay/InputCursor.h
./Source/WebCore/ForwardingHeaders/replay/EmptyInputCursor.h

> LayoutTests/ChangeLog:13
> +        * inspector/replay/window-navigator-plugins-memoized.html: Removed.

TestExpectations:

$ find-expectations 'inspector/replay'
LayoutTests/TestExpectations:184:webkit.org/b/148036 http/tests/inspector/replay/ [ Skip ]
LayoutTests/TestExpectations:201:webkit.org/b/137130 inspector/replay [ Skip ]
LayoutTests/TestExpectations:210:webkit.org/b/131318 http/tests/inspector/replay/document-last-modified-fallback-value.html [ Skip ]

Another directory of tests:
http/tests/inspector/replay

[Build Bot]

Comment on attachment 313978 [details]
Patch

Attachment 313978 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4011187

New failing tests:
fast/frames/iframe-window-focus.html
fast/scrolling/scroll-animator-select-list-events.html
fast/events/keyevent-iframe-removed-crash.html
fast/scrolling/scroll-animator-basic-events.html
fast/scrolling/scroll-animator-overlay-scrollbars-clicked.html
fast/events/keydown-1.html
fast/events/keypress-focus-change.html
editing/input/option-page-up-down.html
fast/events/recorded-keydown-event.html
pageoverlay/overlay-small-frame-mouse-events.html
editing/input/scroll-viewport-page-up-down.html
fast/scrolling/scroll-animator-overlay-scrollbars-hovered.html
fast/dom/access-key-iframe.html

[Build Bot]

Created attachment 313986 [details]
Archive of layout-test-results from ews106 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6

[Blaze Burg]

Created attachment 314146 [details]
Patch

[Build Bot]

Comment on attachment 314146 [details]
Patch

Attachment 314146 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4020897

New failing tests:
pageoverlay/overlay-small-frame-mouse-events.html
fast/scrolling/scroll-animator-overlay-scrollbars-clicked.html
fast/scrolling/scroll-animator-basic-events.html
fast/scrolling/scroll-animator-select-list-events.html
fast/scrolling/scroll-animator-overlay-scrollbars-hovered.html

[Build Bot]

Created attachment 314155 [details]
Archive of layout-test-results from ews106 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6

[Blaze Burg]

Created attachment 314178 [details]
Patch

[Joseph Pecoraro]

Bots are failing to build:

> /usr/lib/ccache/c++ Source/WebCore/page/Page.cpp
> /home/ews/ltilve-gtk-wk2-ews/WebKit/Source/WebCore/page/Page.cpp:105:29: fatal error: UserInputBridge.h: No such file or directory
>  #include "UserInputBridge.h"
>                              ^
> compilation terminated.

[Blaze Burg]

Created attachment 314272 [details]
Patch

[Joseph Pecoraro]

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:2399:72: error: 'class WebCore::EventHandler' has no member named 'handleKeyEvent'; did you mean 'handleWheelEvent'?
>      return page->focusController().focusedOrMainFrame().eventHandler().handleKeyEvent(platform(keyboardEvent));
>                                                                        ^~~~~~~~~~~~~~

[Blaze Burg]

Created attachment 314306 [details]
Patch

[Blaze Burg]

Created attachment 314533 [details]
Patch

[Blaze Burg]

Created attachment 314538 [details]
Patch

[Build Bot]

Attachment 314538 [details] did not pass style-queue:


ERROR: Source/WebCore/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebCore/PAL/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebKit/mac/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/JavaScriptCore/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Source/WebKit2/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
Total errors found: 5 in 22 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Build Bot]

Comment on attachment 314538 [details]
Patch

Attachment 314538 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4047777

New failing tests:
pageoverlay/overlay-small-frame-mouse-events.html
fast/scrolling/scroll-animator-overlay-scrollbars-clicked.html
fast/scrolling/scroll-animator-basic-events.html
fast/scrolling/scroll-animator-select-list-events.html
fast/scrolling/scroll-animator-overlay-scrollbars-hovered.html

[Build Bot]

Created attachment 314543 [details]
Archive of layout-test-results from ews104 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6

[Joseph Pecoraro]

Comment on attachment 314538 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=314538&action=review

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:2311
> -                return page->corePage()->userInputBridge().handleMouseMoveOnScrollbarEvent(platformMouseEvent);
> -            return page->corePage()->userInputBridge().handleMouseMoveEvent(platformMouseEvent);
> +                return frame.eventHandler().passMouseMovedEventToScrollbars(platformMouseEvent);
> +            return frame.eventHandler().handleMouseMoveEvent(platformMouseEvent);

Interesting that these are using `frame`. It looks like UserInputBridge would have used the equivalent of `page->mainFrame()` for some of these. Its possible that using the specific frame is correct, in which case maybe these few failing tests need to be updated.

[Blaze Burg]

Created attachment 314853 [details]
For Landing

[Build Bot]

Attachment 314853 [details] did not pass style-queue:


ERROR: Source/WebCore/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebCore/PAL/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebKit/mac/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/JavaScriptCore/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Source/WebKit2/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
Total errors found: 5 in 21 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Blaze Burg]

Created attachment 314866 [details]
For Landing

[Build Bot]

Attachment 314866 [details] did not pass style-queue:


ERROR: Source/WebCore/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebCore/PAL/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/WebKit/mac/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Source/JavaScriptCore/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
ERROR: Tools/TestWebKitAPI/Configurations/FeatureDefines.xcconfig:0:  Any changes made to FeatureDefines should be made to all of them (changed file does not match Source/WebKit2/Configurations/FeatureDefines.xcconfig). Use sync-feature-defines if possible.  [featuredefines/equality] [5]
Total errors found: 5 in 21 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Joseph Pecoraro]

Comment on attachment 314866 [details]
For Landing

rs=me.

The style bot is complaining about mismatched xcconfigs, so we should resolve that.

[Blaze Burg]

(In reply to Joseph Pecoraro from comment #24)
> Comment on attachment 314866 [details]
> For Landing
> 
> rs=me.
> 
> The style bot is complaining about mismatched xcconfigs, so we should
> resolve that.

ENABLE_WEB_REPLAY never got into that feature defines file, so it's confused.

[Blaze Burg]

Committed r219301: <http://trac.webkit.org/changeset/219301>