- 31 Jul, 2018 40 commits
-
-
Kyle Milka authored
Bug: 850309 Change-Id: I84e20a2895cd1af8e01c86b33f64d787b0055ce2 Reviewed-on: https://chromium-review.googlesource.com/1147215Reviewed-by:
Kristi Park <kristipark@chromium.org> Commit-Queue: Kyle Milka <kmilka@chromium.org> Cr-Commit-Position: refs/heads/master@{#579555}
-
James Cook authored
We used to fall back to the old non-app version, but the app works fine with the focus highlight feature. Switch access is still under development. dmazzoni said not to worry about it. Bug: 841020 Test: manual, turn on "Highlight object with keyboard focus" and use shortcut viewer app Change-Id: If45a9e67add82287e39d20633e98c2e992e71119 Reviewed-on: https://chromium-review.googlesource.com/1153350 Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#579554}
-
Mathieu Perreault authored
To load the NTP faster, we include config.js data in the local ntp page's initial resources instead of later added by local_ntp.js To achieve this, we refactor into the SearchConfigProvider and cache the config data. It will be recomputed on Search provider change. We also now declare the CSP as part of a <meta> tag instead of headers, since in URLDataSource the headers are created on the IO thread and the response on the UI thread. Doing everything in the local_ntp.html response avoids races nicely. Bug: 868432 Change-Id: Iab38ca2b2e032a58c2d404938065a612b08cc69a Reviewed-on: https://chromium-review.googlesource.com/1154687Reviewed-by:
Chris Palmer <palmer@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Mathieu Perreault <mathp@chromium.org> Cr-Commit-Position: refs/heads/master@{#579553}
-
Shimi Zhang authored
We want to set icon to null when shouldShowIcon() returns false for primary action. Bug: 869500, b/111806618 Change-Id: I44203568fb167899ab25fd5d2d3dff13f6723c46 Reviewed-on: https://chromium-review.googlesource.com/1157120Reviewed-by:
Pedro Amaral <amaralp@chromium.org> Commit-Queue: Shimi Zhang <ctzsm@chromium.org> Cr-Commit-Position: refs/heads/master@{#579552}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/b303a427c03d..e782f8472f61 git log b303a427c03d..e782f8472f61 --date=short --no-merges --format='%ad %ae %s' 2018-07-31 bsalomon@google.com Remove GrResourceIOProcessor. 2018-07-31 skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll skia/third_party/skcms 2925dc93401c..f80f027f8ac9 (1 commits) Created with: gclient setdep -r src/third_party/skia@e782f8472f61 The AutoRoll server is located here: https://autoroll.skia.org 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=scroggo@chromium.org Change-Id: I76cd03a32ebdc9f58633e185af5b040e68bd6093 Reviewed-on: https://chromium-review.googlesource.com/1156911Reviewed-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@{#579551}
-
tzik authored
The first reference to a TrackedPreferencesMigrator instance used to be made in its constructor implicitly. Though this case is safe as the reference keeps alive until the other reference is made, it's error prone to make the first reference in the ctor, and an upcoming change to base::Bind rejects to make the first reference. This CL moves the implicit ref count manipulation through base::Bind() out of the constructor, so that we can avoid making the implicit first reference creation. Bug: 866456 Change-Id: I3563296c0f972d57ad8cc0c2f6e32658b9d6293d Reviewed-on: https://chromium-review.googlesource.com/1156706Reviewed-by:
Gabriel Charette <gab@chromium.org> Commit-Queue: Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#579550}
-
Nick Harper authored
TBR=bnc@chromium.org Change-Id: Iaff688980eaf782b8fb1c5923da0125f4b3755e9 Reviewed-on: https://chromium-review.googlesource.com/1152555Reviewed-by:
Nick Harper <nharper@chromium.org> Reviewed-by:
Bence Béky <bnc@chromium.org> Commit-Queue: Nick Harper <nharper@chromium.org> Cr-Commit-Position: refs/heads/master@{#579549}
-
Boris Sazonov authored
Recently added ConsentBumpFragment actually implements the consent bump screen with advanced options, so this CL renames this fragment to ConsentBumpMoreOptionsFragment. Bug: 869426 Change-Id: Id002d77e12420308efb94af9c9f192ab5becd3e1 Reviewed-on: https://chromium-review.googlesource.com/1157008Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#579548}
-
Bence Béky authored
Remove some dead code from HttpAuthHandlerMock: SetResolveExpectation(), NeedsCanonicalName(), ResolveCanonicalName(), OnResolveCanonicalName() methods, Resolve enum, resolve_ member. Change-Id: I87a7904f41433eb5e649ddc039e6d12ceb211edd Reviewed-on: https://chromium-review.googlesource.com/1154347Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#579547}
-
Tessa Nijssen authored
According to the description for Replace() in input_handler.mojom, Replace() will replace the current selection, and if there is none, it will select the word around the caret and replace that word. However, in FrameInputHandlerImpl::Replace(), the word around the caret is only selected if there is a selection (and not just a caret). This is not in line with the description given above. To fix this, the check that the WebLocalFrame has a selection is negated. If the WebLocalFrame does not have a selection, the word around the caret should be selected. Otherwise, the selection should be left as is. Bug: 869512 Change-Id: I288ab40fcfa10e0c04ad442e25f2404e0c700a14 Reviewed-on: https://chromium-review.googlesource.com/1157035Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#579546}
-
Sadrul Habib Chowdhury authored
MESSAGE_DELIVERY_POLICY_WITH_NEXT_SWAP was no longer used in code (only unit-tests was using it). So remove this, and MessageDeliveryPolicy, since MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE is the only remaining enum. Also, delete the associated code. BUG=none Change-Id: I686efef200f7d1d85261b8fb28c0012c7eba5c3f Reviewed-on: https://chromium-review.googlesource.com/1156046 Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#579545}
-
Sahel Sharify authored
This cl moves auto scroll browser tests to a new file. The change makes addition of subsequent auto scroll tests cleaner. Bug: 829775 Change-Id: I8f48db76eb913c5c8235ff035c72c83c10d1c350 Reviewed-on: https://chromium-review.googlesource.com/1155646Reviewed-by:
Timothy Dresser <tdresser@chromium.org> Commit-Queue: Sahel Sharify <sahel@chromium.org> Cr-Commit-Position: refs/heads/master@{#579544}
-
Tatsuhisa Yamaguchi authored
Resolve error opening files whose name contain blacklisted characters for URL spoofing, such as "
🔒 " (U+1F512). This change also removes the tentative hack at crrev.com/c/1085207 for crbug.com/847039. Bug: 847039,849998 Test: unit_tests --gtest_filter=ExternalFileURLUtilTest.*:FileManagerPath.* Test: net_unittests --gtest_filter=EscapeTest.* Change-Id: Icd8ba4c73f09b97e05c9af182d338e709d5876f6 Reviewed-on: https://chromium-review.googlesource.com/1142854 Commit-Queue: Asanka Herath <asanka@chromium.org> Reviewed-by:Asanka Herath <asanka@chromium.org> Reviewed-by:
Matt Giuca <mgiuca@chromium.org> Cr-Commit-Position: refs/heads/master@{#579543}
-
Ella Ge authored
This CL makes use-zoom-for-dsf enabled by default on Android This marks Test NoResizeAfterIframeLoad disabled on Android because it's failing on Android. This also rebaseline 22 gpu_tests: they are failing on Android after enabling the flag because the new result are one pixel off the original reference image. Will test expectations in a followup CL. Bug: 737777, 485650 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 Change-Id: If06cf325a831e229fd6bb1889fe1e40ed4bbe764 Reviewed-on: https://chromium-review.googlesource.com/1055812Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Ella Ge <eirage@chromium.org> Cr-Commit-Position: refs/heads/master@{#579542}
-
kristipark authored
This is to balance the MV tiles when the "add custom link" button is present. Default MV: https://screenshot.googleplex.com/brjYoMmojgO.png Custom links enabled: https://screenshot.googleplex.com/C2zg6XFzEtk.png Max custom links: https://screenshot.googleplex.com/x5KkYLmJ0tz.png Bug: 851194 Change-Id: I5c487f10b9a95a4dcbbefa5a3913fdd6cedeb04d Reviewed-on: https://chromium-review.googlesource.com/1142570 Commit-Queue: Kristi Park <kristipark@chromium.org> Reviewed-by:
-
Xida Chen authored
This reverts commit 38642faa. Reason for revert: <INSERT REASONING HERE> Crashes on Win canary Original change's description: > Reland of: Handle iframe touch ack time out > > The previous CL: > https://chromium-review.googlesource.com/c/chromium/src/+/1102729 > was reverted. The reason is that in the test we have two gesture scroll, > and the second gesture starts before the first TapDown finishes. > > This CL solve the problem. PS#1 is exactly the same as the previous CL > which was reverted, so that it is easier for review. > > TBR=creis@chromium.org > > Bug: 851644 > Change-Id: Ife9bf8f9a37892380e6587b80bcc03366cc0e80d > Reviewed-on: https://chromium-review.googlesource.com/1151094 > Commit-Queue: Xida Chen <xidachen@chromium.org> > Reviewed-by: Xida Chen <xidachen@chromium.org> > Reviewed-by: Timothy Dresser <tdresser@chromium.org> > Cr-Commit-Position: refs/heads/master@{#579074} TBR=creis@chromium.org,tdresser@chromium.org,xidachen@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 851644 Change-Id: I8d1c4753963f22e55f1b338bf2b264376b683660 Reviewed-on: https://chromium-review.googlesource.com/1157036Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#579540}
-
Stephen Martinis authored
Don't really need to specify cores, as only one bot has that builder dimension. Bug: 868024, 869187 Change-Id: Ie59cf98fc59d4bbc869618bc2add13aeedf10759 Reviewed-on: https://chromium-review.googlesource.com/1157179 Commit-Queue: Stephen Martinis <martiniss@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Cr-Commit-Position: refs/heads/master@{#579539}
-
Eric Karl authored
We're seeing large numbers of crashes on this platform, which is a very small percentage of overall population. Disable this for now, while we debug the issue. TBRs for uninteresting/mechanical changes TBR=thestig TBR=piman Bug: 867453 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: I11829fd91ec303dbc1b413af0797ea0a96839338 Reviewed-on: https://chromium-review.googlesource.com/1156998Reviewed-by:
Eric Karl <ericrk@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#579538}
-
Olivier Li authored
The chrome_cleaner logs are separate from Chrome's. Bug: 830892 Change-Id: Idb89dfc5f477451b8135e5cbe9cf02a256473aca Reviewed-on: https://chromium-review.googlesource.com/1157077Reviewed-by:
Joe Mason <joenotcharles@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Oliver Li <olivierli@chromium.org> Cr-Commit-Position: refs/heads/master@{#579537}
-
Mustaq Ahmed authored
Otherwise Frame::Client() returns null for detached frames. The following tests cover this case: - fast/frames/subframe-load-crash-main.html - fast/parser/xhtml-synchronous-detach-crash.html Bug: 865629 Change-Id: Id244983b638244d2ef52a34e2489da0b3d6c2144 Reviewed-on: https://chromium-review.googlesource.com/1145228 Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#579536}
-
Michael Spang authored
Use the concrete type of the dumb buffer for cursors. We don't need polymorphism here, and since cursors don't have framebuffers, this is needed before evolving ScanoutBuffer into a concrete framebuffer type. Bug: 869206 Change-Id: I5ee8162cd103d55df320cbc8e35be29e9ac15562 Reviewed-on: https://chromium-review.googlesource.com/1155911 Commit-Queue: Michael Spang <spang@chromium.org> Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#579535}
-
Sadrul Habib Chowdhury authored
Add some trace-events for the draw operation, and include breakdown steps for draw, swap, and wait-for-ack. Link these trace-events from the surface-aggregation trace-events for various surfaces, so that it is possible to track all the way from begin-frame to swap-ack (and later, presentation-time, when we request that for every frame). BUG=855328 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: I319b551a46a0e3f9039ae5250a6b421079cb9b71 Reviewed-on: https://chromium-review.googlesource.com/1153494Reviewed-by:
-
Peter Collingbourne authored
The attempts to set a symbol_level in these bot configurations were previously ineffective because we previously ignored the symbol_level in sanitizer builds unless one of two flags was passed. That changed with https://chromium-review.googlesource.com/1156125 which made it so that we always respect symbol_level, causing these bots to use a higher symbol_level than evidently necessary. This change restores the previous behaviour by not passing a symbol_level. Bug: 869361 Change-Id: I33ea51ca72f0fc56d8196be530eb29e6b5c33e6a Reviewed-on: https://chromium-review.googlesource.com/1157079 Commit-Queue: Peter Collingbourne <pcc@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#579533}
-
John Chen authored
Delete test case ChromeDriverTest.testHasFocusOnStartup, which is inherently flaky, as any pop-up window can cause Chrome to lose focus. Bug: 868376 Change-Id: Ie9b737791df3eadb85fb9525775963c9a997be7b Reviewed-on: https://chromium-review.googlesource.com/1157072Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Commit-Queue: John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#579532}
-
Scott Violet authored
AuraTestHelper now always create a WindowTreeHost. This resulted in being able to remove some test code around configuring when to create the WindowTreeHost. The only ick that was needed to support this is in AxRemoteHost. AxRemoteHost assumed there was only one root, which isn't the case in test code anymore. I'm hoping that once we make AxRemoteHost support more than one root this code will go away. There was also some mild trickery around window ids. In particular WindowTreeClientPrivate has some functions that pass ids to WindowTreeClient. These ids are used in creating windows, and must be unique. Because AuraTestHelper now always triggers creating a WindowTreeHost, the id must be increased. I've hopefully done things so tests shouldn't have to worry about this. BUG=866634 TEST=covered by tests Change-Id: Ia3d2cd757555478f3cce8929574bc8df8acb3af2 Reviewed-on: https://chromium-review.googlesource.com/1153773 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#579531}
-
Thomas Guilbert authored
This CL adds the Java counterpart to media/base/flinging_controller.h. It also replaces MediaController with FlingingController in Java code. (native side equivalent CL: https://chromium-review.googlesource.com/c/chromium/src/+/1136008) Bug: 790766 Change-Id: I89846acd794e408c233839055c08593973b5b450 Reviewed-on: https://chromium-review.googlesource.com/1141079Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Thomas Guilbert <tguilbert@chromium.org> Cr-Commit-Position: refs/heads/master@{#579530}
-
Urvang Joshi authored
https://aomedia.googlesource.com/aom.git/+log/4f1fd9640433..bc484c485277 $ git log 4f1fd9640..bc484c485 --date=short --no-merges --format='%ad %ae %s' 2018-07-30 wtc Validate payload_size in decoder_peek_si_internal. 2018-06-28 grant.hsu Reduce interpolation filter search 2018-07-26 binpengsmail Cleanup warp motion param in MB_MODE_INFO 2018-07-30 huisu Fix encoder multi thread test failures 2018-07-30 huisu Add const to input parameters of update_state() 2018-07-30 yaowu Fix include order: av1/encoder/encoder.c 2018-07-30 ilie.halip Correctly set symbol file header when using gcc on Windows. 2018-07-30 yaowu Simplify range check of max_gf_interval 2018-07-30 huisu Code cleanup in rd_pick_partition() 2018-07-28 martin arm: Fix building neon txfm with clang 2018-07-27 sanampudi.venkatarao Speed improvement in av1_convolve_y_sr_neon for 32-bit Neon 2018-07-30 yaowu Fix include order 2018-07-27 wtc Move CONFIG_SIZE_LIMIT check in yv12config.c. 2018-07-30 yaowu Remove effectless code 2018-07-30 ddvfinite Rename compute_stats 2018-07-28 jonathan.matthews [normative]Reset film grain params if monochrome 2018-06-23 lu_zero Update the VSX support 2018-07-27 weitinglin Comment out a debug output 2018-07-27 huisu code cleanup in encode_rd_sb_row() 2018-07-27 huisu Remove code about COLLECT_TX_SIZE_DATA (...) Created with: roll-dep src/third_party/libaom/source/libaom R=johannkoenig@google.com Bug: 868651 Change-Id: I16b309f12078b5eb1924c8c3d3b978cb644a761f Reviewed-on: https://chromium-review.googlesource.com/1157033Reviewed-by:
Johann Koenig <johannkoenig@google.com> Commit-Queue: Urvang Joshi <urvang@chromium.org> Cr-Commit-Position: refs/heads/master@{#579529}
-
Mao Huang authored
Original CL: - https://codereview.chromium.org/286953011 tcmalloc: Let DEFINE_string define const char*s. DEFINE_string is used in 3 files in tcmalloc, but we only compile one of these. In this one file, the string is converted to char every time it's used, and since the string is used after global destructors have run it needs to be copied to a second string in a static initializer. Instead of all that silliness, just let DEFINE_string define a const char* (like it does in v8 or webrtc). BUG=94925 R=willchan@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271307 BUG=724399,b:70905156 Change-Id: Iec6efe53f03daff472107e700a9d833d60e33eff Reviewed-on: https://chromium-review.googlesource.com/1130781 Commit-Queue: Gabriel Marin <gmx@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Cr-Commit-Position: refs/heads/master@{#579528}
-
James Cook authored
Wire up a11y hit test actions for remote applications. Select-to-speak works fine once the hit tests are correct. Bug: 868497 Test: added to views_mus_unittests Change-Id: Ia3dcc4cb7b4c3dcab35f0f20d31d6a5ea71f3ec0 Reviewed-on: https://chromium-review.googlesource.com/1153731 Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#579527}
-
Will Harris authored
Previously, a command line where the first argument had an unterminated double-quote would cause a crash. This CL also has some cleanup of dead code. BUG=869194 Change-Id: I20fa0814e35c5d10ab3ad76047c8233297f14a8d Reviewed-on: https://chromium-review.googlesource.com/1155991 Commit-Queue: Will Harris <wfh@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#579526}
-
Paul Miller authored
VariationsUtils.readSeedFile shouldn't log errors for missing seed files, which is expected on an app's first run. BUG=869248 Change-Id: I92d88d10d6806d402b6b64c2f870e76860bf0d69 Reviewed-on: https://chromium-review.googlesource.com/1157116Reviewed-by:
-
Kyle Milka authored
Completely remove the Google Photos option for background customization. If it's just hidden it will still show up after the network state changes. It would also be possible inspect the page and trigger the dialog. Bug: 869230 Change-Id: I312164ae0648c108709c2809382ff1ca610b8c5f Reviewed-on: https://chromium-review.googlesource.com/1156882 Commit-Queue: Kyle Milka <kmilka@chromium.org> Reviewed-by:
-
Muyuan Li authored
Bug: b/111105039 Test: Manual Change-Id: I7c39e63c2b65f557f668687bdc34ced2256462f6 Reviewed-on: https://chromium-review.googlesource.com/1149065Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: Muyuan Li <muyuanli@chromium.org> Cr-Commit-Position: refs/heads/master@{#579523}
-
Shakti Sahu authored
- Decouple the EmptyViewCoordinator from the underlying model. - Add back the concept of loading to the View. Change-Id: I568fcdc32af7aaf42908c56981538c02cc4f61ed Reviewed-on: https://chromium-review.googlesource.com/1150840 Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Reviewed-by:
Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#579522}
-
Mustafa Emre Acer authored
Bug: 814906 Change-Id: I3707ab5bf8231ab0b8805bad5b645d3db9e85811 Reviewed-on: https://chromium-review.googlesource.com/1149062 Commit-Queue: Mustafa Emre Acer <meacer@chromium.org> Reviewed-by:
anthonyvd <anthonyvd@chromium.org> Cr-Commit-Position: refs/heads/master@{#579521}
-
Stephen Martinis authored
Bug: 868024 Change-Id: I4d8b01b7e78fdd9b16b563174af9af41907ee1e5 Reviewed-on: https://chromium-review.googlesource.com/1152289 Commit-Queue: Stephen Martinis <martiniss@chromium.org> Reviewed-by:
Ryan Tseng <hinoka@chromium.org> Reviewed-by:
Nodir Turakulov <nodir@chromium.org> Reviewed-by:
-
Fabrice de Gans-Riberi authored
This layout test has been flaky for some time. TBR=peter@chromium.org Bug: 869470 Change-Id: I1749a81e5172962126c0ee32afaeb7f78e4fca13 Reviewed-on: https://chromium-review.googlesource.com/1157118Reviewed-by:
Fabrice de Gans-Riberi <fdegans@chromium.org> Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org> Cr-Commit-Position: refs/heads/master@{#579519}
-
Dmitry Gozman authored
The dependency was test-only. Extracted ViewportLayersSetup helper class to platform/testing to be reused both in animation and graphics. Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Icf6b0578ceb99c1f7630e6ef5db1f0c6082cb13a Reviewed-on: https://chromium-review.googlesource.com/1155750 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#579518}
-
Bret Sepulveda authored
The translate infobar under test is now obsolete with the launch of Harmony. It appears that the new UI is covered by various other suites (TranslateLanguageBrowserTest, TranslateBubbleViewTest, and TranslateBubbleViewBrowserTest). Bug: 867557 Change-Id: Iab4b7ec611b3383ea6aa1b9dacddaa44d821f185 Reviewed-on: https://chromium-review.googlesource.com/1152527Reviewed-by:
Hajime Hoshi <hajimehoshi@chromium.org> Commit-Queue: Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#579517}
-
Zentaro Kavanagh authored
- Large numbers of redundant requests are degrading the performance of the SMB FSP - This CL short circuits those requests while separately we can try to eliminate their root cause - This filters out 2 types of requests 1) Empty requests. These requests should never even be made since they are requests that explicitly ask for no information. 2) Thumbnail only requests. We never have thumbnail data and always provide a hardcoded response. - By short circuiting them here we can prevent them being sent to the service and incuring any network operations. BUG=chromium:867200,chromium:587231 TEST=build and test Change-Id: Ibb9eaaa175660106dd6a169781cc2ce4a83dafea Reviewed-on: https://chromium-review.googlesource.com/1156989Reviewed-by:May Lippert <maybelle@chromium.org> Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Commit-Queue: Zentaro Kavanagh <zentaro@chromium.org> Cr-Commit-Position: refs/heads/master@{#579516}
-
