- 10 Sep, 2018 32 commits
- 
- 
Jia authoredBug: 881215 Change-Id: I1cd5bf7ea517e426b54df1f4ebbb5a843b991dbe Reviewed-on: https://chromium-review.googlesource.com/1209000Reviewed-by: Michael Martis <martis@chromium.org> Reviewed-by: Dan Erat <derat@chromium.org> Commit-Queue: Jia Meng <jiameng@chromium.org> Cr-Commit-Position: refs/heads/master@{#589843} 
- 
Stuart Langley authoredAs per the attached bug, replace usage of Replace (bool, const std::string&) in filesystem_api_util.cc with base::Optional<std::string> for code health. Also, change from RepeatingCallback to OnceCallback for all filesystem_api_util defined methods. Also, fix linter warnings. There are no logic changes. Bug: 729442, 875700 Change-Id: Ib27634b9ba1372cb007ba6a190608ec4128bbeb3 Reviewed-on: https://chromium-review.googlesource.com/1215425Reviewed-by: Sam McNally <sammc@chromium.org> Reviewed-by: Noel Gordon <noel@chromium.org> Commit-Queue: Stuart Langley <slangley@chromium.org> Cr-Commit-Position: refs/heads/master@{#589842} 
- 
Maksim Sisov authoredThis fixes the following error: wl::Object<wl_buffer> wl_buffer; ^~~~~~~~~ In file included from ../../ui/ozone/platform/wayland/wayland_buffer_manager.h:18:0, from ../../ui/ozone/platform/wayland/wayland_buffer_manager.cc:5: ../../ui/ozone/platform/wayland/wayland_object.h:12:8: error: changes meaning of 'wl_buffer' from 'struct wl_buffer' [-fpermissive] struct wl_buffer; ^~~~~~~~~ TBR=rjkroege@chromium.org Change-Id: I326abd95fec4d26a4d0497e5570ae4e9f0637afb Reviewed-on: https://chromium-review.googlesource.com/1213179 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by:Maksim Sisov <msisov@igalia.com> Cr-Commit-Position: refs/heads/master@{#589841} 
- 
Trent Apted authoredThis is used for unit tests in ui/file_manager, but it is neat and other targets should use it too. It's now pretty stable. It reduces boilerplate in BUILD.gn files and allows foo_unittest.html files to be automatically generated from build dependencies, rather than being handcrafted. js_unit_test.py currently lacks the performance optimization added for js_binary in r581961 to cache dependency subtrees, so it's a bit slow. Upstreaming allows the two python scripts to share the dependency walker in a sane manner. Bug: 860355 Cq-Include-Trybots: luci.chromium.try:closure_compilation Change-Id: Iedddd39907137dd40ec21f507cb560756251c8b1 Reviewed-on: https://chromium-review.googlesource.com/1212364 Commit-Queue: Trent Apted <tapted@chromium.org> Reviewed-by: Naoki Fukino <fukino@chromium.org> Cr-Commit-Position: refs/heads/master@{#589840} 
- 
Hiroki Nakagawa authoredThis CL adds the Save-Data header to browser-initiated requests for shared worker's main script. This has been tested by LayoutTests, but it's no longer available after NetworkService is enabled because LayoutTest's internal API to enable the data saver feature doesn't take effect on the preferences in the browser process. Therefore, this CL adds browser tests instead. Design doc of the PlzWorker: https://docs.google.com/document/d/1Jtn33bvqkqWxq6K7HIA4uU6HLWPTmOD7vFviacfTmhM/edit?usp=sharing Bug: 715632 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I1390205530742dba73057f6a866af8db158afe54 Reviewed-on: https://chromium-review.googlesource.com/1209004Reviewed-by: Tarun Bansal <tbansal@chromium.org> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#589839} 
- 
Maksim Sisov authoredThe memory header was missing from this header. Adding it here. Change-Id: Ic52dc3a194fabb50dd6758e6b1af693358b408cc Reviewed-on: https://chromium-review.googlesource.com/1213247 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#589838} 
- 
Ramin Halavati authoredThe following preferences are removed from the persistent storage list for incognito mode. - kAutoplayAllowed, - kAutoplayWhitelist, - kDeviceOpenNetworkConfiguration, - kDownloadDefaultDirectory, - kDownloadDirUpgraded, - kDownloadExtensionsToOpen, - kDSEGeolocationSettingDeprecated, - kDSEPermissionsSettings, - kDSEWasDisabledByPolicy - kEnabledLabsExperiments - kIncompatibleApplications, - kIsolateOrigins, - kLocalConsentsDictionary - kModuleBlacklistCacheMD5Digest, - kObservedSessionTime, - kOpenNetworkConfiguration, - kProblematicPrograms, - kSitePerProcess, - kThirdPartyBlockingEnabled, - kUnsafelyTreatInsecureOriginAsSecure - kWebDriverOverridesIncompatiblePolicies - kWebShareVisitedTargets These preferences are either never changed from incognito mode, or the changed value is not supposed to be written to user profile. Bug: 861722 Change-Id: I67fdb205175d9656ef07fd3039a2496e74968ceb Reviewed-on: https://chromium-review.googlesource.com/1196384 Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by: Christian Dullweber <dullweber@chromium.org> Reviewed-by: Sam McNally <sammc@chromium.org> Reviewed-by: Christopher Thompson <cthomp@chromium.org> Reviewed-by: Ben Wells <benwells@chromium.org> Reviewed-by: Alexei Svitkine <asvitkine@chromium.org> Reviewed-by: David Trainor <dtrainor@chromium.org> Reviewed-by: Becca Hughes <beccahughes@chromium.org> Reviewed-by: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#589837} 
- 
Hayato Ito authoredRemove several TraversalXXXIterator, such as TraversalAncestorIterator, TraversalChildrenIterator, in favor of reusing TraversalIterator with TraversalXXX wrapper, such as TraversalParent or TraversalSibling. This CL also introduces several short names for some common traversal patterns, such as: TraversalAncestorRange = TraversalRange<TraversalIterator<TraversalParent<T>>>, for better ergonomics. No behavior change. Bug: 880052 Change-Id: I0ccf2a0de672932cff2df38591b8376df9d52175 Reviewed-on: https://chromium-review.googlesource.com/1205924 Commit-Queue: Hayato Ito <hayato@chromium.org> Reviewed-by: Yoichi Osato <yoichio@chromium.org> Cr-Commit-Position: refs/heads/master@{#589836} 
- 
Leo Zhang authoredThe feature was previously created under the IME service, I refactored it by moving it to ChromeOS features for a simple structure and passing it to the IME extension. This feature is used for switching whether the IME component extension is able to connect to IME service. Bug: 837156,880245 Change-Id: I832b60e1bcc90708771a935dc9f3172067eb8544 Reviewed-on: https://chromium-review.googlesource.com/1209131Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by: Darren Shen <shend@chromium.org> Commit-Queue: Leo Zhang <googleo@chromium.org> Cr-Commit-Position: refs/heads/master@{#589835} 
- 
Satoshi Niwa authoredmojo CL : 1164875 ARC CL : ag/4895169 BUG=b:111816608 TEST=manually tested with caroline Change-Id: I0fbcf66c91f3035fcd973d03a4ee63400a096ad8 Reviewed-on: https://chromium-review.googlesource.com/1203710Reviewed-by: Shuhei Takahashi <nya@chromium.org> Reviewed-by: Naoki Fukino <fukino@chromium.org> Commit-Queue: Satoshi Niwa <niwa@chromium.org> Cr-Commit-Position: refs/heads/master@{#589834} 
- 
Han Leon authoredThis CL is the reland of https://chromium-review.googlesource.com/c/chromium/src/+/816085, which got reverted because it caused time out on service worker threads when transferring scripts. Now we found out the root cause: |ThreadSafeScriptContainer::waiting_url_| is a KURL, since it's supposed to be accessed on both the IO thread and the service worker thread, we must make a deep copy of KURL when assigning value to it, which was being missed by the original CL. This CL: - removes the unnecessary abstract layer WebServiceWorkerInstalledScriptsManager.h between Content and Blink. - merges web_service_worker_installed_scripts_manager_impl.{cc,h} out of content/renderer into blink/renderer/modules/service_worker/. - merges thread_safe_script_container.{cc,h} out of content/renderer into blink/renderer/modules/service_worker/. - revises related test codes. BUG=792778 Change-Id: I5225d1e1414a5e3859c9c108a86fe5f7447e8987 Reviewed-on: https://chromium-review.googlesource.com/1208903Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by: Kentaro Hara <haraken@chromium.org> Reviewed-by: Makoto Shimazu <shimazu@chromium.org> Commit-Queue: Han Leon <leon.han@intel.com> Cr-Commit-Position: refs/heads/master@{#589833} 
- 
Yutaka Hirano authoredThe test is flaky because speculative preloads start, and in some cases finish loading before disabling console message. Let's use testharness.js in order to ignore console messages. Bug: 882292 Change-Id: Ic6627f0bb941e7299ac611c2fd10e6469e1ed5e9 Reviewed-on: https://chromium-review.googlesource.com/1214978Reviewed-by: Kouhei Ueno <kouhei@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#589832} 
- 
Eriko Kurimoto authoredEach types of log (speech, event ...) can be filtered from URL params. To enable users to change URL easily, LogPage is now on tab, not panel. Bug: 879042 Test: None Change-Id: I64b80f828707f60df23afab6f57c68135199c57c Reviewed-on: https://chromium-review.googlesource.com/1195407 Commit-Queue: Eriko Kurimoto <elkurin@google.com> Reviewed-by: David Tseng <dtseng@chromium.org> Reviewed-by: Yuki Awano <yawano@chromium.org> Cr-Commit-Position: refs/heads/master@{#589831} 
- 
Donna Wu authoredThis CL changes the third parameter of GrantDevicePermission in UsbChooserContext from |guid| to |device_info|, and stores more info about the ephemeral devices, so that we can get rid of direct references to UsbService in subsequent CLs. Bug: 699790 Change-Id: Ifaa741cec5f4ddb888c3377c2ad0742e677fb521 Reviewed-on: https://chromium-review.googlesource.com/1203877 Commit-Queue: Donna Wu <donna.wu@intel.com> Reviewed-by: Raymes Khoury <raymes@chromium.org> Reviewed-by: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#589830} 
- 
Dominick Ng authoredSite engagement levels in the renderer are only used for some metrics. We have decided to not use engagement for gating APIs directly in the renderer. This CL removes the IPC messaging for sending engagement to the renderer to reduce the IPC overhead (1 message every navigation, +1 message every time the coarse engagement level changed over the lifetime of a page). BUG=655415 Change-Id: I8b04f75b11631cd0102717b37bf665a2a503c79d Reviewed-on: https://chromium-review.googlesource.com/1208995Reviewed-by: Nasko Oskov <nasko@chromium.org> Reviewed-by: Avi Drissman <avi@chromium.org> Reviewed-by: Ben Wells <benwells@chromium.org> Reviewed-by: Kentaro Hara <haraken@chromium.org> Commit-Queue: Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#589829} 
- 
Noel Gordon authoredAdd an Ash EnableVirtualKeyboard mojo test API, and use it in FilesApp browser test fixtures that run in Ash tablet mode. No change in behavior: this is laying scaffold to allow testing of the Virtual Keyboard and its use in Files App in tablet mode [1]. [1] No tests of the Virtual Keyboard feature from Files App currently, and issue 762664, issue 762694 indicate this gap in test coverage. Test: browser_tests --gtest_filter="*FilesApp*TabletMode" Bug: 879434 Change-Id: I8c0c2ee8ef8e208cb721019c3cdca85f9448c5c8 Reviewed-on: https://chromium-review.googlesource.com/1210422 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by: Sam McNally <sammc@chromium.org> Reviewed-by: James Cook <jamescook@chromium.org> Reviewed-by: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#589828} 
- 
Hwanseung Lee authoredWhen RemovedFrom called, ContainerNode object was passed as a parameter. and it is never nullptr. so it can be changed to reference instead of pointer. Bug: 874385 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I231b13529f83540c7ad64d023d7421bc7a3f8ef6 Reviewed-on: https://chromium-review.googlesource.com/1215142Reviewed-by: Hayato Ito <hayato@chromium.org> Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com> Cr-Commit-Position: refs/heads/master@{#589827} 
- 
Stuart Langley authoredThe introduction of team drives means that calling DriveAPIService::GetChangeList can return a ChangeList object that contains both FileResource and TeamDriveResource entries. The sync engine code is not TeamDriveResource aware and assumes that ChangeList can only contain FileResource entries, and calling ChangeResource::File() will return a nullptr that later causes an AV in PopulateFileDetailsByFileResource. This CL filters out TeamDriveResource from the ChangeList object. Verified with new unit test. Bug: 850228 Change-Id: I1d8f4e8a99df8b8b04683eccaaceae4843020bda Reviewed-on: https://chromium-review.googlesource.com/1212366 Commit-Queue: Stuart Langley <slangley@chromium.org> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#589826} 
- 
angle-chromium-autoroll authoredhttps://chromium.googlesource.com/angle/angle.git/+log/1cee042197da..a735ee2f6df8 git log 1cee042197da..a735ee2f6df8 --date=short --no-merges --format='%ad %ae %s' 2018-09-10 jiajia.qin@intel.com ES31: Support shader storage block in D3D11 compiler - Part1 Created with: gclient setdep -r src/third_party/angle@a735ee2f6df8 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=ynovikov@chromium.org Change-Id: I77c24b27ae8524de79664eb2d3cbc46ab9afa354 Reviewed-on: https://chromium-review.googlesource.com/1215462Reviewed-by: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#589825} 
- 
depot-tools-chromium-autoroll authoredhttps://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/66badbd1cb28..25c380cf9d2d git log 66badbd1cb28..25c380cf9d2d --date=short --no-merges --format='%ad %ae %s' 2018-09-10 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@25c380cf9d2d The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=agable@chromium.org Change-Id: Ifeec74e766adeac734e462d05544a42bbab6e8ec Reviewed-on: https://chromium-review.googlesource.com/1215421Reviewed-by: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#589824} 
- 
chrome-release-bot authored# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=govind@chromium.org Change-Id: Ife4121f6a478b2dc215a86e45b141722ed38bc6d Reviewed-on: https://chromium-review.googlesource.com/1215383Reviewed-by: chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#589823} 
- 
Tatsuhisa Yamaguchi authoredChanges the RequestCaptureState() to send media capture state bundled with account id, so that receiver can identify which one represents which user's status. Test: manual Test: ash_unittest --gtest_filter=MultiProfileMediaTrayItemTest.* Bug: 879269 Change-Id: I4e808d7e88d7904673869f282b41d7c37ecc8b12 Reviewed-on: https://chromium-review.googlesource.com/1203756 Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Reviewed-by: Daniel Cheng <dcheng@chromium.org> Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org> Cr-Commit-Position: refs/heads/master@{#589822} 
- 
Yutaka Hirano authoredBug: 882292 Change-Id: I1baaabbe07a8eeaa5dd42fb6d657c1f77861bd56 Tbr: toyoshim@chromium.org No-Try: true Reviewed-on: https://chromium-review.googlesource.com/1214970Reviewed-by: Yutaka Hirano <yhirano@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#589821} 
- 
Yuki Yamada authoredThis reverts commit 9f87a1ee. https://chromium-review.googlesource.com/c/chromium/src/+/1212202 Regarding to crbug.com/881688 : This CL adds the check for if listener is empty or not before calling it, but it should not be empty at that time. This is because listener object could be collected prematurally by garbage collection. This will be fixed after unified GC is launched, or wrapper-tracing is applied appropriately to all the EventTarget (some of EventTargets need to be ActiveScriptWrappable). Bug: 872138, 881688, 878658, 606900, 849236 Change-Id: I9b224e0d261e1567c4c4f7778bd0d6044b7f6202 Reviewed-on: https://chromium-review.googlesource.com/1212376Reviewed-by: Kentaro Hara <haraken@chromium.org> Reviewed-by: Hayato Ito <hayato@chromium.org> Reviewed-by: Hitoshi Yoshida <peria@chromium.org> Reviewed-by: Yuki Shiino <yukishiino@chromium.org> Commit-Queue: Yuki Yamada <yukiy@google.com> Cr-Commit-Position: refs/heads/master@{#589820} 
- 
Chromium WPT Sync authoredUsing wpt-import in Chromium f4e5d36f. With Chromium commits locally applied on WPT: 488a36e8 "[IntersectionObserver] Fix the concept of "tracking document"" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/24432 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=raphael.kubo.da.costa@intel.com No-Export: true Change-Id: I0109a5cac31b4c7698672d598340de96d5389924 Reviewed-on: https://chromium-review.googlesource.com/1215419 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#589819} 
- 
Kenichi Ishibashi authoredServiceWorkerSubresourceLoader records loading milestones. These milestones are used to create web-exposed PerformanceResourceTiming. Before this CL, ServiceWorkerSubresourceLoader recorded |service_worker_ready_time| before calling GetControllerServiceWorker(). This doesn't seem a good timing to record it because: - |service_worker_ready_time| is used to create `fetchStart` of PerformanceResourceTiming. - `fetchStart` should be recorded after the controller service worker is ready to handle fetch event. - But the worker may not be running until we call GetControllerServiceWorker(). This CL moves the timing to record |service_worker_raedy_time| and some others after calling GetControllerServiceWorker(), as calling it makes sure that the controller is running and ready to handle a fetch event. The wpt ervice-workers/service-worker/resource-timing.https.html should cover this change. Bug: 782958 Change-Id: Ic6f6abd34ffdc84e002357a905e7a7726e6b5870 Reviewed-on: https://chromium-review.googlesource.com/1212244 Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#589818} 
- 
Yutaka Hirano authoredThis reverts commit 488a36e8. Reason for revert: Speculative revert for - http/tests/intersection-observer/v2/cross-origin-effects.html - http/tests/intersection-observer/v2/cross-origin-occlusion.html Original change's description: > [IntersectionObserver] Fix the concept of "tracking document" > > As explained in the bug, "tracking document" should be associated with > IntersectionObservation, not IntersectionObserver. > > One side effect of this change is that for a given observer, each of its > observations runs its algorithm independently, so that notifications may > not be generated in the order in which the observations were created > (i.e., the order of calls to observer.observe()). To preserve the > idiomatic web platform behavior that notifications are delivered in > the order in which the targets were observed, this patch moves the > storage of pending notifications from IntersectionObserver to > IntersectionObservation. > > BUG=879798 > R=chrishtr@chromium.org > > Change-Id: I9b6f6ad8a26387f0c072ccdff7f18cea9a88004c > Reviewed-on: https://chromium-review.googlesource.com/1200388 > Reviewed-by: Chris Harrelson <chrishtr@chromium.org> > Commit-Queue: Stefan Zager <szager@chromium.org> > Cr-Commit-Position: refs/heads/master@{#589678} TBR=szager@chromium.org,chrishtr@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 879798, 882280 Change-Id: I885276c561c8d6f21afdc059d080d58fac3b899e Reviewed-on: https://chromium-review.googlesource.com/1215423Reviewed-by: Yutaka Hirano <yhirano@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#589817} 
- 
Ben Wells authoredBug: 881634 Change-Id: Id275bfe3ff4ceb6b4989f03a96b13fcb68fd460f Reviewed-on: https://chromium-review.googlesource.com/1212209Reviewed-by: Darren Shen <shend@chromium.org> Reviewed-by: Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Ben Wells <benwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#589816} 
- 
depot-tools-chromium-autoroll authoredhttps://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/f022f91453f7..66badbd1cb28 git log f022f91453f7..66badbd1cb28 --date=short --no-merges --format='%ad %ae %s' 2018-09-09 kbr@chromium.org Increase timeout to Http instance passed to authenticator. Created with: gclient setdep -r src/third_party/depot_tools@66badbd1cb28 The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. BUG=chromium:882266 TBR=agable@chromium.org Change-Id: Iacb3446ec5eec19c9163a4be685bb50fe2de509c Reviewed-on: https://chromium-review.googlesource.com/1214815Reviewed-by: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#589815} 
- 
skia-chromium-autoroll authoredhttps://skia.googlesource.com/skia.git/+log/d22a4ff1f201..038f252b7691 git log d22a4ff1f201..038f252b7691 --date=short --no-merges --format='%ad %ae %s' 2018-09-09 angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/angle2 3b9680534909..1cee042197da (1 commits) Created with: gclient setdep -r src/third_party/skia@038f252b7691 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=caryclark@chromium.org Change-Id: Ic1a04883bf3f44b88e727da36cdf5a8f0be62249 Reviewed-on: https://chromium-review.googlesource.com/1214817Reviewed-by: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#589814} 
- 
Chromium WPT Sync authoredUsing wpt-import in Chromium 7d0daa8a. With Chromium commits locally applied on WPT: 488a36e8 "[IntersectionObserver] Fix the concept of "tracking document"" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/24431 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=raphael.kubo.da.costa@intel.com No-Export: true Change-Id: Iae4bcc7cfc51b7a0ebf7ea3a1b2b40cf06da78de Reviewed-on: https://chromium-review.googlesource.com/1214816 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#589813} 
- 
Julie Jeongeun Kim authoredWhen the mouse cursor exits from the browser window, 'CompoundEventFilter' doesn't update the cursor and it doesn't know whether ET_MOUSE_EXITED happens when the window is destroying or the mouse exits from the window. WindowTreeHostPlatform keeps the previous cursor and WaylandWindow, one of PlatformWindows, also keeps it. This patch resets the cursor from WindowTreeHostPlatform on ET_MOUSE_EXITED to update it when it reenters the window. Test: aura_unittests Bug: 879448 Change-Id: I52919aaadef4a0934eb6ad88c1f4d4ba75f802b0 Reviewed-on: https://chromium-review.googlesource.com/1198714 Commit-Queue: Julie Jeongeun Kim <jkim@igalia.com> Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#589812} 
 
