Go to back/forward list items after a process-swapped navigation
<rdar://problem/38690544>
Created attachment 336388 [details] Patch
Attachment 336388 [details] did not pass style-queue: ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:121: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:130: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:132: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:135: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:141: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] Total errors found: 5 in 38 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 336394 [details] Patch
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Attachment 336394 [details] did not pass style-queue: ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:121: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:130: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:132: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:135: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:141: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] Total errors found: 5 in 39 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 336405 [details] Patch
Attachment 336405 [details] did not pass style-queue: ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:121: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:130: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:132: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:135: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:141: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] Total errors found: 5 in 40 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 336421 [details] Patch
Attachment 336421 [details] did not pass style-queue: ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:121: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:130: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:132: More than one command on the same line [whitespace/newline] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:135: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:141: Multi-line string ("...") found. This lint script doesn't do well with such strings, and may give bogus warnings. They're ugly and unnecessary, and you should use concatenation instead". [readability/multiline_string] [5] Total errors found: 5 in 42 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 336421 [details] Patch Clearing flags on attachment: 336421 Committed r229926: <https://trac.webkit.org/changeset/229926>
All reviewed patches have been landed. Closing bug.
This change broke the Windows build: https://build.webkit.org/builders/Apple%20Win%20Release%20%28Build%29/builds/8448 C:\cygwin\home\buildbot\slave\win-release\build\Source\WebKitLegacy\win\WebView.cpp(3436): error C2027: use of undefined type 'WebCore::NavigationPolicyCheck' [C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\DerivedSources\ForwardingHeaders\WebCore/HistoryController.h(43): note: see declaration of 'WebCore::NavigationPolicyCheck' C:\cygwin\home\buildbot\slave\win-release\build\Source\WebKitLegacy\win\WebView.cpp(3436): error C2065: 'Require': undeclared identifier [C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] C:\cygwin\home\buildbot\slave\win-release\build\Source\WebKitLegacy\win\WebView.cpp(5963): error C2027: use of undefined type 'WebCore::NavigationPolicyCheck' [C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\DerivedSources\ForwardingHeaders\WebCore/HistoryController.h(43): note: see declaration of 'WebCore::NavigationPolicyCheck' C:\cygwin\home\buildbot\slave\win-release\build\Source\WebKitLegacy\win\WebView.cpp(5963): error C2065: 'Require': undeclared identifier [C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj]
(In reply to Ryan Haddad from comment #13) > This change broke the Windows build: > https://build.webkit.org/builders/Apple%20Win%20Release%20%28Build%29/builds/ > 8448 > > C:\cygwin\home\buildbot\slave\win- > release\build\Source\WebKitLegacy\win\WebView.cpp(3436): error C2027: use of > undefined type 'WebCore::NavigationPolicyCheck' > [C:\cygwin\home\buildbot\slave\win- > release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] > > C:\cygwin\home\buildbot\slave\win- > release\build\WebKitBuild\Release\DerivedSources\ForwardingHeaders\WebCore/ > HistoryController.h(43): note: see declaration of > 'WebCore::NavigationPolicyCheck' > C:\cygwin\home\buildbot\slave\win- > release\build\Source\WebKitLegacy\win\WebView.cpp(3436): error C2065: > 'Require': undeclared identifier > [C:\cygwin\home\buildbot\slave\win- > release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] > C:\cygwin\home\buildbot\slave\win- > release\build\Source\WebKitLegacy\win\WebView.cpp(5963): error C2027: use of > undefined type 'WebCore::NavigationPolicyCheck' > [C:\cygwin\home\buildbot\slave\win- > release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] > > C:\cygwin\home\buildbot\slave\win- > release\build\WebKitBuild\Release\DerivedSources\ForwardingHeaders\WebCore/ > HistoryController.h(43): note: see declaration of > 'WebCore::NavigationPolicyCheck' > C:\cygwin\home\buildbot\slave\win- > release\build\Source\WebKitLegacy\win\WebView.cpp(5963): error C2065: > 'Require': undeclared identifier > [C:\cygwin\home\buildbot\slave\win- > release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] Follow-up Windows build fix: <https://trac.webkit.org/changeset/229934>
(In reply to Chris Dumez from comment #14) > (In reply to Ryan Haddad from comment #13) > > This change broke the Windows build: > > https://build.webkit.org/builders/Apple%20Win%20Release%20%28Build%29/builds/ > > 8448 > > > > C:\cygwin\home\buildbot\slave\win- > > release\build\Source\WebKitLegacy\win\WebView.cpp(3436): error C2027: use of > > undefined type 'WebCore::NavigationPolicyCheck' > > [C:\cygwin\home\buildbot\slave\win- > > release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] > > > > C:\cygwin\home\buildbot\slave\win- > > release\build\WebKitBuild\Release\DerivedSources\ForwardingHeaders\WebCore/ > > HistoryController.h(43): note: see declaration of > > 'WebCore::NavigationPolicyCheck' > > C:\cygwin\home\buildbot\slave\win- > > release\build\Source\WebKitLegacy\win\WebView.cpp(3436): error C2065: > > 'Require': undeclared identifier > > [C:\cygwin\home\buildbot\slave\win- > > release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] > > C:\cygwin\home\buildbot\slave\win- > > release\build\Source\WebKitLegacy\win\WebView.cpp(5963): error C2027: use of > > undefined type 'WebCore::NavigationPolicyCheck' > > [C:\cygwin\home\buildbot\slave\win- > > release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] > > > > C:\cygwin\home\buildbot\slave\win- > > release\build\WebKitBuild\Release\DerivedSources\ForwardingHeaders\WebCore/ > > HistoryController.h(43): note: see declaration of > > 'WebCore::NavigationPolicyCheck' > > C:\cygwin\home\buildbot\slave\win- > > release\build\Source\WebKitLegacy\win\WebView.cpp(5963): error C2065: > > 'Require': undeclared identifier > > [C:\cygwin\home\buildbot\slave\win- > > release\build\WebKitBuild\Release\Source\WebKitLegacy\WebKitLegacy.vcxproj] > > Follow-up Windows build fix: > <https://trac.webkit.org/changeset/229934> Thank you.