Bug 72596 – [Qt] Layer violation: WebCore::dnsPrefetch uses QWebSettings::globalSettings()

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Simon Hausmann]

This is trivial to remove, the callers already check WebCore::Settings

[Simon Hausmann]

Created attachment 115560 [details]
[Qt] Layer violation: WebCore::dnsPrefetch uses QWebSettings::globalSettings()

[WebKit Review Bot]

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

Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9

Last 3072 characters of output:
M	Source/WebKit2/UIProcess/API/qt/qwebdownloaditem_p.h
	M	Source/WebKit2/UIProcess/API/qt/qquickwebview_p.h
	A	Source/WebKit2/UIProcess/API/qt/qquickwebpage_p_p.h
	M	Source/WebKit2/UIProcess/API/qt/qquickwebpage_p.h
	M	Tools/qmake/mkspecs/features/default_post.prf
	M	Tools/qmake/mkspecs/features/default_pre.prf
	M	Tools/qmake/mkspecs/features/qtwebkit.prf
	A	Tools/qmake/mkspecs/features/qtwebkit-private.prf
	M	Tools/WebKitTestRunner/qt/PlatformWebViewQt.cpp
	M	Tools/ChangeLog
	M	Tools/MiniBrowser/qt/BrowserWindow.h
	M	Tools/MiniBrowser/qt/UrlLoader.cpp
	M	Tools/MiniBrowser/qt/BrowserWindow.cpp
W: -empty_dir: trunk/Source/WebKit/qt/declarative/private
W: -empty_dir: trunk/Source/WebKit2/UIProcess/API/qt/qquickwebpage.h
W: -empty_dir: trunk/Source/WebKit2/UIProcess/API/qt/qquickwebview.h
W: -empty_dir: trunk/Source/WebKit2/UIProcess/API/qt/qquickwebviewprivateextension.cpp
W: -empty_dir: trunk/Source/WebKit2/UIProcess/API/qt/qquickwebviewprivateextension_p.h
W: -empty_dir: trunk/Source/WebKit2/UIProcess/API/qt/qwebdownloaditem.h
W: -empty_dir: trunk/Source/WebKit2/UIProcess/API/qt/qwebpreferences.h
r100603 = 5b10400dce635df634c34ef2967dba3a593ef12f (refs/remotes/origin/master)
First, rewinding head to replay your work on top of it...
Applying: [Qt] Move WebKit2 C++ APIs to private API and build QML extension plugin on top of that
Using index info to reconstruct a base tree...
<stdin>:222: new blank line at EOF.
+
warning: 1 line adds whitespace errors.
Falling back to patching base and 3-way merge...
warning: too many files (created: 165678 deleted: 9), skipping inexact rename detection
CONFLICT (rename/delete): Rename Source/WebKit/qt/declarative/private/plugin.cpp->Source/WebKit2/UIProcess/API/qt/qwebdownloaditem_p_p.h in [Qt] Move WebKit2 C++ APIs to private API and build QML extension plugin on top of that and deleted in HEAD
CONFLICT (rename/delete): Rename Source/WebKit/qt/declarative/private/private.pro->Source/WebKit/qt/declarative/public.pri in [Qt] Move WebKit2 C++ APIs to private API and build QML extension plugin on top of that and deleted in HEAD
CONFLICT (rename/delete): Rename Source/WebKit2/UIProcess/API/qt/qquickwebviewprivateextension_p.h->Source/WebKit2/UIProcess/API/qt/qquickwebpage_p_p.h in [Qt] Move WebKit2 C++ APIs to private API and build QML extension plugin on top of that and deleted in HEAD
Auto-merging Source/WebKit2/ChangeLog
CONFLICT (content): Merge conflict in Source/WebKit2/ChangeLog
Auto-merging Source/WebKit2/UIProcess/API/qt/tests/qmltests/qmltests.pro
Auto-merging Source/WebKit2/UIProcess/API/qt/tests/qmltests/tst_qmltests.cpp
Failed to merge in the changes.
Patch failed at 0001 [Qt] Move WebKit2 C++ APIs to private API and build QML extension plugin on top of that

When you have resolved this problem run "git rebase --continue".
If you would prefer to skip this patch, instead run "git rebase --skip".
To restore the original branch and stop rebasing run "git rebase --abort".

rebase refs/remotes/origin/master: command returned error: 1

Died at Tools/Scripts/update-webkit line 157.


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

[Simon Hausmann]

Created attachment 115562 [details]
[Qt] Layer violation: WebCore::dnsPrefetch uses QWebSettings::globalSettings()

[Simon Hausmann]

Comment on attachment 115562 [details]
[Qt] Layer violation: WebCore::dnsPrefetch uses QWebSettings::globalSettings()

Clearing flags on attachment: 115562

Committed r100614: <http://trac.webkit.org/changeset/100614>

[Simon Hausmann]

All reviewed patches have been landed.  Closing bug.