- 
- 09 Sep, 2018 8 commits
- 
- 
angle-chromium-autoroll authoredhttps://chromium.googlesource.com/angle/angle.git/+log/3b9680534909..1cee042197da git log 3b9680534909..1cee042197da --date=short --no-merges --format='%ad %ae %s' 2018-09-09 fjhenigman@chromium.org Suppress ...shaders.builtin_variable.fragcoord_w. Created with: gclient setdep -r src/third_party/angle@1cee042197da The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=ynovikov@chromium.org Change-Id: Id567387eb78fe024672025e0eca7ea5cd744e956 Reviewed-on: https://chromium-review.googlesource.com/1214814Reviewed-by: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#589811} 
- 
Chromium WPT Sync authoredUsing wpt-import in Chromium dce6d45e. With Chromium commits locally applied on WPT: 488a36e8 "[IntersectionObserver] Fix the concept of "tracking document"" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/24430 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=raphael.kubo.da.costa@intel.com No-Export: true Change-Id: I6c42e6d3324b37af4b756bb183be37070f7284c9 Reviewed-on: https://chromium-review.googlesource.com/1214813 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#589810} 
- 
Koji Ishii authoredThe code to paint the background of scrolling contents layer was missing in NGBoxFragmentPainter. This patch copies the logic from BoxPainter. Issue 861623 found that this code path runs when all of the following conditions are true: * In Chrome (not in content_shell). * When display / CSS pixel >= 1.5. * When background is set to 'overflow: auto' box. This patch also fixes 3 CSS contain tests that has 'contain: paint', 'overflow-[xy]: scroll', and background color to an inline formatting context. Bug: 861623 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I31d0b407e794409552dcf89f5c8005fafc5f99a6 Reviewed-on: https://chromium-review.googlesource.com/1215143Reviewed-by: Emil A Eklund <eae@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#589809} 
- 
Jungshik Shin authoredThis roll has two Changes: https://chromium.googlesource.com/chromium/deps/icu/+/8b8d2590a https://chromium.googlesource.com/chromium/deps/icu/+/7ca3ffa77 The former is to add 'quarter'-related items to the locale data for Ecma 402 Intl implementation. The latter is to fix locale id handling API bugs in ICU also required by v8. It also defines USE_CHROMIUM_ICU to 1 so that builds NOT using Chromium's copy of ICU can take a different code path if necessary (e.g. Node.js with system ICU or ICU without the above two fixes; see https://chromium-review.googlesource.com/1215223 ). In a follow-up CL, USING_SYSTEM_ICU check in Blink will be replaced by checking USE_CHROMIUM_ICU. TBR=ftang@chromium.org, gsathya@chromium.org Bug: v8:7869, v8:8135 Test: v8's RelativeTime format test Test: test262/intl402/Intl/getCanonicalLocales/* Change-Id: Ie0475df396639ad3333de38ad1cdd24e9bfe252c Reviewed-on: https://chromium-review.googlesource.com/1215362 Commit-Queue: Jungshik Shin <jshin@chromium.org> Reviewed-by: Jungshik Shin <jshin@chromium.org> Cr-Commit-Position: refs/heads/master@{#589808} 
- 
Ehsan Karamad authoredThe initial attempt was causeing build failures in "mac-rel" bot. The cause seems to be the use of static initializers. This CL moves them to a method. This is a reland of 92df23c1 Original change's description: > Implement MimeHandlerViewFrameController > > This CL implements the renderer side of MimeHandlerView based on > cross-process frames (behind a flag). > > The new subclass of MimeHandlerViewContainerBase will use a content > frame inside a plugin element for loading the MimeHandlerViewGuest's > extension. > > The current implementation is not complete and the missing features > such as postMessage support will be added in the follow-up CLs. > > This CL also introduces a new class of parametric browser tests > (MimeHandlerViewCrossProcessTest) which will be the destination of > the current MimeHandlerViewTests as more features are integrated into > the frame-based version. The new test class runs the test on both > BrowserPlugin-based and cross-process-frame-based versions. > > Document: https://docs.google.com/document/d/10g7Y9cprYKkch9JZ0TBUWaEnHBJT1nzhskQIt1nHbWM/edit#heading=h.qw81a2bk3v6w > > Bug: 659750 > TBR=thestig@chromium.org > > Change-Id: Ia69aeed47f4fa1c7f5b81cdde71a8d8b5ff59165 > Reviewed-on: https://chromium-review.googlesource.com/1187231 > Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> > Reviewed-by: Sam McNally <sammc@chromium.org> > Reviewed-by: Ehsan Karamad <ekaramad@chromium.org> > Reviewed-by: Alex Moshchuk <alexmos@chromium.org> > Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org> > Reviewed-by: James MacLean <wjmaclean@chromium.org> > Cr-Commit-Position: refs/heads/master@{#589284} TBR=alexmos@chromium.org,lazyboy@chromium.org,sammc@chromium.org,wjmaclean@chromium.org Bug: 659750, 881696 Change-Id: I94c779664981738222c8d79ca0b10ab06174a4a7 Reviewed-on: https://chromium-review.googlesource.com/1213369Reviewed-by: Lei Zhang <thestig@chromium.org> Reviewed-by: Ehsan Karamad <ekaramad@chromium.org> Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> Cr-Commit-Position: refs/heads/master@{#589807} 
- 
perfetto-chromium-autoroll authoredhttps://android.googlesource.com/platform/external/perfetto.git/+log/660b95093bcd..5313adfefcab git log 660b95093bcd..5313adfefcab --date=short --no-merges --format='%ad %ae %s' 2018-09-09 primiano@google.com TraceProcessor: Add clipping support to sched table + minor fixes 2018-09-09 primiano@google.com trace_processor: Counters table Created with: gclient setdep -r src/third_party/perfetto@5313adfefcab The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. BUG=chromium:111252261 TBR=perfetto-bugs@google.com Change-Id: I76bf99a197ad1e4b159cb456fa821f93401acf0e Reviewed-on: https://chromium-review.googlesource.com/1214812Reviewed-by: perfetto-chromium-autoroll <perfetto-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: perfetto-chromium-autoroll <perfetto-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#589806} 
- 
Chromium WPT Sync authoredUsing wpt-import in Chromium 8ea72f74. With Chromium commits locally applied on WPT: 488a36e8 "[IntersectionObserver] Fix the concept of "tracking document"" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/24414 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=raphael.kubo.da.costa@intel.com No-Export: true Change-Id: Iae8963abcf0120f1c700494c795abf32371ca806 Reviewed-on: https://chromium-review.googlesource.com/1214809 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#589805} 
- 
skia-chromium-autoroll authoredhttps://skia.googlesource.com/skia.git/+log/40c7c64572bc..d22a4ff1f201 git log 40c7c64572bc..d22a4ff1f201 --date=short --no-merges --format='%ad %ae %s' 2018-09-09 skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com Update SKP version Created with: gclient setdep -r src/third_party/skia@d22a4ff1f201 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=caryclark@chromium.org Change-Id: I6395cbf6cdb1a7a19f213b9597142b04d1926aeb Reviewed-on: https://chromium-review.googlesource.com/1214807Reviewed-by: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#589804} 
 
-