- 01 Mar, 2018 40 commits
-
-
Jialiu Lin authored
If enterprise policy specified the SafeBrowsingWhitelistDomains, Chrome should trust URLs in these domains. This CL covers: 1. Safe Browsing will not check for dangerous resources if their URLs match these domains (a.k.a no SB interstitial shown) Note that, we don't plan to skip SSL error checking even if domains are whitelisted. 2. Safe Browsing's password protection service will not check for password reuse if the page URL matches these domains. Changes in download protection, client phishing/malware detection, incident reporting, ads sampling will be in separate CLs. Bug: 811454 Change-Id: Ia439642ed1d68890ace4b8049aaec187158e2f5c Reviewed-on: https://chromium-review.googlesource.com/927531Reviewed-by:
Pavel Yatsuk <pavely@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Maxim Kolosovskiy <kolos@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Varun Khaneja <vakh@chromium.org> Commit-Queue: Jialiu Lin <jialiul@chromium.org> Cr-Commit-Position: refs/heads/master@{#540184}
-
Ken Rockot authored
This needed to be done at some point. The motivation here was to fix an extensions WebRequest API test with the Network Service enabled. This also plumbs through a signal so extensions API test JS code can detect whether Network Service is enabled. This is used to support conditional expectations in WebRequest tests since the test-only event behavior may differ slightly between Network Service and old networking. Bug: 721414 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I009061f038eb3171721fe7645444695a77f3cfd3 Reviewed-on: https://chromium-review.googlesource.com/940447 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#540183}
-
Jeremy Roman authored
A fairly direct copy of CoerceJavaScriptNullOrUndefinedToJavaValue. Bug: 794320 Change-Id: I6d057198262f589334d100800da63151280a38ce Reviewed-on: https://chromium-review.googlesource.com/941902Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#540182}
-
Scott Chen authored
TBR: dpapad Bug: 812947 Change-Id: Ib7193b8859e2a105d4d802b267f4fce6775e7a42 Reviewed-on: https://chromium-review.googlesource.com/942883 Commit-Queue: Scott Chen <scottchen@chromium.org> Reviewed-by:
Scott Chen <scottchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#540181}
-
Jeremy Roman authored
A fairly direct copy of CoerceJavaScriptStringToJavaValue. Bug: 794320 Change-Id: I4e7cbd020fe9702d59c84a32e0e986f970feace1 Reviewed-on: https://chromium-review.googlesource.com/941788Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#540180}
-
Douglas Creager authored
This lets us use it in tests under //chrome/browser. Bug: 704259 Change-Id: Ic814dd88e5a676e13a0b9629fc422ec70df4f6cd Reviewed-on: https://chromium-review.googlesource.com/941642 Commit-Queue: Douglas Creager <dcreager@google.com> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#540179}
-
Gauthier Ambard authored
This CL adjusts the blur effect used by the toolbar to the one requested in crbug.com/807778. It also adjust the color of the tools menu button to match the one from the other buttons. Bug: 807284 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia03bacf8563cf4b0f9a33a1b4e57dff47b67bfbc Reviewed-on: https://chromium-review.googlesource.com/941441 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#540178}
-
Jeremy Roman authored
A fairly direct copy of CoerceJavaScriptBooleanToJavaValue Bug: 794320 Change-Id: Ifac567f8b5d300a89d4057e47bb45a28863e715f Reviewed-on: https://chromium-review.googlesource.com/940346Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#540177}
-
Guido Urdaneta authored
This mainly serves as a test for https://crbug.com/816323, but also removes two DCHECKs that were wrong. This CL also removes UserMediaProcessor::RequestInfo::HasPendingSources() since it was used just for implementing two DCHECKs, one of which is no longer valid due to other recent changes. No behavior change is intended. Bug: 816597, 816323 Change-Id: Icfddae5f9a9709e9702f696cb70f289697bdcb6b Reviewed-on: https://chromium-review.googlesource.com/938041Reviewed-by:
Harald Alvestrand <hta@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#540176}
-
Rob Buis authored
Fix errorprone UseCorrectAssertInTests warnings and make sure it is treated as error after this CL. http://errorprone.info/bugpattern/UseCorrectAssertInTests Bug: 803482 Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ic337a2e0313986961af4390c38bfa47bfdb46325 Reviewed-on: https://chromium-review.googlesource.com/938642Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Tao Bai <michaelbai@chromium.org> Reviewed-by:
Helen Li <xunjieli@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: Rob Buis <rob.buis@samsung.com> Cr-Commit-Position: refs/heads/master@{#540175}
-
Dianna Hu authored
1. Rename SpdySettingsIds --> SpdyKnownSettingsId. 2. Add a "using SpdySettingsId = uint16_t" statement, and replace where appropriate (e.g., raw on-the-wire values; should be a non-functional change), 3. Perform minor refactoring of the utility method for converting a SETTINGS ID to a string. This is a first step to additional support for unknown SETTINGS IDs. Also perform some minor lint clean-ups to appease `git cl lint`, including string and base::StringPiece --> SpdyString(Piece) (note that the original error was to include <string>), adding #includes, and fixing line lengths. This CL lands server change 186360913 by diannahu. BUG=488484 Change-Id: I99c7efea92e11e355255e54ee70f4a8d1d6496b4 Reviewed-on: https://chromium-review.googlesource.com/938603 Commit-Queue: Dianna Hu <diannahu@chromium.org> Reviewed-by:
Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#540174}
-
W. James MacLean authored
Modify this text fixture to parameterize it over multiple device scale factors. Bug: 680606 Change-Id: I247afdf39ab9e640c711372ca6d07c66897d6bbd Reviewed-on: https://chromium-review.googlesource.com/943341Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Commit-Queue: James MacLean <wjmaclean@chromium.org> Cr-Commit-Position: refs/heads/master@{#540173}
-
Dave Tapuska authored
The InputPane APIs first appeared in RS1 version. RS2 or later SDK is required to build Chrome so changing the version shouldn't be a problem. BUG=817501 Change-Id: I1ed1f4a630ffe1d913dee47c03ad4fd45f3c6759 Reviewed-on: https://chromium-review.googlesource.com/941913 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#540172}
-
Peter Beverloo authored
This test was disabled almost four years ago for unknown reasons and passes fine on my machine. Let's see if the bots agree. https://codereview.chromium.org/209293002 Change-Id: Id62e8c362b7b73861697ad1a82c0189471f60080 Reviewed-on: https://chromium-review.googlesource.com/891188Reviewed-by:
Anita Woodruff <awdf@chromium.org> Commit-Queue: Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#540171}
-
Fady Samuel authored
This reverts commit cef2d52e. Reason for revert: It's just plain wrong. It broke some viz tests and it caused the breakage in 817771. We probably want an EvictLastActivatedSurfaces({framesinks}); API instead. Original change's description: > Surface synchronization: Don't evict surface on DelegatedFrameHost::WasResized > > If we evict a surface before the previous CompositorFrame activates with it > as a fallback, then we will attempt to add an invalid reference and we may > end up with missing surfaces in surface aggregator. This CL combats the issue > by avoiding evicting frames during resize. > > We will still only update the primary surface ID if the tab is not hidden, > and we will clear the fallback surface ID if the tab is hidden. > > Bug: 813157, 672962 > Change-Id: Ie0b408af503ec9fefdcccc5fb0607cf015734052 > Reviewed-on: https://chromium-review.googlesource.com/938678 > Reviewed-by: Saman Sami <samans@chromium.org> > Commit-Queue: Fady Samuel <fsamuel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#539639} TBR=fsamuel@chromium.org,samans@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 813157, 672962 Change-Id: If7a78d81d23e646bb52f1d1f666613c6e54af383 Reviewed-on: https://chromium-review.googlesource.com/943161Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Commit-Queue: Fady Samuel <fsamuel@chromium.org> Cr-Commit-Position: refs/heads/master@{#540170}
-
Aaron Leventhal authored
Ensure that setting relationships in Views plumbs through information into accessibility API. Bug: None Change-Id: I01d6c9e1924046a5465e2a83ff0e22d6170ad8a0 Reviewed-on: https://chromium-review.googlesource.com/922766 Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#540169}
-
Tim Volodine authored
Downstream glue layer patch crrev.com/i/579513 has landed, so the old upstream getTracingController code is not used anymore and can be removed. BUG=781249,808023 TBR=torne@chromium.org Change-Id: I398087a0df8e652528c79de1a2ce16fa99a9423b Reviewed-on: https://chromium-review.googlesource.com/943441Reviewed-by:
Tim Volodine <timvolodine@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Commit-Queue: Tim Volodine <timvolodine@chromium.org> Cr-Commit-Position: refs/heads/master@{#540168}
-
jonross authored
Some viz_content_unittests have been fixed, re-enabling them. TBR=kylechar@chromium.org TEST=RenderWidgetHostTest.FrameToken_RendererCrash, RenderWidgetHostTest.NewContentRenderingTimeout, RenderWidgetHostTest.SwapCompositorFrameWithBadSourceId Bug: 760320 Change-Id: Ib52cd12d381575a134893d1d2d6f4b51021b4428 Reviewed-on: https://chromium-review.googlesource.com/943462Reviewed-by:
Jonathan Ross <jonross@chromium.org> Cr-Commit-Position: refs/heads/master@{#540167}
-
Sandra Sun authored
According to spec, https://drafts.csswg.org/css-scroll-snap-1/#scroll-types, programmatically scrolling such as scrollTo and scrollBy should land on snap points. Bug: 803606 Change-Id: I1e70d88f31a2b2f93c92cc063adf653ed923adeb Reviewed-on: https://chromium-review.googlesource.com/889936Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Sandra Sun <sunyunjia@chromium.org> Cr-Commit-Position: refs/heads/master@{#540166}
-
Gang Wu authored
Turned on Typed URL for USS by default, and remove typed URL syncable service. Also remove profile_sync_service_typed_url_unittest.cc. Background: profile sync service tests are existing before integration tests, and now integration tests covered most of them, so delete profile_sync_service_typed_url_unittest.cc here. Bug: 81368 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I7f53f8b73f09b2b25a7202dff59689b699796965 Reviewed-on: https://chromium-review.googlesource.com/928097Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Reviewed-by:
Pavel Yatsuk <pavely@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Gang Wu <gangwu@chromium.org> Cr-Commit-Position: refs/heads/master@{#540165}
-
Peter Wen authored
Overhead should be tracked but these symbols are not actionable. Making them padding-only removes them from size diffs. Bug: 814078 Change-Id: I68991f54d4ee990c7bd675c4dd0f22e5337eb9f4 Reviewed-on: https://chromium-review.googlesource.com/941587 Commit-Queue: Peter Wen <wnwen@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#540164}
-
Kevin Bailey authored
Feature change which adds a button to tab switch suggestions (under a flag). Currently the flag does nothing but let the user evaluate it. In a subsequent change, the button will invert the selection logic such that <enter> will navigate, <shift-enter> will switch and the button will switch. Bug: 780835 Change-Id: I4453598f95db22fed8cc0bde50ef6d9788a205b2 Reviewed-on: https://chromium-review.googlesource.com/919247 Commit-Queue: Kevin Bailey <krb@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#540163}
-
Gabriel Charette authored
And use it instead of flaky sleep timings in tests that exercise the reclaim timeout. Bonus: this speeds up these tests from ~1500ms to ~500ms in the normal case. R=fdoray@chromium.org, robliao@chromium.org Bug: 735701, 803900 Change-Id: I3b271fda6c7b42033ac7b3c128e8663e80acebb3 Reviewed-on: https://chromium-review.googlesource.com/937505 Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#540162}
-
Peter Wen authored
Original CL: https://crrev.com/c/938122 Fix: - Division by zero when there is no uncompressed pak files. TBR=agrieve@chromium.org Bug: 814078,817534 Change-Id: Id3198a84dce95074d13ca20afe4caec45631b0cc Reviewed-on: https://chromium-review.googlesource.com/943221Reviewed-by:
Peter Wen <wnwen@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#540161}
-
Eric Seckler authored
The new test verifies that animated GIFs don't produce CompositorFrames for animate_only BeginFrames and that their animation is aligned with virtual time progression. Also replaces use of --enable-surface-synchronization flag with Feature and adds a trace event for impl-side invalidations to cc::Scheduler. Bug: 646774 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I24f5a3ab14588f787b3dd571223b4737574929d8 Reviewed-on: https://chromium-review.googlesource.com/941562Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Reviewed-by:
Alex Clarke <alexclarke@chromium.org> Commit-Queue: Eric Seckler <eseckler@chromium.org> Cr-Commit-Position: refs/heads/master@{#540160}
-
Emily Hanley authored
Change-Id: Ie32821761795bdaaffe3d6a2c0bf6054bd436230 Reviewed-on: https://chromium-review.googlesource.com/931926 Commit-Queue: Emily Hanley <eyaich@chromium.org> Reviewed-by:
Ashley Enstad <ashleymarie@chromium.org> Cr-Commit-Position: refs/heads/master@{#540159}
-
Ioana Pandele authored
Bug:817785 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Id735d1bd561de497e351d0482fa0e21b3c874359 Reviewed-on: https://chromium-review.googlesource.com/943106Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Ioana Pandele <ioanap@chromium.org> Cr-Commit-Position: refs/heads/master@{#540158}
-
jonross authored
More tests have been fixed, re-enabling on FYI to start to verify no flakes. TBR=kylechar@chromium.org TEST=viz_content_browsertests, viz_browser_tests Bug: 760181 Change-Id: I3d27ff8372fbe10b46efeb074f2bf07d60e14f43 Reviewed-on: https://chromium-review.googlesource.com/943243Reviewed-by:
Jonathan Ross <jonross@chromium.org> Cr-Commit-Position: refs/heads/master@{#540157}
-
Michael Hablich authored
This is a roll-back to v8 6.6.346. This is the most stable candidate out of the latest Canaries. If this commit results in failures in Blink please contact the Blink sheriff. They may need to make changes to Blink because of the roll-back. When in doubt please try to contact the committer and reviewers of this CL before reverting it. TBR=machenbach@chromium.org NOTRY=true Change-Id: I9ca1490f7d5f647abd7788acd3ff385f64d1acc5 Reviewed-on: https://chromium-review.googlesource.com/942902 Commit-Queue: Michael Hablich <hablich@chromium.org> Reviewed-by:
Michael Hablich <hablich@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#540156}
-
Harald Alvestrand authored
These counters measure non-standard features, except for the "dtmf" attribute which measures the standard feature that is replacing a non-standard feature (createDTMFSender). Bug: 812587 Change-Id: Iffd97a87fbc81a3c9b6f9a8afbd3c775220da1d1 Reviewed-on: https://chromium-review.googlesource.com/941802 Commit-Queue: Harald Alvestrand <hta@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#540155}
-
Gabriel Charette authored
While auditing the code I noticed that this was a problem: if a worker cleans up while waiting for n workers to be idle, the idle stack will shrink and this condition may never be reached. This prevents this completely for WaitForAllWorkersIdleForTesting() and greatly reduces the likelihood for WaitForWorkersIdleForTesting(n). WaitForAllWorkersIdleForTesting() is safe because "n" is obtained under the lock. Whereas an external call to WaitForWorkersIdleForTesting(n) could wait on a worker which was already detached. But in practice if the reclaim timeout is long enough and the main thread invokes this immediately after unblocking a test task and waiting for it to resolve, the race would require the main thread to be preempted for the full reclaim timeout for no reason which seems highly unlikely. Audited current usage and added a comment as such. R=fdoray@chromium.org, robliao@chromium.org Bug: 816170, 735701 Change-Id: Ie7b4fd81c04f53cc9597ebc02a25b39d4f1d658f Reviewed-on: https://chromium-review.googlesource.com/937361 Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#540154}
-
Keishi Hattori authored
Limit trace depth for VisitPersistents so we don't trace most of the object graph in one go. Bug: 757440 Change-Id: Ib43956d4b6f128169ada158e849f6e9668591378 Reviewed-on: https://chromium-review.googlesource.com/937421 Commit-Queue: Keishi Hattori <keishi@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#540153}
-
Rakina Zata Amni authored
This change is the continuation of crrev.com/c/925763. With this change, CSSOM changes to constructed stylesheets are reflected on all TreeScopes where the style is applied to. Spec draft: https://wicg.github.io/construct-stylesheets/#dom-cssstylesheet-cssstylesheet Bug: 807560 Change-Id: I1cddeea9dc346aa238fa4d0aabe233397166e2a5 Reviewed-on: https://chromium-review.googlesource.com/938872Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Rakina Zata Amni <rakina@chromium.org> Cr-Commit-Position: refs/heads/master@{#540152}
-
clamy authored
This CL replaces the usage of a ResourceDispatcherHostDelegate in CrossSite* content_browsertests by TestNavigationManagers. It also moves the remaining CrossSiteResourceHandlerTest into the CrossSiteTransferTests. Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: Ie10a45ce406471d4286fbc893e870cb044c4c4ac Reviewed-on: https://chromium-review.googlesource.com/922182 Commit-Queue: Camille Lamy <clamy@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#540151}
-
Mounir Lamouri authored
This is done by keeping track of whether a playback was autoplay initiated when starting and always returning this value to the WatchTimeReporter. Bug: 814268 Change-Id: Id34b73047df7a68ee8bb19d6f25285930a85d60e Reviewed-on: https://chromium-review.googlesource.com/923984Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#540150}
-
Francois Doray authored
With this CL, when a WebContents is occluded on aura: - WebContents::GetVisibility returns OCCLUDED - RenderWidgetHostImpl::WasHidden is called on all RenderWidgetHostImpls that belong to the WebContents. That allows: - Changing the priority of renderer processes to background. - Not rendering frames unnecessarily. Bug: 668690 Change-Id: Ie1c86d271276576a6b5c7adfb11bd0e42d5d491e Reviewed-on: https://chromium-review.googlesource.com/937820Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#540149}
-
Ramin Halavati authored
Network traffic annotation is added to content/renderer/media/webrtc/peer_connection_dependency_factory.cc Bug: 656607 Change-Id: Ie88c13b8ab6f28e7604f48d134c231b16ab18a61 Reviewed-on: https://chromium-review.googlesource.com/934127 Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Taylor Brandstetter <deadbeef@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#540148}
-
Christopher Cameron authored
The device_scale_factor_ and painted_device_scale_factor_ members of LayerTreeHost are mutually exclusive -- a tree will either use one or the other (and leave the unused member as 1). Prior to this cleanup we would call SetDeviceScaleFactor or SetPaintedDeviceScaleFactor to update these variables. Add a LayerTreeSettings member to select the device scale factor mode, and make LayerTreeHost::SetViewportSizeAndScale update the specified variable. Remove the remaining calls to SetPaintedDeviceScaleFactor (from Android's compositor) and remove the function. Remove RenderWidgetCompositor::SetDeviceScaleFactor, and several related WebKit functions that called this. The call was only necessary to work around a bug where we did not specify the "original" (aka "non-emulated") device scale factor in RenderWidget::UpdateCompositorSurface non-IsUseZoomForDSFEnabled mode. R=dgozman,piman,fsamuel TBR=pdr (for public/) Bug: 805774 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I233727da320b14b041bb48515da3cfb0d2be1ea4 Reviewed-on: https://chromium-review.googlesource.com/938958 Commit-Queue: ccameron <ccameron@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#540147}
-
kylechar authored
The VizDisplayCompositor feature moves the display compositor off the browser main thread and into the GPU process on a dedicated thread. Make sure that the display compositor thread is recorded separately in the thread_times benchmark and counted as part of the fast path total. Bug: 778749 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: Ic18c3564ecf5e192f2dcf1e2e272dac3c99d8873 Reviewed-on: https://chromium-review.googlesource.com/937398 Commit-Queue: kylechar <kylechar@chromium.org> Reviewed-by:
Ned Nguyen <nednguyen@google.com> Reviewed-by:
Brian Anderson <brianderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#540146}
-
Bryce Thomas authored
This event notifies the dev tools client when a web font has successfully loaded. Event information includes the font-family, src, and other font properties, as well as the resolved platform font family name. R=dgozman@chromium.org Bug: 803226 Change-Id: I2bb399ce1c08b0a397f8eb94d95aa85556ebb2c1 Reviewed-on: https://chromium-review.googlesource.com/905829 Commit-Queue: Dominik Röttsches <drott@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/master@{#540145}
-