- 07 Dec, 2017 16 commits
-
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/fd9e0b8f5f92..11506300e6a6 $ git log fd9e0b8f5..11506300e --date=short --no-merges --format='%ad %ae %s' 2017-12-06 hnakashima Fix ps rendering. Created with: roll-dep src/third_party/pdfium The AutoRoll server is located here: https://pdfium-roll.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. TBR=dsinclair@chromium.org Change-Id: I158842a7e3f8b45b19623f5dad9335317338d4e3 Reviewed-on: https://chromium-review.googlesource.com/812221 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#522276}
-
Nate Chapin authored
This changes ResourceClient to hold a Member<Resource> as ResourceOwner did. Existing ResourceOwner subclasses can just switch to the appropriate ResourceClient subclass. Classes that were using a ResourceClient directly no longer need to hold their own Member<Resource> and can instead use SetResource/ClearResource/GetResource, implemented in ResourceClient, to manage the resource. Bug: 790778, 652966 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Id21231355a12a27e1241c87dbb153c2a8629de8e Reviewed-on: https://chromium-review.googlesource.com/794370 Commit-Queue: Nate Chapin <japhet@chromium.org> Reviewed-by:
Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#522275}
-
Chris Harrelson authored
A filter on a canvas can itself lead to origin tainting, for reasons other than that the canvas contents are tainted. This CL changes to distinguish these two causes, so that we recompute filters on content-tainting change. Bug: 778506 Change-Id: I3cec8ef3b2772f2af78cdd4b290520113092cca6 Reviewed-on: https://chromium-review.googlesource.com/811767Reviewed-by:
Fredrik Söderquist <fs@opera.com> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#522274}
-
Kyle Horimoto authored
The field was erroneously named. The field's meaning actually indicates whether enrollment was done through ARC++. It is possible to have multiple enrollments of a single device: one through native CrOS and one through ARC++. The new name is "isArcPlusPlusEnrollment". Bug: 778026, 672263 Change-Id: I0be749e32e1569602efa30c5a9df768d7a45d15c Reviewed-on: https://chromium-review.googlesource.com/811830Reviewed-by:
Jeremy Klein <jlklein@chromium.org> Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#522273}
-
Miguel Casas authored
This CL adds |buffer->data_size()| to the decode TRACE_EVENT() in vpx_video_decoder.cc. The addition helps correlating the time it takes to decode with the encoded buffer size; examples of how it looks like: https://i.imgur.com/LUXijLO.png (https://imgur.com/a/1Gllg) or https://i.imgur.com/67QfgKj.png (https://imgur.com/a/3PhQt) Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I240b6f49d61fa385045e3a67fb4fc1e6e4c96fe0 Reviewed-on: https://chromium-review.googlesource.com/812160Reviewed-by:
Dan Sanders <sandersd@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#522272}
-
Matthew Jones authored
This reverts commit cf0d7c2a. Reason for revert: Fix interface impl bug Original change's description: [Home] Separate swipe detecting logic from the bottom sheet This change moves all of the swipe detecting from the bottom sheet into its own class. A relatively simple interface has been introduced to prevent needing the entire bottom sheet to detect swipes. BUG= TBR=tedchoc@chromium.org Change-Id: Ie77d5c2bcec7874787bd23bf347c6abd8716b492 Reviewed-on: https://chromium-review.googlesource.com/812466Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#522271}
-
Christian Fremerey authored
Reland "Reland "Enable video capture service by default on platforms where it is rolling out with M61."" This is a reland of 0fdb7a80 The reason for the second revert was anouther Webkit layout tests failing with this CL. With https://chromium-review.googlesource.com/c/chromium/src/+/811865 landed to fix the test, this CL should now be able to reland without modification. Original change's description: > Reland "Enable video capture service by default on platforms where it is rolling out with M61." > > This is a reland of 1e2e99ef > > The reason for the revert was a Webkit layout test failing with this CL. With > https://chromium-review.googlesource.com/c/chromium/src/+/806442 landed to fix that test, > this CL should now be able to reland without modification. > > Original change's description: > > Enable video capture service by default on platforms where it is rolling > > out with M61. > > > > Bug: 721812 > > Change-Id: I1de0f625a1fb106e2dea664675e94267e66fc2fd > > Reviewed-on: https://chromium-review.googlesource.com/719416 > > Commit-Queue: Christian Fremerey <chfremer@chromium.org> > > Reviewed-by: Emircan Uysaler <emircan@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#517761} > > TBR=emircan@chromium.org > > Bug: 721812 > Change-Id: If69cdb660b0b97a28084b6bd0610234d5abb232f > Reviewed-on: https://chromium-review.googlesource.com/809530 > Reviewed-by: Christian Fremerey <chfremer@chromium.org> > Commit-Queue: Christian Fremerey <chfremer@chromium.org> > Cr-Commit-Position: refs/heads/master@{#521800} TBR=emircan@chromium.org Bug: 721812 Change-Id: I3287fbfc810caf3d7472c0871a9e0bb73c871841 Reviewed-on: https://chromium-review.googlesource.com/812484 Commit-Queue: Christian Fremerey <chfremer@chromium.org> Reviewed-by:
Christian Fremerey <chfremer@chromium.org> Cr-Commit-Position: refs/heads/master@{#522270}
-
Matt Mueller authored
Also renames / removes some X509Certificate methods to make more sense in the use_byte_certs world: os_cert_handle() -> cert_buffer() GetIntermediateCertificates() -> intermediate_buffers() CreateFromHandle() -> CreateFromBuffer() CreateOSCertHandleFromBytes() -> CreateCertBufferFromBytes() CreateOSCertHandlesFromBytes() -> CreateCertBuffersFromBytes() GetDEREncoded() -> x509_util::CryptoBufferAsStringPiece() IsSameOSCert() -> x509_util::CryptoBufferEqual() DupOSCertHandle() -> x509_util::DupCryptoBuffer() FreeOSCertHandle() -> N/A (everything uses bssl::UniquePtr<CRYPTO_BUFFER>) Bug: 769549 Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I5d94158ea5e7ba5e9e3b9f2f5e76221564c9683b Reviewed-on: https://chromium-review.googlesource.com/775540 Commit-Queue: Matt Mueller <mattm@chromium.org> Reviewed-by:
Lambros Lambrou <lambroslambrou@chromium.org> Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Reviewed-by:
Raymes Khoury <raymes@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Misha Efimov <mef@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Jialiu Lin <jialiul@chromium.org> Reviewed-by:
Ryan Sleevi <rsleevi@chromium.org> Cr-Commit-Position: refs/heads/master@{#522269}
-
Doug Turner authored
In this CL, we create a new implementation of NavigationThrottle, called PolicyBlacklistNavigationThrottle in which we call into a policy::URLBlacklistManager implementation during calls to WillStartRequest(). If the requested URL is in the blocklist, we’d return CANCEL otherwise PROCEED. This CL also introduces a KeyedService for accessing the policy::URLBlacklistManager. Bug: 756642 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo;master.tryserver.chromium.linux:linux_site_isolation Change-Id: I2d9ca0982e78b03be191b67e61eea14764150e68 Reviewed-on: https://chromium-review.googlesource.com/745326 Commit-Queue: Doug Turner <dougt@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#522268}
-
W. James MacLean authored
These files appear to have been missing from the BUILD.gn file, and I suspect causing the try-bots not to run tests on CLs that touch them. Bug: Change-Id: I694b759396838498bca6de3550551f1aedba04e1 Reviewed-on: https://chromium-review.googlesource.com/812109Reviewed-by:
Lucas Gadani <lfg@chromium.org> Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Commit-Queue: James MacLean <wjmaclean@chromium.org> Cr-Commit-Position: refs/heads/master@{#522267}
-
Tim Brown authored
This will be the replacement backup way of getting the setting, rather than using the deprecated gconf. I will remove the gconf code in a separate CL. gsettings app to change values and test behavior in chromium. Test: Force use of gsettings for config (see patchset 3) and used Bug: 768027 Change-Id: I957edc5f03c20f42e74d0ba9d1e57a352cc2d418 Reviewed-on: https://chromium-review.googlesource.com/809730Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Elliot Glaysher <erg@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#522266}
-
Danyao Wang authored
The removed code was originally added to fix a security bug in iOS 9 (crbug.com/610297). The issue seems to be fixed in WKWebView now so this patch is no longer necessary. This is step 2 to removing _lastRegisteredRequestURL from CRWWebController. Bug: 789993,792515,610297 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I2eaf5ec47d156ce71d724752fb6d35001c044b3f Reviewed-on: https://chromium-review.googlesource.com/812174 Commit-Queue: Danyao Wang <danyao@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#522265}
-
Christopher Grant authored
Now that we have element types, we can ditch special-casing and remove a TODO. BUG= R=bshe, ymalik Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I22d00c5722f78da4e3bef92ece25ae3231a67981 Reviewed-on: https://chromium-review.googlesource.com/810926 Commit-Queue: Christopher Grant <cjgrant@chromium.org> Reviewed-by:
Yash Malik <ymalik@chromium.org> Cr-Commit-Position: refs/heads/master@{#522264}
-
Robert Flack authored
Bug: None Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I18af9c1d0486446d5cb321b1376a233e6b652755 Reviewed-on: https://chromium-review.googlesource.com/806679Reviewed-by:
David Reveman <reveman@chromium.org> Reviewed-by:
Ian Vollick <vollick@chromium.org> Commit-Queue: Robert Flack <flackr@chromium.org> Cr-Commit-Position: refs/heads/master@{#522263}
-
Francois Doray authored
The NetworkChangeNotifier must be created after TaskScheduler because it posts tasks to it. Bug: 789442 Change-Id: Ie6066064a5f840666160bac76d95ad16d3363708 Reviewed-on: https://chromium-review.googlesource.com/810925Reviewed-by:
Vitaly Buka <vitalybuka@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#522262}
-
Gyuyoung Kim authored
base/memory/ptr_util.h includes are changed with <memory> as well. Bug: 755727 Change-Id: I12c15e3258153550cd11721b32a76519664ba3e2 Reviewed-on: https://chromium-review.googlesource.com/804612Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Commit-Queue: Gyuyoung Kim <gyuyoung.kim@lge.com> Cr-Commit-Position: refs/heads/master@{#522261}
-
- 06 Dec, 2017 24 commits
-
-
Dominick Ng authored
BUG=None Change-Id: I2608465fce0ca012d906f8190da541770c5b44ef Reviewed-on: https://chromium-review.googlesource.com/809915Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#522260}
-
Ilya Sherman authored
R=rdevlin.cronin@chromium.org Bug: None Change-Id: I921fa487be7e8007ef525e8b7f9602bdb64662f3 Reviewed-on: https://chromium-review.googlesource.com/810095Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#522259}
-
Christopher Grant authored
For now, this capability is gated by the keyboard-enabled flag. BUG= Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I050b9dff9552ce3cff6c82f81296f4657d810ccf Reviewed-on: https://chromium-review.googlesource.com/811064 Commit-Queue: Christopher Grant <cjgrant@chromium.org> Reviewed-by:
Yash Malik <ymalik@chromium.org> Cr-Commit-Position: refs/heads/master@{#522258}
-
Nicolas Pena authored
This CL updates links in some idls so they point to the correct interfaces. Change-Id: I9865ed3c499f8b476b4da1a3acbdc7cd6ae976ea Reviewed-on: https://chromium-review.googlesource.com/802294Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Shubhie Panicker <panicker@chromium.org> Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Cr-Commit-Position: refs/heads/master@{#522257}
-
Maria Khomenko authored
The new metric is called PrivateSwapFootprint. It records the value of VmSwap from /proc/<pid>/status and is available only on Linux and Android. This metric is useful for evaluating changes in swap use with memory ablation. BUG=791802 Change-Id: Ica436198fd00d85cd5f5896ce994547a73aea27f Reviewed-on: https://chromium-review.googlesource.com/807358 Commit-Queue: Maria Khomenko <mariakhomenko@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Primiano Tucci <primiano@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#522256}
-
Darren Shen authored
This patch implements StylePropertyMap.update for CSS properties using get and set. Some tests were wrong because they didn't return a value in the update callback. Spec: https://drafts.css-houdini.org/css-typed-om-1/#dom-stylepropertymap-update Bug: 785132 Change-Id: I6b8a06bfc1dcf0bb35603237d3507d5d3da96182 Reviewed-on: https://chromium-review.googlesource.com/788478Reviewed-by:
nainar <nainar@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#522255}
-
Tom Sepez authored
Consolidate comments in header. Move methods around to match order in header. Call common validation routines in broker_command.h from all places. Change-Id: I5317064bc6784d28586685ebd44802c90398abbb Reviewed-on: https://chromium-review.googlesource.com/812205Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#522254}
-
Ian Kilpatrick authored
The existing code for this LayoutBlock::ComputeBlockPreferredLogicalWidths was actually reasonably good, but due to re-use of variables, and a few things which happened over time made it a little convoluted. This hopefully is easier to follow. Bug: 635619 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I6b80bc2228473108e18c5680783668259a8d91b2 Reviewed-on: https://chromium-review.googlesource.com/784675 Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#522253}
-
Katie Dektar authored
This would catch a regression that happened last week where mousemove events were not passed properly to the background extension, should it re-occur in the future. Bug: 789598 Change-Id: Idd61c1e63f9dfa4f0e87603b62b0cc0a31a34902 Reviewed-on: https://chromium-review.googlesource.com/806647 Commit-Queue: Katie D <katie@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#522252}
-
Evan Stade authored
This reverts commit 6e56b952. Reason for revert: I realized this crashes if a captive portal is detected during login (before there's an active profile). Original change's description: > Chrome OS - Update captive portal notification to use > NotificationDisplayService. > > Test by commenting out most of OnPortalDetectionCompleted() (in addition > to unit tests). > > Bug: 783018 > Change-Id: I403967f28883c588d9c2e5fd43effe05e41f7eda > Reviewed-on: https://chromium-review.googlesource.com/807445 > Reviewed-by: Scott Violet <sky@chromium.org> > Reviewed-by: Steven Bennetts <stevenjb@chromium.org> > Commit-Queue: Evan Stade <estade@chromium.org> > Cr-Commit-Position: refs/heads/master@{#522214} TBR=stevenjb@chromium.org,sky@chromium.org,estade@chromium.org Change-Id: I951a56b8bb5a9c44e6aeed157cb6cbda88e2be93 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 783018 Reviewed-on: https://chromium-review.googlesource.com/812470Reviewed-by:
Evan Stade <estade@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#522251}
-
Charles Harrison authored
There have been chrome traces which indicate this throttle is expensive on Android. We should perform some measurements in the wild to see how often this is true. Bug: 792524 Change-Id: I4b35949e0265c9e3b291d623789cbf4fd32d85f0 Reviewed-on: https://chromium-review.googlesource.com/811828Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#522250}
-
Christopher Grant authored
- If PrepareToDraw indicates a scene change, do layout. - Trigger scene rendering if the cursor moved. - Move a static assert to live with TextInputInfo. - Add more tests. BUG= R=mthiesse, ymalik Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I4f05832610c0d8e0676599bc92b9a926afbbca57 Reviewed-on: https://chromium-review.googlesource.com/809827 Commit-Queue: Christopher Grant <cjgrant@chromium.org> Reviewed-by:
Yash Malik <ymalik@chromium.org> Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Cr-Commit-Position: refs/heads/master@{#522249}
-
Reza.Zakerinasab authored
This change removes ImageData usage from toBlob/toDataURL code path from HTMLCanvasElement calls down to the CanvasAsyncBlobCreator. Bug: 792110, 657870 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I3e222a68767c1a476f09c1b6188699f62449b95a Reviewed-on: https://chromium-review.googlesource.com/809097Reviewed-by:
Justin Novosad <junov@chromium.org> Reviewed-by:
Olivia Lai <xlai@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Mohammad Reza Zakerinasab <zakerinasab@chromium.org> Cr-Commit-Position: refs/heads/master@{#522248}
-
Xianzhu Wang authored
Using the FOR_EACH_XXX(V) pattern, the reason bits are automatically defined without holes or duplicates, with compile-time checks for ranges and validity of the strings array. This makes it easier to add or remove reasons without adjusting the numbers of bits to shift. Also define methods to access the previously exposed data structures. Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Ib88ec47d1cf0ad548f478c30fb40436ba1db12e8 Reviewed-on: https://chromium-review.googlesource.com/811425 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#522247}
-
Dale Curtis authored
Today WTR doesn't report background audio nor anything for video only streams; this change populate UKM with this data. Additionally, audio background data is reported to UMA (because otherwise we'd just lose a large chunk of what we report today), while video only data is just reported to UKM for now. Video only can be enabled later by simply adding the histograms.xml entries and updating the filter in WatchTimeRecorder. All of this is done by essentially removing all restrictions from WatchTimeReporter. We now always construct a background reporter and we always report shown/hidden status regardless of audio/video presence. This will cause a large drop off in Audio listen time since all background time will go into a new background bucket now. While the two metrics should sum to close to the same value, some loss is expected until http://crbug.com/779192 is fixed to roll-up unreported watch time for transition states. BUG=775731,775685,775681 TEST=expanded existing tests to cover all functionality. Change-Id: I55eb69b074d00750b5bd9ed47585380f92c882ca Reviewed-on: https://chromium-review.googlesource.com/809620Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Chrome Cunningham <chcunningham@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#522246}
-
Sergey Ulanov authored
This is a reland of eec981f1 Original change's description: > Add unittests for AudioOutputStream. > > Previosly there were no cross-platforms tests for AudioOutputStream. > This CL adds new tests that will run on all platforms. The tests > are based on Windows-specific tests in media/audio/win. > > Bug: 737802 > Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel > Change-Id: I2accac0d7dc5e5d5ac813923ec6b83c0af6c9da6 > Reviewed-on: https://chromium-review.googlesource.com/702688 > Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> > Reviewed-by: Olga Sharonova <olka@chromium.org> > Reviewed-by: Dale Curtis <dalecurtis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#509522} Bug: 737802 Change-Id: Ia3b7953437b484f37a339a90e3db9af1f315bdd9 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Reviewed-on: https://chromium-review.googlesource.com/809540Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#522245}
-
Devlin Cronin authored
The Finch Features API (base::Feature and base::FeatureList) now satisfy all the requirements that FeatureSwitch needed to (the ability to toggle from command line, runtime flag, or field trial). The two FeatureSwitch items that used the Field Trial API (yield_between_content_scripts and native_crx_bindings) have been migrated to the Features API. Remove FeatureSwitch support for field trials. Bug: 784476 Bug: 585569 Change-Id: I265b5f687707aeee2e7daf2f45df44ab094e479c Reviewed-on: https://chromium-review.googlesource.com/811290Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#522244}
-
Devlin Cronin authored
Add a common ScriptContext getter to retrieve a ScriptContext from a given v8::Context that works across threads. This previously existed, but only in native_extension_bindings_system.cc. Pull it out to a common file to a) make it reusable from other consumers (custom hooks will need to use this) and b) remove (direct) threading knowledge from the bindings system. Bug: 653596 Change-Id: I09a0e4ea76bb891ab1871ca1deb0d315e6e9789d Reviewed-on: https://chromium-review.googlesource.com/812173Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#522243}
-
Steven Bennetts authored
Bug: 792641 Change-Id: I6fe3df805cf2efcfded5c3633d07f4931314864c Reviewed-on: https://chromium-review.googlesource.com/809931Reviewed-by:
Wenzhao (Colin) Zang <wzang@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#522242}
-
Jay Civelli authored
Changing names in PDFToPWGRasterConverterService related files from all caps to the style guide preferred way: PDF -> Pdf, PWG -> pwg. Bug: 766451 Change-Id: I6e3329d218a98917bf0a2474d46612dcf91976db Reviewed-on: https://chromium-review.googlesource.com/811450 Commit-Queue: Jay Civelli <jcivelli@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#522241}
-
Peter Kasting authored
Bug: 789931, 789947 TBR: zea, qinmin Change-Id: I340a4dd9b3b1176d8bafe3a21f52e26522c41ebe Reviewed-on: https://chromium-review.googlesource.com/811990 Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#522240}
-
Alexei Filippov authored
Bug: Change-Id: I55707cbd5cfd5100b72ad6f4c2cd8195ba927b8d Reviewed-on: https://chromium-review.googlesource.com/809497 Commit-Queue: Alexei Filippov <alph@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#522239}
-
Tibor Goldschwendt authored
Due to a merge mistake the VLOG for failed loading will never be executed. Fix this. Change-Id: I37fe4e2a72e22b1b84eb1fcb8d791371d883ab13 Reviewed-on: https://chromium-review.googlesource.com/812210Reviewed-by:
Christopher Grant <cjgrant@chromium.org> Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org> Cr-Commit-Position: refs/heads/master@{#522238}
-
Marc-Antoine Ruel authored
https://chromium.googlesource.com/infra/luci/client-py.git/+log/6fd3c7b6eb7c..4bd9152f8a97 $ git log 6fd3c7b6e..4bd9152f8 --date=short --no-merges --format='%ad %ae %s' 2017-12-05 maruel client/tools: add reproduce.py 2017-12-04 maruel client: isolated unicode fix for Windows 2017-12-04 maruel client: isolated unicode fix on Ubuntu 2017-12-04 maruel tests: more unicode 2017-12-01 maruel client: Fix emoji handling at various places 2017-12-01 maruel example: put tons of emojis in the examples 2017-11-29 maruel subprocess42: be more tolerant to unicode. 2017-11-29 maruel Fix regression in fc3c34e59202ee and bfab799cddab1 2017-11-28 maruel swarming_bot: migrate env mutation from task_runner to run_isolated. 2017-11-23 huangwe Allow None response when the item size is 0 in gRPC isolate push 2017-11-22 iannucci [swarming] Add env_prefixes TaskProperty. 2017-11-20 huangwe Raise gRPC error when gRPC calls for pushing blobs return None 2017-11-13 maruel client: fix missing lock in DiskCache.cleanup() 2017-11-08 vadimsh Refactor httpserver_mock in attempt to get rid of deadlocks. 2017-11-08 vadimsh Add minimal support for Cavium CPUs. Created with: roll-dep src/tools/swarming_client TBR=iannucci@chromium.org Bug: Change-Id: Id39cf767dc415f16c3c08e7bf8340b58a56834f5 Reviewed-on: https://chromium-review.googlesource.com/811205Reviewed-by:
Marc-Antoine Ruel <maruel@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org> Cr-Commit-Position: refs/heads/master@{#522237}
-