summaryrefslogtreecommitdiffstats
path: root/src/core
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Use .NOTPARALLEL on Windows tooAllan Sandfeld Jensen2017-03-032-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | With Jom 1.1 supporting the special target we can use it to also avoid compiling and linking debug and release in parallel on Windows too. Change-Id: Ie3307f1f40fd877d04f9fea53c02fe7a37c48824 Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
* | | Re-enable v8 snapshots for cross buildsAllan Sandfeld Jensen2017-03-031-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | They were not working due to a typo causing v8_current_cpu to be set wrong in configure_host.pro, that has been fixed in the meantime. Change-Id: I18ca533806d65fb9bf6e6c26c0cb01d97ed124dd Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | | Fix browser process crash when IBus is enabledPeter Varga2017-03-031-0/+8
| | | | | | | | | | | | | | | Change-Id: Ic48b34473c808bcfdc80acdccab4883a8e9d5715 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Ensure the existence of the browser context's storage directoryJoerg Bornemann2017-03-021-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For on-disk profiles, the browser context's storage directory must exist before calling VisitedLinkMaster::Init(). This fixes the error message [2844:2877:0213/183645:ERROR:visitedlink_master.cc(73)] Failed to open file /home/username/.local/share/appname/QtWebEngine/Default/Visited Links Before, the directory was implicitly created by chromium's disk cache backend initialization code which runs later than VisitedLinkMaster::Init(). Change-Id: I3860a6874b8002292020b9ebaec22f8343ebd258 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | | Fix cross-compiled desktop linux buildsAllan Sandfeld Jensen2017-03-022-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | With embedded-builds now being a configure option we can have cross compiled desktop builds, and should support them. Change-Id: I43d9f26dc13bc1709c24d1420868e3c8bbbc458f Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | | Fix linux-clang buildsAllan Sandfeld Jensen2017-03-023-2/+13
| | | | | | | | | | | | | | | | | | | | | Pass through the right GN args to make clang builds work. Change-Id: I99d7f0caaed8a8c594a8b40f18207f2f95fe03b1 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | | Make xscrnsaver optionalAllan Sandfeld Jensen2017-03-021-0/+1
|/ / | | | | | | | | | | | | | | Allow building on machines where xscrnsaver is not present, but turning of the corresponding features. Change-Id: Iabf7063fcc25f68f5a9fefc95a1caf164da49fed Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Don't use_external_popup_menu on macOS so that popups actually appearAlexandru Croitor2017-03-011-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | Using the use_external_popup_menu option forces Chromium to compile a different code path for displaying popup menus on macOS, which is not implemented by the Qt overrides, and thus no popups appear. Thus that option should be disabled, and instead the old code path should be used, just like in previous versions of Qt. Change-Id: Ic0f0dca2e097aac2c01e729d7a91425ce3f52674 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Fix build when Qt Location is not presentAlexandru Croitor2017-03-011-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | In case if Qt Location is not present, the compiler tries to instantiate a unique_ptr from a forward declared class, which fails (specifically content::LocationProvider). Fix is to include the header which contains the LocationProvider class. Change-Id: I4a398bc121883923257bb358ad7c76fc372bf8bb Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Support cross-builds and alternative compilers with GNAllan Sandfeld Jensen2017-03-013-29/+57
| | | | | | | | | | | | | | | | Adds GN versions of the gyp_configure targets, to create custom toolchains we can use to override compilers used by GN. Change-Id: Ic3a9cb7e83f845180396faff632f72f9c24f3365 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | GeoLocation adaptations for Chromium 55Allan Sandfeld Jensen2017-03-014-67/+53
| | | | | | | | | | | | | | Restructures geolocation classes to fit new code Change-Id: Ic370bd4cef4ba3f7c98931761d180fccd0b82cb7 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Add beginFrameProviderAllan Sandfeld Jensen2017-03-012-5/+59
| | | | | | | | | | | | | | | | We now need to tick/sync frames ourselves. This provides a quick way to do that without touching too much code. Change-Id: I3820d7ef8dbc8e3a224a48ee28c489bb830bb1da Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Adapt to changes in selection API in Chromium 55Allan Sandfeld Jensen2017-03-013-37/+16
| | | | | | | | | | | | | | Aura now uses the TextInputManager for tracking selections. Change-Id: I86634b987b0184a90d3d7125e568fcdab0b2e930 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Adapt devtools to Chromium 55Allan Sandfeld Jensen2017-03-015-155/+84
| | | | | | | | | | | | | | A restructuring of all our devtools integration classes. Change-Id: I2a32b9b2e66c17b72aaa3bbf246a22d211409aa5 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Simple adaptations to Chromium 55Allan Sandfeld Jensen2017-03-0152-228/+347
| | | | | | | | | | | | | | The simplest adaptations to API and build changes in Chromium 55 Change-Id: I923fa188690a04902492317807f72f006bcab9c6 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Fix crashes in removeFromCanvasMichael Brüning2017-02-151-7/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | After reparenting the view, the compositor data may still be the same, but the node might be a different one. Check if the list of scene graph nodes is empty when making the decision on whether to build a new node tree or not. Also advance the node iterator for missing render passes. Task-number: QTBUG-58472 Change-Id: I28dfb40bda89470c7cdaec0aef8a976e307f5b6a Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Fix release builds for debug and release buildsMichal Klocek2017-02-152-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | Small typo makes debug builds make release. Fix core lib and covert tool compilation for debug builds. Updates chromium: b0bb0288 Fix debug builds compilation Change-Id: Id478ab5e2a2a1b3d5716002f76f252451f69f5ac Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Disable debug symbols in webcore and v8baseAllan Sandfeld Jensen2017-02-141-0/+2
| | | | | | | | | | | | | | | | Disable the largest sources of debug symbols like we did with GYP builds. Change-Id: I23f6c88a7929eaabfa477896a332a229872a83f5 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Handle reduce_binary_size for GNv5.9.0-alpha1Allan Sandfeld Jensen2017-02-131-0/+3
| | | | | | | | | | | | | | Switches to -Os optimizations. Change-Id: Ia40380dce5f908d64c5d411687d42d13728c85cd Reviewed-by: Michael Brüning <michael.bruning@qt.io>
* | Fix rsp linking on windowsMichal Klocek2017-02-101-9/+8
| | | | | | | | | | | | | | | | | | Rsp file format is not correct currently on windows, as a workaround use selfmade rsp file. Task-number: QTBUG-58710 Change-Id: I0cd6f3f5e29285330b1abbb30177bbd56f064e0d Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Merge remote-tracking branch 'origin/5.8' into 5.9Allan Sandfeld Jensen2017-02-083-6/+28
|\| | | | | | | Change-Id: I0c3654ad097809fe10bcc6a15827a0af3006c4e6
| * Do not create directories for overridden pathsJoerg Bornemann2017-02-071-4/+18
| | | | | | | | | | | | | | | | | | When overriding Chromium's default paths do not create non-existent directories anymore. Just print a diagnostic message. Task-number: QTBUG-54258 Change-Id: Ie642a630ad5b4aebc4e80e3eace7db7489a82907 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * Provisional Fix for Navigation-Breakage on right-click in Flash appletViktor Engelmann2017-02-021-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When one right-clicks into a flash applet, chromium believes that a context menu was open (even though PepperRendererHostFactoryQt::CreateResourceHost returns NULL, which only causes an ASSERT on chromiums side). Because of this, the navigation is disabled, although no context menu is displayed. This fix just sends a message to chromium, telling it that "the context menu" is now "closed", which causes another ASSERT, but re-enables the navigation. Task-number: QTBUG-57924 Change-Id: Id6d8150a7d4944162bab81362153f8621d35bb4c Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * Avoid crash in desktop linux with --disable-gpuAllan Sandfeld Jensen2017-02-011-1/+1
| | | | | | | | | | | | | | Don't try to access null OpenGL contexts. Change-Id: I7d0e607e8d29d3cfba8d6e98cd142b951c960281 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Fix running qmake recursivelyMichal Klocek2017-02-071-1/+8
| | | | | | | | | | | | | | Move gn run call form make step to qmake step. Change-Id: I01913aef9dff505df3319755ae7009144919accf Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Refactor release and debug handlingMichal Klocek2017-02-063-23/+8
| | | | | | | | | | | | | | | | | | | | Fix release and debug builds. Refactor ninja and gn builds as a sub projects, these are only build as release. Call core_generator and gn_run separately for debug and release builds. Change-Id: I638023214cdc98a5101d190486452cc4a1229486 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Fix escaping of double-quotesAllan Sandfeld Jensen2017-02-031-1/+1
| | | | | | | | | | | | | | | | We are using double-quoted strings and should therefore be escaping double-quotation marks and not single-quotation ones. Change-Id: I0ae3bb45365843647b38c88a22379f89843dff54 Reviewed-by: Michal Klocek <michal.klocek@qt.io>
* | Switch linux desktop builds to gnMichal Klocek2017-02-021-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates chromium: 1fd9f11 Remove linking with libatomic fa90180 Fix missing dependency on libffi b5a3338 Fix broken unit test for gn 70ea2bc Fix gn compilation for rhel6.6, rhel7.2 f408e89 Improve .pri output 492b20a Remove gtk2 dependency in tests 42e3484 Allow Service Workers without HTTP headers 7a1be30 Do not require gnome-keyring 11e5c2d Revert "Add mus and catapult project files" 891fb3a Remove various test and telemetry dependencies ddd6053 Do not use last_commit_position for gn build e85719e Windows and macOS GN integration 8fda2ae Make more features configurable 93db377 [Backport] Don't schedule wholeSubtreeInvalid sets [ChangeLog] Switch linux desktop builds to gn Change-Id: I0f766b9ee9ea65a2f92d9818ea5fb34a2e65c52d Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Merge dev into 5.9Oswald Buddenhagen2017-02-0124-131/+684
|\ \ | | | | | | | | | Change-Id: Id919bfd67d9d435fd5a05a892151a12ae5bcd3c0
| * | Assert WindowOpenDisposition enum valuesAllan Sandfeld Jensen2017-02-011-0/+11
| | | | | | | | | | | | | | | | | | | | | We are depending on them matching but were not asserting it. Change-Id: I86e9a9a2f516e8a149d82077884bf08b4d8aee51 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * | Make WebRTC support configurableAllan Sandfeld Jensen2017-01-312-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | WebRTC is a big feature that Chromium has left configurable for a long time, we can forward that configurability to our users to allow them control it directly instead of just enabling it on desktop and disabling it on embedded. Change-Id: I4445e2189112205ddf1368bd5e197bab2df92092 Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * | Fix build with 5.8 and make embedded configurableAllan Sandfeld Jensen2017-01-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid checking features only available in 5.9 directly. This also changes the embedded-build configuration to being a feature that can be set at configure time. Change-Id: I9b42dcf696734a83e5c8d6e1cc07562cbeb3f993 Reviewed-by: Michael Brüning <michael.bruning@qt.io> Reviewed-by: Michal Klocek <michal.klocek@qt.io>
| * | Merge remote-tracking branch 'origin/5.8' into devLiang Qi2017-01-305-37/+101
| |\| | | | | | | | | | Change-Id: Ic36a80502730d404ab96dfa11ff1eb0ac6eb302d
| | * Forward mouse buttons and keyboard modifiers while draggingJoerg Bornemann2017-01-261-1/+27
| | | | | | | | | | | | | | | | | | | | | | | | Until now we always pretended that the left mouse button was used and no modifier key was pressed. Change-Id: I0f4b84ed87e7c94289c14c13a6cd12c3beda671a Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Fix canceling drag and drop actionsJoerg Bornemann2017-01-262-15/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RenderViewHost::DragSourceEndedAt must be called after dropping and after canceling a drag. Otherwise scripts running on the web page do not get properly notified. Task-number: QTBUG-58037 Change-Id: Ia245131c59f03f3331158bea6cc2f53bce50f989 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | * Fix confirming IME composition and newline insertionPeter Varga2017-01-253-21/+48
| | | | | | | | | | | | | | | | | | Task-number: QTBUG-55766 Change-Id: I3be3b224dfef9ecc1ce39f4f20ec5c1d13c0fa69 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | macOS GN integrationAlexandru Croitor2017-01-278-76/+128
| | | | | | | | | | | | | | | Change-Id: I89850d43c8f11ec54b3a47318ef0b3f083ae3dee Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | GN configuration cleanup and preparationAllan Sandfeld Jensen2017-01-263-14/+49
| | | | | | | | | | | | | | | | | | | | | | | | Reintroduces the shared linux.pri configuration and moves configuration shared by all architectures to common.pri Change-Id: Iff4d1e6e3d98280223cce0c2c0b74d71ef99df5f Reviewed-by: Michael Brüning <michael.bruning@qt.io>
| * | Merge "Merge remote-tracking branch 'origin/5.8' into dev" into refs/staging/devLiang Qi2017-01-257-21/+32
| |\ \
| | * | Merge remote-tracking branch 'origin/5.8' into devLiang Qi2017-01-257-21/+32
| | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/3rdparty tests/auto/widgets/qwebengineview/tst_qwebengineview.cpp Change-Id: I070173576fc4be53689ce0dd9e1fd4133f5814da
| | | * Fix handling of empty input method eventsPeter Varga2017-01-221-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | Task-number: QTBUG-55766 Change-Id: I4e6ade8f000f66ff1bb28f3b856ae140834292f1 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | | * Make input method hints available via RWHV delegate widgetPeter Varga2017-01-202-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moreover, set ImHiddenText hint for password fields and add back the corresponding widget auto test. Task-number: QTBUG-55766 Change-Id: I3f76e19c8c33e11f3d9f515b6dc7d6e998c3c9a4 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | | * Disable Q_ASSERTs for a release buildKai Koehne2017-01-181-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Qt WebEngine uses the core_gyp_generator.pro and the gyp_generator.prf files to extract defines set by Qt. Anyhow, only one core_generated.gyp file is written for debug and release builds, which hence misses the QT_NO_DEBUG define specific to debug builds. Work around this by explicitly adding back the define. Task-number: QTBUG-58103 Change-Id: I8684aa08417efc93878d70817211e9f66623c78c Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| | | * Fix conversion from blink::WebDragOperation to Qt::DropActionJoerg Bornemann2017-01-184-19/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blink::WebDragOperation is a bit field, even if it is not used in its blink::WebDragOperationsMask incarnation. In particular, WebDragOperationGeneric can be set in addition to other operations. This fixes situations where UpdateDragCursor is called with multiple bits set in its WebDragOperationArgument and the drag action will be spuriously ignored. Also directly pass the WebDragOperation we get from UpdateDragCursor to chromium's API without converting to QDropAction and then converting it back. Task-number: QTBUG-58037 Change-Id: I5c85699de534771f84db69abf7b98e779872a0f7 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| * | | Add gn build of qtwebengine for linuxMichal Klocek2017-01-259-96/+468
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit uses gn instead of gyp to build on desktop linux. Use WEBENGINE_CONFIG+=use_gn to use gn during the build instead of gyp. Change-Id: Ifd3d8d0835b47c323a8d39c320eb55e5e1024dee Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* / / Add SUBDIRS in dependency orderSamuli Piippo2017-01-311-7/+8
|/ / | | | | | | | | | | | | | | | | Once more, fix the order of SUBDIRS, since recursive qmake call does not handle .depends Task-number: QTBUG-45706 Change-Id: Ibe578a9f5e80079d1ba917915f169e0e0bc4203e Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
* | Merge remote-tracking branch 'origin/5.8' into devAllan Sandfeld Jensen2017-01-186-10/+30
|\| | | | | | | Change-Id: If16bfc6f0fbfd0040e13a8a3cbaa113fda10f387
| * QWebEngineDownloadItem::path() should not be percentage encodedAllan Sandfeld Jensen2017-01-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the name of the download item is derived from the URL, we currently end up reporting a partially percentage encoded name. This is problematic if our users try to undo the the percentage encoding and opens them out to download-folder escaping files that can install hooks in the user's home directory. [ChangeLog][DownloadItem] (QWebEngine)DownloadItem::path() was previously incorrectly returning percentage encoded file-names when the suggested path was based on URL. This has been corrected. Note percentage decoding the path generally before is not just incorrect when the path is not based on URL, but also dangerous as it can lead to downloads that escape the download folder. Task-number: QTBUG-58155 Change-Id: Ie23a4ff5d4e4c353df72e617bb2b00e1935cd6c1 Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
| * Avoid overriding new and deleteAllan Sandfeld Jensen2017-01-061-1/+3
| | | | | | | | | | | | | | | | As a minimum this produces valgrind warnings of mismatched new and delete. No runtime issues have been observed outside of valgrind though. Change-Id: I5ebb6b380f25f117434633e55dd7fdf04260f0bc Reviewed-by: David Faure <david.faure@kdab.com>
| * Replace ASSERT in FaviconManager::setIcon by returnViktor Engelmann2017-01-051-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | The ASSERT in FaviconManager::setIcon can be triggered by what seems to be a race condition. Apparently the old request can be gone (so m_inProgressRequests.contains(id) is false), but m_inProgressRequests can still be non-empty, because of a new request to the same URL, so this case is not covered by checking m_inProgressRequests.isEmpty() before. Task-number: QTBUG-57900 Change-Id: I2f26c05e5c97e4bd8d1cea03e8005cf61f2b0c98 Reviewed-by: Michael Brüning <michael.bruning@qt.io>