Bug 71887 – [Qt] Enable geolocation by default

• Component: WebKit Qt
• Status: RESOLVED INVALID

[Kenneth Rohde Christiansen]

SSIA

[Kenneth Rohde Christiansen]

Created attachment 114240 [details]
Patch

[WebKit Review Bot]

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

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Tools/ChangeLog', u'Tools/Scripts/build-we..." exit_code: 1

Tools/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Total errors found: 1 in 2 files


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

[Simon Hausmann]

Comment on attachment 114240 [details]
Patch

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

I believe enabling this by default is going to break the build for setups where the necessary dependencies are not installed, because the build system does not automatically disable the feature again. The existing logic only covers enabling it, if the corresponding qt module is found. It doesn't cover disabling it if it's not found.

>> Tools/ChangeLog:1

> 
> ChangeLog entry has no bug number  [changelog/bugnumber] [5]

Style bot is correct here: There _is_ a bug, but it's not mentioned in ChangeLog :)

[Jocelyn Turcotte]

=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.