- 31 Aug, 2018 40 commits
-
-
Miguel Casas authored
This CL cleans up a bit the jpeg_decode_accelerator_unittest file; singleton JpegDecodeAcceleratorTestEnvironment created a number of temporary files on SetUp() (via CreateTestJpegImage() method) that were removed on TearDown(). It also created by hand the content of TestImageFile objects. All this logic is moved to the struct TestImagefile (renamed to ParsedJpegImage), so that a) the struct is better defined in terms of what it holds and how is it initialized (CreateFromFile() and CreateBlackImage() are the factory methods), b) there's one place only to fill in its sizes (InitializeSizes()) and c) there's no need for temporary files written to disk and deleted. Also for good measure: - GetOriginalOrTestDataFilePath() is simplified and folded inline. - Couple of consts/constexprs here and there. - Initialize members in ctor initialization lists (ISO ctor body). Test: compile and run j_d_a_unittest on nautilus Bug: 877694 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: Ib2cb4ba218df84955757298e7d0cd85503ee1c98 Reviewed-on: https://chromium-review.googlesource.com/1197372 Commit-Queue: Miguel Casas <mcasas@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Cr-Commit-Position: refs/heads/master@{#588153}
-
Yuwei Huang authored
Turns out I forgot to include the resize-to-fit feature in Android client's What's New text. This CL adds it back. Bug: 877704 Change-Id: I277c5e5eaefd3247769b7cdf17c84f95a8db0072 Reviewed-on: https://chromium-review.googlesource.com/1198565Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Commit-Queue: Yuwei Huang <yuweih@chromium.org> Cr-Commit-Position: refs/heads/master@{#588152}
-
braveyao authored
webrtc::ScreenCapturerMac requires to be started and destroyed on same thread. The current SequencedTaskRunner can't satisfy this and will cause random crashes. This cl is to use an exclusive capture thread as same as the one in content/browser/media/capture/desktop_capture_device.cc. Bug: 877982,851883 Change-Id: I6508add3660f63e1d2539dd2175a131f869a836a Reviewed-on: https://chromium-review.googlesource.com/1199806 Commit-Queue: Weiyong Yao <braveyao@chromium.org> Reviewed-by:
Julien Isorce <julien.isorce@chromium.org> Cr-Commit-Position: refs/heads/master@{#588151}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/9951cbf4ca79..7bfe8438a94b git log 9951cbf4ca79..7bfe8438a94b --date=short --no-merges --format='%ad %ae %s' 2018-08-31 bsalomon@google.com Test glbetex and glbert configs on BonusConfigs bots 2018-08-31 swiftshader-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/swiftshader 5fb5e817a20d..fe5861bf5bb1 (1 commits) 2018-08-31 stani@google.com Support more hardware buffer formats 2018-08-31 michaelludwig@google.com Fixes for primitive handling for uniforms 2018-08-31 jvanverth@google.com Simplify perspective shadows to match future MD specs. 2018-08-31 fmalita@chromium.org [skottie] Lazily-initialized default font manager 2018-08-31 benjaminwagner@google.com Unomit passing tests for Radeon. 2018-08-31 benjaminwagner@google.com Remove GTX1070 and GTX1050 jobs. 2018-08-31 bsalomon@google.com Fix GrAAConvexTessellator colinear point removal. 2018-08-31 benjaminwagner@google.com [infra] Add rack dimension for QuadroP400 tasks. 2018-08-31 benjaminwagner@google.com Unomit passing tests for Win Intel. 2018-08-31 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-08-31 brianosman@google.com Revert "Add support for gray + colorspace to SkCodec" Created with: gclient setdep -r src/third_party/skia@7bfe8438a94b 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 BUG=chromium:113555199,chromium:869172,chromium:784662 TBR=stephana@chromium.org Change-Id: I75ea11ce12a9a2eed101571b2adfde0c7c93ced8 Reviewed-on: https://chromium-review.googlesource.com/1199694Reviewed-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@{#588150}
-
Will Harris authored
This suppresses a number of warnings that will be enabled in the future. Remove unneeded extra warnings, as these are covered by a global config. ../../third_party/ots/src/prep.cc(25,20): warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long long') to 'uint32_t' (aka 'unsigned int') [-Wshorten-64-to-32] this->m_length = length; ~ ^~~~~~ ../../third_party/ots/src/fpgm.cc(24,18): warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long long') to 'uint32_t' (aka 'unsigned int') [-Wshorten-64-to-32] this->length = length; ~ ^~~~~~ ../../third_party/ots/src/post.cc(104,44): warning: implicit conversion loses integer precision: 'std::vector<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > > >::size_type' (aka 'unsigned long long') to 'const unsigned int' [-Wshorten-64-to-32] const unsigned num_strings = this->names.size(); ~~~~~~~~~~~ ~~~~~~~~~~~~^~~~~~ ../../third_party/ots/src/cvt.cc(28,18): warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long long') to 'uint32_t' (aka 'unsigned int') [-Wshorten-64-to-32] this->length = length; ~ ^~~~~~ ../../third_party/ots/src/cmap.cc(153,49): warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long long') to 'uint32_t' (aka 'unsigned int') [-Wshorten-64-to-32] ranges[i].id_range_offset_offset = subtable.offset(); ~ ~~~~~~~~~^~~~~~~~ ../../third_party/ots/src/cmap.cc(884,42): warning: implicit conversion loses integer precision: 'std::vector<ots::OpenTypeCMAPSubtableVSRecord, std::allocator<ots::OpenTypeCMAPSubtableVSRecord> >::size_type' (aka 'unsigned long long') to 'const unsigned int' [-Wshorten-64-to-32] const unsigned num_records = records.size(); ~~~~~~~~~~~ ~~~~~~~~^~~~~~ ../../third_party/ots/src/cmap.cc(886,30): warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long long') to 'uint32_t' (aka 'unsigned int') [-Wshorten-64-to-32] !out->WriteU32(this->subtable_0_5_14_length) || ~~~~~~~~ ~~~~~~^~~~~~~~~~~~~~~~~~~~~~ ../../third_party/ots/src/cmap.cc(901,44): warning: implicit conversion loses integer precision: 'std::vector<ots::OpenTypeCMAPSubtableVSRange, std::allocator<ots::OpenTypeCMAPSubtableVSRange> >::size_type' (aka 'unsigned long long') to 'const unsigned int' [-Wshorten-64-to-32] const unsigned num_ranges = ranges.size(); ~~~~~~~~~~ ~~~~~~~^~~~~~ ../../third_party/ots/src/cmap.cc(916,48): warning: implicit conversion loses integer precision: 'std::vector<ots::OpenTypeCMAPSubtableVSMapping, std::allocator<ots::OpenTypeCMAPSubtableVSMapping> >::size_type' (aka 'unsigned long long') to 'const unsigned int' [-Wshorten-64-to-32] const unsigned num_mappings = mappings.size(); ~~~~~~~~~~~~ ~~~~~~~~~^~~~~~ ../../third_party/ots/src/cmap.cc(963,40): warning: implicit conversion loses integer precision: 'std::vector<ots::OpenTypeCMAPSubtableRange, std::allocator<ots::OpenTypeCMAPSubtableRange> >::size_type' (aka 'unsigned long long') to 'const unsigned int' [-Wshorten-64-to-32] const unsigned num_groups = groups.size(); ~~~~~~~~~~ ~~~~~~~^~~~~~ ../../third_party/ots/src/cmap.cc(985,40): warning: implicit conversion loses integer precision: 'std::vector<ots::OpenTypeCMAPSubtableRange, std::allocator<ots::OpenTypeCMAPSubtableRange> >::size_type' (aka 'unsigned long long') to 'const unsigned int' [-Wshorten-64-to-32] const unsigned num_groups = groups.size(); ~~~~~~~~~~ ~~~~~~~^~~~~~ ../../third_party/ots/src/cff.cc(69,36): warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long long') to 'uint32_t' (aka 'unsigned int') [-Wshorten-64-to-32] index->offset_to_next = table->offset(); ~ ~~~~~~~^~~~~~~~ ../../third_party/ots/src/cff.cc(112,28): warning: implicit conversion loses integer precision: 'unsigned long long' to 'unsigned int' [-Wshorten-64-to-32] object_data_offset + (rel_offset - 1)); // less than length(), 1GB. ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~ ../../third_party/ots/src/ots.cc(545,23): warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long long') to 'Cr_z_uLongf' (aka 'unsigned long') [-Wshorten-64-to-32] uLongf dest_len = *table_length; ~~~~~~~~ ^~~~~~~~~~~~~ ../../third_party/ots/src/ots.cc(776,33): warning: implicit conversion loses integer precision: 'unsigned long long' to 'uint32_t' (aka 'unsigned int') [-Wshorten-64-to-32] out.length = end_offset - out.offset; ~ ~~~~~~~~~~~^~~~~~~~~~~~ ../../third_party/ots/src/ots.cc(793,21): warning: implicit conversion loses integer precision: 'const size_t' (aka 'const unsigned long long') to 'off_t' (aka 'long') [-Wshorten-64-to-32] if (!output->Seek(table_record_offset)) { ~~~~ ^~~~~~~~~~~~~~~~~~~ ../../third_party/ots/src/ots.cc(819,39): warning: implicit conversion loses integer precision: 'unsigned long long' to 'off_t' (aka 'long') [-Wshorten-64-to-32] if (!output->Seek(head_table_offset + 8)) { ~~~~ ~~~~~~~~~~~~~~~~~~^~~ ../../third_party/ots/src/ots.cc(826,21): warning: implicit conversion loses integer precision: 'const size_t' (aka 'const unsigned long long') to 'off_t' (aka 'long') [-Wshorten-64-to-32] if (!output->Seek(end_of_file)) { ~~~~ ^~~~~~~~~~~ BUG=879657 Change-Id: I7a69310b9672657f7373bdc79b6a4b4cb9181c12 Reviewed-on: https://chromium-review.googlesource.com/1199489 Commit-Queue: Will Harris <wfh@chromium.org> Reviewed-by:Behdad Esfahbod <behdad@chromium.org> Cr-Commit-Position: refs/heads/master@{#588149}
-
sebsg authored
Remove the master toggle. Add sublabels for the profile and credit card toggles. Add row to inform users if an extention is managing the toggles. Change the title from "Addresses" to "Addresses and more". Bug: 860526, 877801 Change-Id: I2d0d156108667d9463263f4e490761a506bf9a13 Reviewed-on: https://chromium-review.googlesource.com/1196924 Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org> Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:
Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#588148}
-
Matthew Jones authored
Contextual search's observer is unreliable for show/hide events of the feature. This patch replaces the use of that observer with an observer that watches any overlay panel and its visibility. Bug: 878006 Change-Id: I271b61504ca99ceebe308f7a65bdb4c06ee64cc7 Reviewed-on: https://chromium-review.googlesource.com/1197166 Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#588147}
-
Marina Ciocea authored
When running audio service sandboxed on Mac, the audio process is a daemon process and should not connect to launchservicesd. Bug: 874785 Change-Id: I84ffe96fea85d3d54fb9b1cca1d441b5d6f085a1 Reviewed-on: https://chromium-review.googlesource.com/1194073Reviewed-by:
Olga Sharonova <olka@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Marina Ciocea <marinaciocea@chromium.org> Cr-Commit-Position: refs/heads/master@{#588146}
-
chrome://webrtc-logs/Elad Alon authored
Before this CL, chrome://webrtc-logs/ only displayed textual logs. After this CL, event logs are also displayed. Bug: 775415 Change-Id: Iadfaf6173b118edb11641736b566c96b346c376e Reviewed-on: https://chromium-review.googlesource.com/1189483 Commit-Queue: Elad Alon <eladalon@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Henrik Grunell <grunell@chromium.org> Cr-Commit-Position: refs/heads/master@{#588145}
-
Christopher Cameron authored
Take the existing BridgedNativeWidgetHost and break it into two parts - a mojo interface views_bridge_mac::mojo::BridgedNativeWidgetHost - a helper and "stuff we haven't mojo-ified-yet" interface BridgedNativeWidgetHostHelper - this is not named BridgedNativeWidgetHost to avoid conflicts - this will be moved to //ui/views_bridge_mac soon (build is tangled) Add mojo support for ui::DialogButton, because it's used by this interface. For keyboard event handling, use the BridgedNativeWidgetHostHelper interface because a mojo-compatible interface would be hard for the callers to use. Bug: 859152 Change-Id: If6802ab4d6e518184cd3fa57eef5d8a510b80e59 Reviewed-on: https://chromium-review.googlesource.com/1198014Reviewed-by:
-
Antonio Gomes authored
TBR=dullweber@chromium.org (for tools/traffic_annotation/summary/annotations.xml) BUG=773295 Change-Id: Id47e63bd76376712ab8f0a8a68edd501d8db4227 Reviewed-on: https://chromium-review.googlesource.com/1197045 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Joe Downing <joedow@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#588143}
-
James Wallace-Lee authored
When a tab-modal dialog is shown, the page underneath it is inert. But NVDA reads the page underneath the dialog when users press Insert+B to read the whole dialog. To fix this, we hide the rest of the browser from the accessibility tree when a tab-modal dialog is visible by modifying AXPlatformNodeDelegate::PopulateChildWidgetVector to return only the dialog. Bug: 775680 Change-Id: I2a5ec2eeef5ffe50ebd8819836c89ae73ddfdd36 Reviewed-on: https://chromium-review.googlesource.com/1189054 Commit-Queue: James Wallace-Lee <jamwalla@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#588142}
-
Xing Liu authored
This CL obseletes two expired histograms. And update the expired date for several others. These metrics are kept: Download.DeleteRetentionTime.Video, Android.DownloadManager.ViewRetentionTime.Video: This is recent work that PM may not have time to check the data yet. Download.ParallelDownloadAddStreamSuccess, Download.ParallelDownloadRequestCount: Still useful to monitor feature health. Bug: 876072,876070 Change-Id: Iab713f6dccf674556bd1d67d16b2f99bb3c29d5a Reviewed-on: https://chromium-review.googlesource.com/1199491Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#588141}
-
Ovidio Henriquez authored
This change adds validation checks to the USBDeviceFilter parameters passed into navigator.usb.requestDevice. Bug: 854703 Change-Id: I921a7f30a9e2ec5d46d3a74801de594565ab8a6f Reviewed-on: https://chromium-review.googlesource.com/1196034Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Ovidio Henriquez <odejesush@chromium.org> Cr-Commit-Position: refs/heads/master@{#588140}
-
depot-tools-chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/a4b67d055334..4259762b3597 git log a4b67d055334..4259762b3597 --date=short --no-merges --format='%ad %ae %s' 2018-08-31 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@4259762b3597 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: Ie4d420f3db00e4c39a0038a227c7453b332fbeef Reviewed-on: https://chromium-review.googlesource.com/1199690Reviewed-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@{#588139}
-
Dominick Ng authored
With UserActivationV2, user gestures no longer expire in 1 second. This CL consumes the user activation when the method is called in order to avoid multiple usages of the beforeinstallpromptevent.prompt() method with only one activation. This CL also fixes a test to trigger beforeinstallpromptevent.prompt() on keyUp rather than keyPress. This avoids a race condition where the user activation is generated *after* the consuming call to prompt(), causing subsequent tests to fail as they assumed the gesture had been consumed when it was actually being immediately regenerated. BUG=869756 Change-Id: Ie9249c0933a5c8f263de4171ec2559cd4f0e80f8 Reviewed-on: https://chromium-review.googlesource.com/1196622Reviewed-by:
Mustaq Ahmed <mustaq@chromium.org> Reviewed-by:
Matt Giuca <mgiuca@chromium.org> Commit-Queue: Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#588138}
-
nacl-chromium-autoroll authored
https://chromium.googlesource.com/native_client/src/native_client.git/+log/2b59aaeee137..d636a4fae3c1 git log 2b59aaeee137..d636a4fae3c1 --date=short --no-merges --format='%ad %ae %s' 2018-08-31 dschuff@chromium.org Make posix code compile with -Wimplicit-fallthrough Created with: gclient setdep -r src/native_client@d636a4fae3c1 The AutoRoll server is located here: https://autoroll.skia.org/r/nacl-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:868204 TBR=mseaborn@chromium.org Change-Id: Ic373070648e879ba67572bff0d849362f6da1ff3 Reviewed-on: https://chromium-review.googlesource.com/1199692Reviewed-by:
nacl-chromium-autoroll <nacl-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: nacl-chromium-autoroll <nacl-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#588137}
-
Eric Karl authored
We currently unlock pre-decoded images after 2 frames of any type. This doesn't always give callers a chance to use their images (if main thread is a bit backed up). Instead we should wait for the caller to actually use the image. Additionally, we drop images after 250ms to prevent excessive growth if a user fails to use images in a timely manner. See the linked bug for example traces which show the benefits. This re-land updates image_decoder_runner.js to avoid errors due to variable decoding times leading to image cache exhaustion. R=khushalsagar Bug: 869491 Change-Id: I90c6665684b74c779810b6dee963b2a62b26cb2a Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Reviewed-on: https://chromium-review.googlesource.com/1159203 Commit-Queue: Eric Karl <ericrk@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#583171} Reviewed-on: https://chromium-review.googlesource.com/1196235Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#588136}
-
Leonard Grey authored
GPU v1 sandbox removes condition since we don't have the os-version variable there (per offline discussion). Bug: 879646 Change-Id: I06b29d1c6b4b6fd621a5b2497b0a2b78fe36c85f Reviewed-on: https://chromium-review.googlesource.com/1200184Reviewed-by:
-
Marton Hunyady authored
Rename the following values for better understanding and consistency: - RollbackWithFullPowerwash to RollbackAndPowerwash - RollbackWithRestoreOnly to RollbackOnlyIfRestorePossible Bug: 878736 Change-Id: Ia617d4e6402b3ad5cd5a0791735160b8dbe593b3 Reviewed-on: https://chromium-review.googlesource.com/1196376Reviewed-by:
Pavol Marko <pmarko@chromium.org> Commit-Queue: Marton Hunyady <hunyadym@chromium.org> Cr-Commit-Position: refs/heads/master@{#588134}
-
Clark DuVall authored
This reverts commit 22b92b16. Reason for revert: Fixed bug where listeners would not be notified when waiting for initial connection status. This change fixes the bug and adds a regression test. In addition, this puts the change behind the ResourceRequestAllowedMigration feature, as suggested by asvitkine@. We can test this on Canary to make sure the Variations.SeedFreshness metric is not affected. Note that the Variations.ResourceRequestsAllowed will have different behavior with this change, as now instead of just sending RESOURCE_REQUESTS_ALLOWED when the network is connected, it will send RESOURCE_REQUESTS_NOT_ALLOWED_NETWORK_STATE_NOT_INITIALIZED and then RESOURCE_REQUESTS_ALLOWED_NOTIFIED. This hopefully should not affect Variations.SeedFreshness. See diffs starting at Patchset 1 for changes from last reland. Some of the old code needed to be added back to support both versions behind the feature flag. Original change's description: > Revert "Reland "Reland "Migrate ResourceRequestAllowedNotifier to NetworkConnectionTracker""" > > This reverts commit 34141782. > > Reason for revert: Caused Finch seed freshness regression, see http://crbug.com/879252 and > http://crbug.com/879115 > > Original change's description: > > Reland "Reland "Migrate ResourceRequestAllowedNotifier to NetworkConnectionTracker"" > > > > This is a reland of e00fddec > > > > Needed to initialize variables in ChromeBrowserMainBrowserTest for asan/msan. > > Verified this works with msan build. > > > > Original change's description: > > > Reland "Migrate ResourceRequestAllowedNotifier to NetworkConnectionTracker" > > > > > > This is a reland of a9ed46b7 > > > > > > ChromeBrowserMainBrowserTest.VariationsServiceStartsRequestOnNetworkChange > > > was flaky in the original change. Added logic in the test to wait for the > > > connection type change. > > > > > > Only diffs from original are in chrome/browser/chrome_browser_main_browsertest.cc > > > > > > Original change's description: > > > > Migrate ResourceRequestAllowedNotifier to NetworkConnectionTracker > > > > > > > > A getter is used for NetworkConnectionTracker because some services that > > > > use ResourceRequestAllowedNotifier are initialized early in browser > > > > startup (e.g. VariationsService), and only perform the initialization > > > > of ResourceRequestAllowedNotifier later on the UI thread. The getter > > > > allows us to run get the connection tracker at that point so we don't get > > > > DCHECKs about being on the UI thread when running > > > > content::GetNetworkConnectionTracker(). > > > > > > > > This also moves the NetworkConnectionTracker in ios/ from BrowserState to > > > > ApplicationContext, which is available everywhere. It also matches non-IOS > > > > usage more closely, since we have it as a global there. > > > > > > > > Bug: 868021 > > > > Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet > > > > Change-Id: I130c6b47feb90f0f7f0776ccc65666414a1ae802 > > > > Reviewed-on: https://chromium-review.googlesource.com/1180360 > > > > Reviewed-by: Eugene But <eugenebut@chromium.org> > > > > Reviewed-by: Robert Sesek <rsesek@chromium.org> > > > > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > > > > Reviewed-by: Robbie McElrath <rmcelrath@chromium.org> > > > > Commit-Queue: Clark DuVall <cduvall@chromium.org> > > > > Cr-Commit-Position: refs/heads/master@{#584849} > > > > > > TBR=eugenebut@chromium.org,rsesek@chromium.org,rmcelrath@chromium.org,jam@chromium.org > > > > > > Bug: 868021 > > > Change-Id: I5941b72474657159f0d4a1e6667fd77a3c475887 > > > Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet > > > Reviewed-on: https://chromium-review.googlesource.com/1185602 > > > Reviewed-by: Clark DuVall <cduvall@chromium.org> > > > Reviewed-by: Robbie McElrath <rmcelrath@chromium.org> > > > Commit-Queue: Clark DuVall <cduvall@chromium.org> > > > Cr-Commit-Position: refs/heads/master@{#585387} > > > > TBR=eugenebut@chromium.org,rsesek@chromium.org,rmcelrath@chromium.org,jam@chromium.org > > > > Bug: 868021, 876861 > > Change-Id: I46fccf072d0b3080603e97c73ff055ac7c45e723 > > Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet > > Reviewed-on: https://chromium-review.googlesource.com/1187081 > > Commit-Queue: Clark DuVall <cduvall@chromium.org> > > Reviewed-by: Clark DuVall <cduvall@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#585565} > > TBR=jam@chromium.org,eugenebut@chromium.org,rsesek@chromium.org,rmcelrath@chromium.org,cduvall@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 868021, 876861, 879252, 879115 > Change-Id: If0417f53386a94de0ec3b64c283df123c673ec03 > Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs > Reviewed-on: https://chromium-review.googlesource.com/1197325 > Commit-Queue: Clark DuVall <cduvall@chromium.org> > Reviewed-by: Clark DuVall <cduvall@chromium.org> > Cr-Commit-Position: refs/heads/master@{#587694} TBR=eugenebut@chromium.org Bug: 868021, 876861, 879252, 879115 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I2190d82f8de1188763b6f1bb1de2b8762d23aa07 Reviewed-on: https://chromium-review.googlesource.com/1198102 Commit-Queue: Clark DuVall <cduvall@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Robbie McElrath <rmcelrath@chromium.org> Cr-Commit-Position: refs/heads/master@{#588133}
-
John Budorick authored
Change-Id: I6bfcf3efbe4dbcfa769afb77e732050f6527703c Reviewed-on: https://chromium-review.googlesource.com/1199766Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#588132}
-
Sébastien Séguin-Gagnon authored
From Enabled to FullLaunch Change-Id: I7f6c6ccc924d7aa012e648e0c5eab9c3fe289494 Reviewed-on: https://chromium-review.googlesource.com/1199563Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#588131}
-
François Beaufort authored
This makes sure Picture-in-Picture window always shows up inside the work area when surface ID is updated. Bug: 873657 Change-Id: Ibc9a640d527f51e5311a411b86af23eb61617f37 Reviewed-on: https://chromium-review.googlesource.com/1196363Reviewed-by:
apacible <apacible@chromium.org> Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Cr-Commit-Position: refs/heads/master@{#588130}
-
Chromite Chromium Autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/1940d7df3f6d..68de241167e0 git log 1940d7df3f6d..68de241167e0 --date=short --no-merges --format='%ad %ae %s' 2018-08-31 gmeinke@chromium.org parallel_emerge: remove saving of install plan Created with: gclient setdep -r src/third_party/chromite@68de241167e0 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-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=chrome-os-gardeners@chromium.org Change-Id: Icdcd82d4d9376b0ad769a532b0715c15610de554 Reviewed-on: https://chromium-review.googlesource.com/1199693Reviewed-by:
Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#588129}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/cd6bcbe53ba7..6fc719e87628 git log cd6bcbe53ba7..6fc719e87628 --date=short --no-merges --format='%ad %ae %s' 2018-08-31 zmo@chromium.org Remove removed folder from telemetry isolate 2018-08-31 chiniforooshan@chromium.org Tracing: Fix an input latency bug 2018-08-31 pasko@chromium.org devil: refactor perf_control.py for upcoming N5X support 2018-08-31 anthonyalridge@google.com Improve dot stacking algoritm to handle small numbers. 2018-08-31 anthonyalridge@google.com Flip axes for CFG. 2018-08-31 achuith@chromium.org oobe: Catch WebSocketException. Created with: gclient setdep -r src/third_party/catapult@6fc719e87628 The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-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 BUG=chromium:879523,chromium:383566,chromium:866423,chromium:866423,chromium:879353 TBR=sullivan@chromium.org Change-Id: Ieb839c3ef97435a3479cfb2b98d5441308e1dd70 Reviewed-on: https://chromium-review.googlesource.com/1199691Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#588128}
-
Alexandr Ilin authored
This CL adds end to end tests for the LoadingPredictor. There are two categories of tests: * Web platform link rel features tests. * Learning from past navigations tests. Bug: 602396 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: Ic51c77f7289ec08b99341ef1d51a6fa01c64309b Reviewed-on: https://chromium-review.googlesource.com/1185191 Commit-Queue: Alexandr Ilin <alexilin@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#588127}
-
Chris Palmer authored
Like IPC, but for code that has heightened memory-safety requirements. Bug: None Change-Id: I9a7e2ba77b140e560f05cb850136697f3c5fb95a Reviewed-on: https://chromium-review.googlesource.com/1196006Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Reviewed-by:
-
Xiangjun Zhang authored
WindowAdoptionAgent is used by TabCapture API to find a root Window to adopt the Webcontents native view containing the tab content being captured on AURA platform. This CL moves the implement internal to OffscreenTab to allow it being used by the mirroring service to support mirroring an OffscreenTab. Purely code moving, implementation doesn't change at all. Bug: 734672 Change-Id: I2b663e24f91496c844f64257ded1da43461ca736 Reviewed-on: https://chromium-review.googlesource.com/1192207Reviewed-by:
Yuri Wiitala <miu@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Commit-Queue: Xiangjun Zhang <xjz@chromium.org> Cr-Commit-Position: refs/heads/master@{#588125}
-
Mathias Carlen authored
Bug: 806868 Change-Id: Ic799b514679510bbebd333a5e96d42cd0506a5f5 Reviewed-on: https://chromium-review.googlesource.com/1196365Reviewed-by:
Cait Phillips <caitkp@chromium.org> Reviewed-by:
Ganggui Tang <gogerald@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Commit-Queue: Mathias Carlen <mcarlen@chromium.org> Cr-Commit-Position: refs/heads/master@{#588124}
-
Ben Wagner authored
https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/96b5e500909c..2c8e6279a7bf $ git log 96b5e5009..2c8e6279a --date=short --no-merges --format='%ad %ae %s' 2018-08-28 wl Minor doc fix. 2018-08-26 wl * src/truetype/ttobjs.c (tt_size_read_bytecode): Trace CVT values. 2018-08-26 wl ChangeLog*: s/\t/ /. 2018-08-26 ramakrishnan.nikhil * configure: Copy assets required by docwriter. 2018-08-26 wl * src/pshint/pshalgo.c (psh_hint_overlap): Fix numeric overflow. 2018-08-26 wl Minor tracing adjustments. 2018-08-26 wl [truetype] Avoid nested frames. 2018-08-26 wl Various minor clean-ups. 2018-08-25 wl Add missing ChangeLog entry for the last four commits. 2018-08-25 ramakrishnan.nikhil * builds/*/*: Prepare build system for docwriter. 2018-08-25 ramakrishnan.nikhil * src/tools/docmaker: Remove `docmaker'. 2018-08-25 ramakrishnan.nikhil * docs/reference: Add resources for `docwriter'. 2018-08-24 ramakrishnan.nikhil * include/*.*: Convert comments to markdown. 2018-08-23 wl Add macros for handling over-/underflowing `FT_Int64' values. 2018-08-23 wl ftbitmap.h: Improve documentation. 2018-08-22 wl [truetype] Improve legibility of `glyf' parsing. 2018-08-22 wl Minor formatting and documentation fixes. 2018-08-21 wl * src/sfnt/ttcpal.c (tt_face_load_cpal): Add missing safety check. 2018-08-19 wl Update `CHANGES' file. 2018-08-18 wl [psaux] Avoid slow PS font parsing in case of error. 2018-08-18 wl [cff] Fix heap buffer overflow in old engine. 2018-08-16 yangx92 * builds/mac/ftmac.c (parse_fond): Fix buffer overrun. 2018-08-16 wl * builds/*/ftsystem.c (FT_COMPONENT): Updated also. 2018-08-15 apodtele Revert BDF copyright years. 2018-08-15 apodtele Ouch. BDF copyright year. 2018-08-15 apodtele [bdf] Don't track duplicate encodings. 2018-08-15 wl Don't use `trace_' prefix for FT_COMPONENT arguments. 2018-08-14 wl Use formatting string in FT_TRACEX calls for non-simple arguments. 2018-08-14 apodtele [bdf] Remove unused fields. 2018-08-14 wl [cff] Fix another segv in old engine. 2018-08-14 wl [cff] Fix missing error handling. 2018-08-13 apodtele [bdf] Remove unused overflow storage. 2018-08-14 wl [cff] Fix segv. 2018-08-13 apodtele [bdf] Use unsigned types. 2018-08-13 wl * src/type42/t42parse.c (t42_parse_sfnts): One more format check. 2018-08-12 prince.cherusker Remove deprecated constant in documentation. 2018-08-11 wl * src/base/ftcalc.c (FT_Matrix_Check): Fix integer overflow. 2018-08-10 apodtele * src/sfnt/ttsbit.c (tt_sbit_decoder_load_compound): Follow specs. Created with: roll-dep src/third_party/freetype/src R=bungeman@chromium.org,drott@chromium.org CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_chromium_msan_rel_ng PDFium-Issue: pdfium:1142 Change-Id: I05138182f832a75a09f63595a766324f9fa80d7a Reviewed-on: https://chromium-review.googlesource.com/1194495Reviewed-by:
Emil A Eklund <eae@chromium.org> Commit-Queue: Ben Wagner <bungeman@chromium.org> Cr-Commit-Position: refs/heads/master@{#588123}
-
Tanja Gornak authored
TBR=pavely@chromium.org Bug: 801985 Change-Id: Ib1f9729452c29d9a5a991785aff4a867014067cc Reviewed-on: https://chromium-review.googlesource.com/1193846 Commit-Queue: Tatiana Gornak <melandory@chromium.org> Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#588122}
-
Findit authored
This reverts commit 2ea45c06. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 588100 as the culprit for failures in the build cycles as shown on: https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzJlYTQ1YzA2MmMwZDU5ZWQ4MWVjNWYxODVhMjNmZmIzOWMzNGZmMDMM Sample Failed Build: https://ci.chromium.org/buildbot/chromium.linux/Linux%20Builder%20%28dbg%29%2832%29/87361 Sample Failed Step: compile Original change's description: > Sandbox audio service on Linux. > > Run audio service process in sandboxed mode on Linux. Sandbox is enabled behind finch experiment flag: > --enable-features=AudioServiceStreams,AudioServiceOutOfProcess,AudioServiceSandbox > > TBR=mpdenton@chromium.org, palmer@chromium.org, tsepez@chromium.com > > > Bug: 850878 > Change-Id: Iba6b84adc8c4a62ac4c20d5b36aa4f21b36a1574 > Reviewed-on: https://chromium-review.googlesource.com/1160844 > Commit-Queue: Marina Ciocea <marinaciocea@chromium.org> > Reviewed-by: Tom Sepez <tsepez@chromium.org> > Reviewed-by: Nasko Oskov <nasko@chromium.org> > Reviewed-by: Robert Sesek <rsesek@chromium.org> > Reviewed-by: Olga Sharonova <olka@chromium.org> > Cr-Commit-Position: refs/heads/master@{#588100} Change-Id: I64a6f98f7b2a89e47324a37bf8e91fda6caa0e7d No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 850878 Reviewed-on: https://chromium-review.googlesource.com/1199855 Cr-Commit-Position: refs/heads/master@{#588121}
-
angle-chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/064458a8f818..89e829793028 git log 064458a8f818..89e829793028 --date=short --no-merges --format='%ad %ae %s' 2018-08-31 geofflang@chromium.org Vulkan: Allow gaps in vertex attributes by packing them. 2018-08-31 geofflang@chromium.org Lock around all EGL and GL calls with a global mutex. 2018-08-31 jmadill@chromium.org Vulkan: Optimize ContextVk::updateDefaultAttributes. 2018-08-31 jmadill@chromium.org Vulkan: Refactor VertexArrayVk::streamVertexData. 2018-08-31 jmadill@chromium.org Vulkan: Fix depth range scaling. 2018-08-31 geofflang@chromium.org Skip DrawElementsIndexOutOfRangeWithStaticDraw on NVIDIA D3D11 FL9.3. Created with: gclient setdep -r src/third_party/angle@89e829793028 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=geofflang@chromium.org Change-Id: I64b81eba80029a7c210d21e0acce474dad7e3299 Reviewed-on: https://chromium-review.googlesource.com/1199707Reviewed-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@{#588120}
-
BUG=762641 TBR=xiyuan@google.com Change-Id: I9826e11db700c9c120bf61399486b5dd57ea7aba Reviewed-on: https://chromium-review.googlesource.com/1199852 Commit-Queue: ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Reviewed-by:
ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#588119}
-
Krishna Govind authored
R=cmasso@chromium.org Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: Ib24ca8256fd51be6060915393f73fc731b2bdca3 Reviewed-on: https://chromium-review.googlesource.com/1200304Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#588118}
-
Matthew Jones authored
Change-Id: I5d4f64bfae7b35a8bc91d6e3a2ae080494c0011e Reviewed-on: https://chromium-review.googlesource.com/1199612Reviewed-by:
-
Tommy C. Li authored
This is in preparation for launching on all channels where M69 has been released. Bug: 842236 Change-Id: I305368731bca3da6d59842a76dc558b5b4135844 Reviewed-on: https://chromium-review.googlesource.com/1196912 Commit-Queue: Tommy Li <tommycli@chromium.org> Commit-Queue: Justin Donnelly <jdonnelly@chromium.org> Reviewed-by:
-
Ken Buchanan authored
Currently when the mouse cursor moves between OOPIF boundaries while a selection is being dragged, the frame containing the selection stops receiving mouse events. This causes a number of bugs such as the relevant frame not being aware of when the mouse button is released. This CL causes Blink's SelectionController to signal the browser process for mouse capture to the current RenderWidgetHost. Bug: 876662, 864957, 647378 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: Id9d4bb6bdb6a3fa33b4e15af35a0588e952755fe Reviewed-on: https://chromium-review.googlesource.com/1197409Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Commit-Queue: Ken Buchanan <kenrb@chromium.org> Cr-Commit-Position: refs/heads/master@{#588115}
-
Mike Wittman authored
Relanding with fix for ChromeContentRendererClientMetricsTest. Renames targets to better reflect what they provide. Removes the :metrics dep from the target for child processes to avoid including unnecessary browser-process-only code in those processes. Breaks the direct dependency of CallStackProfileBuilder on :metrics by introducing a receiver callback abstraction. TBR=sky Bug: 878509, 879513, 804942 Change-Id: Ib83ad55a31b98022531a8dad740c7d5516db8caf Reviewed-on: https://chromium-review.googlesource.com/1199764Reviewed-by:
Mike Wittman <wittman@chromium.org> Commit-Queue: Mike Wittman <wittman@chromium.org> Cr-Commit-Position: refs/heads/master@{#588114}
-
