- 08 May, 2018 40 commits
-
-
Hwanseung Lee authored
We should switch from DOMString to CSSOMString as resolved in CSSWG resolved to use CSSOMStrings[1]. and there is idl issue when union type have set of typedef value. so this case did not change and will follow up (https://crbug.com/838890) [1]https://github.com/w3c/css-houdini-drafts/issues/687#issuecomment-379669504 Bug: 834164 Change-Id: Id4768a6edbcdc17bfa72cffd4a63c81d8124d256 Reviewed-on: https://chromium-review.googlesource.com/1034618 Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com> Reviewed-by:
Darren Shen <shend@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#556760}
-
Marina Ciocea authored
Remove audio thread reset before ScopedAllowBlocking. Bug: 672469 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: I8e0ca007d87aa3dc1a7b520cdc8e6976e79e3448 Reviewed-on: https://chromium-review.googlesource.com/1049665Reviewed-by:
Olga Sharonova <olka@chromium.org> Commit-Queue: Olga Sharonova <olka@chromium.org> Cr-Commit-Position: refs/heads/master@{#556759}
-
Alexander Hendrich authored
Added a policy to limit the user's choices for their main OS language. The user can only select one of the provided languages in this list. If the current user's language is not allowed, the user's preferred languages are checked for an allowed entry. Otherwise the language is set to the first valid entry in this policy or to 'en-US' as default. Bug: 695042 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I5c17868a97339ec32b04483734b68843ec26d2a8 Reviewed-on: https://chromium-review.googlesource.com/1034056Reviewed-by:
Pavol Marko <pmarko@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Alexander Hendrich <hendrich@chromium.org> Cr-Commit-Position: refs/heads/master@{#556758}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 9fdaf87d. With Chromium commits locally applied on WPT: 1f468ca9 "Reland "Adds a test for basic WebRTC video codec conformance."" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/17156 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md TBR=danyao No-Export: true Change-Id: I8a0e76f6ac9bf829e45c9f94b163a3fb9ac407a1 Reviewed-on: https://chromium-review.googlesource.com/1049368 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#556757}
-
Ramin Halavati authored
New binaries of traffic annotation auditor and extractor are rolled for Windows and Linux. TBR=msramek@chromium.org Bug: 690323 Change-Id: If51ad7baf059c3b727034ec73fb2c7100fc8b298 Reviewed-on: https://chromium-review.googlesource.com/1049653Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Martin Šrámek <msramek@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#556756}
-
Anita Woodruff authored
- This is the follow-up CL to add tests for https://crrev.com/1035267, which added code to run a one-time reset of all blocked site channels. - The new tests are similar to the existing tests for the one-time migration to site channels. - For testing the DSE exception conditions, it was necessary to set up a fake TemplateURLService - did this by copying the approach of CreateTemplateURLService in contextual_search_delegate_unittest.cc to initialize it with a particular default search engine. R=peter@chromium.org Bug: 835232 Change-Id: I2f61e8a48997a0685e0afaa38d33caaaadecb188 Reviewed-on: https://chromium-review.googlesource.com/1044194 Commit-Queue: Anita Woodruff <awdf@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#556755}
-
afdo-chromium-autoroll authored
This CL may cause a small binary size increase, roughly proportional to how long it's been since our last AFDO profile roll. For larger increases (around or exceeding 100KB), please file a bug against gbiv@chromium.org. Additional context: https://crbug.com/805539 The AutoRoll server is located here: https://afdo-chromium-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=gbiv@chromium.org Change-Id: I7edee97c51e9758cb680ed9f0be8e829116597d9 Reviewed-on: https://chromium-review.googlesource.com/1049173 Commit-Queue: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#556754}
-
Ilya Nikolaevskiy authored
This CL adds origin trial implementation and forwards it to RtcPeerConnectionHander. The flag should be forwarded further down to actual video codec selection in the later CL. Intention to experiment: https://groups.google.com/a/chromium.org/forum/#!searchin/blink-dev/vp8/blink-dev/vbYCDv5ve5w/pq-uV_QRAwAJ Bug: 794608 Change-Id: I5c50a2b402bea720b57e8ebb2a36a261fb512743 Reviewed-on: https://chromium-review.googlesource.com/1030173 Commit-Queue: Ilya Nikolaevskiy <ilnik@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Cr-Commit-Position: refs/heads/master@{#556753}
-
Rohit Rao authored
Also moves the implementation of BookmarkTableCellTitleEditDelegate to BookmarkHomeViewController, as that was only referenced by one of the data source methods. Makes several BookmarkTableView methods public in order to ease migration. BUG=840381 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia4e1f41ed76c44a763771799b0ec35da90d8ef21 Reviewed-on: https://chromium-review.googlesource.com/1048073 Commit-Queue: Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#556752}
-
Yutaka Hirano authored
ReadableStreamBytesConsumer expected that the results from ReadableStreamReaderDefaultRead should be Promise<Object> because that is provided from ReadableStream provided by blink, but it's possible to inject arbitrary values with the promise assimilation. This CL adds additional checks for such injection. Bug: 840320 Change-Id: I7b3c6a8bfcf563dd860b133ff0295dd7a5d5fea5 Reviewed-on: https://chromium-review.googlesource.com/1049413 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Adam Rice <ricea@chromium.org> Cr-Commit-Position: refs/heads/master@{#556751}
-
Guido Urdaneta authored
TBR=catmullings@chromium.org Bug: 801316 Change-Id: I92df8309043b434d0465a1b098ddf63da0ef27a8 Reviewed-on: https://chromium-review.googlesource.com/1049646Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#556750}
-
Kent Tamura authored
- Avoid global variables Wrap the whole code with 'main()' function - Don't use obsolete 'string' module - Don't add parenthesis to assert. - Remove trailing whitespace - Remove unused variable 'saved_by_reusing' - Remove whitespace around a keyword argument =. This CL has no behavior changes. Bug: 838772 Change-Id: Ib79344a4c5a671c24a528ed918156e22b0d2c65d Reviewed-on: https://chromium-review.googlesource.com/1049407Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#556749}
-
Jan Wilken Doerrie authored
This change introduces BluetoothAdapterWinrt, which will contain an adapter implementation using Mirosoft UWP BLE APIs. This change merely provides a skeleton, where each member function is NOTIMPLEMENTED(). Further functionality will be provided in future CLs. Bug: 579202, 821766 Change-Id: I207e14de292f9d24b7d16710437e0dae839c82ec Reviewed-on: https://chromium-review.googlesource.com/1046589 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#556748}
-
Harald Alvestrand authored
TBR=ksakamoto@chromium.org Bug: chromium:840659 Change-Id: Id25d5f8678296cf0e797fc668cdf384e01354433 Reviewed-on: https://chromium-review.googlesource.com/1049645Reviewed-by:
Harald Alvestrand <hta@chromium.org> Reviewed-by:
Kunihiko Sakamoto <ksakamoto@chromium.org> Commit-Queue: Harald Alvestrand <hta@chromium.org> Cr-Commit-Position: refs/heads/master@{#556747}
-
Charlie Andrews authored
AFAICT, this class is completely unused. Change-Id: I7c6ddf03e6094515833c34a3550457fd8a6b6471 Reviewed-on: https://chromium-review.googlesource.com/1049027 Commit-Queue: Ned Nguyen <nednguyen@google.com> Reviewed-by:
Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#556746}
-
Tommy Steimel authored
This CL changes the CSS to align the current time and duration with the newly-positioned media controls buttons. Bug: 840255 Change-Id: I1a55eeb032ba8f2b968b8bf04fda96396f573bab Reviewed-on: https://chromium-review.googlesource.com/1048029Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#556745}
-
Guido Urdaneta authored
TBR=finnur@chromium.org Bug: 792313 Change-Id: Ia48dbdf710a3898fe1393404402295884d8e9c98 Reviewed-on: https://chromium-review.googlesource.com/1049554Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#556744}
-
Fredrik Söderquist authored
From using AtomicString (for 'mask', 'marker-*', 'fill' and 'stroke') or the string representation of SVGResources ('filter' and 'clip-path'), convert to using SVGResources. Add an implementation of SVGResourceClient (SVGElementResourceClient) and register it with the existing resources. Invalidation code is moved here. This makes the invalidation code slightly more homogeneous between the various code-paths. The new SVGResourceClient implementation is stored in SVGElementRareData as needed. The various computed style representations are converted to use the new StyleSVGResource, which is a ref-counted wrapper around an SVGResource and its underlying string representation (which is needed for serialization.) Style building is changed to produce this new wrapper (rather than an AtomicString.) A helper for serialization is added to ComputedStyleUtils. The kHasPendingResources flag is removed since it no longer used, and the hook in Element is removed. All LayoutSVGResource* subclasses are changed to take SVGResourceClients instead of LayoutObjects, and the |clients_| set is removed in favor of the set in SVGResource. A bunch of helpers are added for updating (adding/removing) the client registrations. The various LayoutSVG subclasses are updated to call these helpers. TBR=junov@chromium.org Bug: 769774 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I62418465251532240c4e895e5c4d253ed9e53b9a Reviewed-on: https://chromium-review.googlesource.com/977965 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:Fredrik Söderquist <fs@opera.com> Reviewed-by:
Philip Rogers <pdr@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#556743}
-
Joel Hockey authored
Run python tools/metrics/histograms/update_extension_histograms.py to fix enums.xml. I don't believe that these features would have had (much) use, so it should be safe to fix this. ACCESSIBILITY_PRIVATE_ONSELECTTOSPEAKSTATECHANGEREQUESTED was inadvertantly added in http://crosreview.com/1030901, problem compounded a little in http://crosreview.com/1043666 Change-Id: I9f6b2b52fe97ff4c7bb75f079158d9ae4455d319 Tbr: isherman@chromium.org Reviewed-on: https://chromium-review.googlesource.com/1049349Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#556742}
-
Noel Gordon authored
Enabled in RELEASE, now re-enable this test in DEBUG/MSAN/ASAN. Bug: 835626 Change-Id: Ie214d9ef7241ab084772010b2a854e50ef051b67 Reviewed-on: https://chromium-review.googlesource.com/1049347Reviewed-by:
Naoki Fukino <fukino@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#556741}
-
Christian Dullweber authored
When urls are removed from chrome://history, the site-engagement score is not removed if it reaches zero. This CL fixes the issue as it is not neccessary to keep a score of zero around. Bug: 838875 Change-Id: Ia580ee49f2c4df8cda46f33c36b6551580dafef7 Reviewed-on: https://chromium-review.googlesource.com/1046765Reviewed-by:
calamity <calamity@chromium.org> Commit-Queue: Christian Dullweber <dullweber@chromium.org> Cr-Commit-Position: refs/heads/master@{#556740}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/e3f3bcb2..9ef8a461 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_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;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org Change-Id: I1c65bbcf6ce40050b917cb288293b9226d1563ab Reviewed-on: https://chromium-review.googlesource.com/1049367Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#556739}
-
Patrik Höglund authored
I followed the conventions in chromium.linux at ci.chromium.org. Bug: chromium:840277 Change-Id: I7dd757c00f6e4450d1070ebc503f0d8cdd6887e4 Reviewed-on: https://chromium-review.googlesource.com/1046565 Commit-Queue: Patrik Höglund <phoglund@chromium.org> Reviewed-by:
Ryan Tseng <hinoka@chromium.org> Cr-Commit-Position: refs/heads/master@{#556738}
-
Jinsuk Kim authored
WindowEventObserverManager is a new class that will take care of all window-related events. - WindowAndroid is now updated through WebContents, and then ContentObserverManager to trigger observer events. - More work have been moved to actual observers (GestureListenerManager, SelectionPopupController, SelectPopup, etc) Bug: 598880 Change-Id: Ie4b9b209fe3bf67373fd420799effea210e93e5d Reviewed-on: https://chromium-review.googlesource.com/1018640 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#556737}
-
Mike Frysinger authored
The default xterm doesn't provide us with extra fun colors, so update the default to allow that. This keeps us in sync w/hterm too. Bug: 800049 Test: compiled & ran it Change-Id: I648ec77ab5adc8542d2673848f666e5e8568db8b Reviewed-on: https://chromium-review.googlesource.com/905264Reviewed-by:
Toni Barzic <tbarzic@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org> Cr-Commit-Position: refs/heads/master@{#556736}
-
Renee Wright authored
and remove the old Polymer dialogue Bug: 822504 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I30bf627099d257363bb4cd8177c07fbd147974ec Reviewed-on: https://chromium-review.googlesource.com/1037043 Commit-Queue: Renée Wright <rjwright@chromium.org> Reviewed-by:
Nicholas Verne <nverne@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#556735}
-
Erik Luo authored
The new setting is in the toolbar behind the gear. Screenshot: https://imgur.com/a/09myIJH Bug: 810176 Change-Id: Ifbd2a2bfc6712e2f2415e384c040f5520b1e2347 Reviewed-on: https://chromium-review.googlesource.com/1045180 Commit-Queue: Erik Luo <luoe@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#556734}
-
Magnus Jedvert authored
We currently use '@JNINamespace' everywhere in WebRTC for setting the WebRTC namespace, but it would be cleaner to do this from the build file and the python script already has support for it, we just need to expose it. Bug: webrtc:8278 Change-Id: Ibc0529f353572225f0d677dbf7ab0e5883fa197d Reviewed-on: https://chromium-review.googlesource.com/1044191Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: Magnus Jedvert <magjed@chromium.org> Cr-Commit-Position: refs/heads/master@{#556733}
-
Chromium WPT Sync authored
Using wpt-import in Chromium ae6a05c5. With Chromium commits locally applied on WPT: ae6a05c5 "webrtc wpt: pass test in promise_test and async_test" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/17155 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md TBR=qyearsley No-Export: true Change-Id: Iee13af32481de808478ed3946eb4bea31e038382 Reviewed-on: https://chromium-review.googlesource.com/1049565 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#556732}
-
Harald Alvestrand authored
This reverts commit 8d1dda31. Reason for revert: Now disabling test on relevant platforms. Original change's description: > Revert "Adds a test for basic WebRTC video codec conformance." > > This reverts commit 72aef1a4. > > Reason for revert: Failing on Win / Mac / Linux bots > > Unexpected Failures: > * external/wpt/webrtc/protocol/video-codecs.html > * virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/protocol/video-codecs.html > > WebKit Win7 https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Win7/63947 > WebKit Win10 https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Win10/34576 > WebKit Mac10.10 https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Mac10.10/46846 > WebKit Mac10.11 https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Mac10.11/32883 > WebKit Linux Trusty https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty/44403 > > (It passes on Mac 10.12.) > > Original change's description: > > Adds a test for basic WebRTC video codec conformance. > > > > This tests that the browser conforms to RFC 7742, "WebRTC Video > > Processing and Codec Requrements". > > > > It establishes a new subdirectory for protocol-related WebRTC tests. > > > > Bug: chromium:839753 > > Change-Id: Ia4795c4532fb74ae76e6a47329a41807479113c5 > > Reviewed-on: https://chromium-review.googlesource.com/1043886 > > Commit-Queue: Harald Alvestrand <hta@chromium.org> > > Reviewed-by: Philip Jägenstedt <foolip@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#556454} > > TBR=mflodman@webrtc.org,hta@chromium.org,foolip@chromium.org > > Change-Id: Id3789b69df63d7e41be3573ca1fbb5a07b00e7e7 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: chromium:839753 > Reviewed-on: https://chromium-review.googlesource.com/1049088 > Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org> > Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org> > Cr-Commit-Position: refs/heads/master@{#556645} TBR=mflodman@webrtc.org,hta@chromium.org,ksakamoto@chromium.org,foolip@chromium.org Change-Id: I867cf04bfc7cd48afea3b5a26f0af77eef731434 Bug: chromium:839753, chromium:840659 Reviewed-on: https://chromium-review.googlesource.com/1049525Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Harald Alvestrand <hta@chromium.org> Commit-Queue: Harald Alvestrand <hta@chromium.org> Cr-Commit-Position: refs/heads/master@{#556731}
-
Alex Kalugin authored
My change from here is inappropriate: https://chromium-review.googlesource.com/c/chromium/src/+/1046966 shlex.split in non-posix mode just splits words in whitespaces, so '-D"MY_DEFINE=\"MY VALUE\""' is still split in two arguments '-DMY_DEFINE="MY' and 'VALUE"'. This did not caused problems for me right away because after joining split_command into one string the command line happens to be properly quoted. In fact shlex.split properly splits '-D"MY_DEFINE=\"MY VALUE\""', we just have to reescape split_command instead of just joining it as string. Change-Id: I7cdbf47946d1a114d7c787cf9dee897224f2fc09 Reviewed-on: https://chromium-review.googlesource.com/1049548Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Kalugin Alexander <akalugin@yandex-team.ru> Cr-Commit-Position: refs/heads/master@{#556730}
-
Alexander Alekseev authored
Bug: 835784 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Id3be7b118eb36749bb65e07d6d96e7c03bc52c1c Reviewed-on: https://chromium-review.googlesource.com/1049010Reviewed-by:
Wenzhao (Colin) Zang <wzang@chromium.org> Commit-Queue: Alexander Alekseev <alemate@chromium.org> Cr-Commit-Position: refs/heads/master@{#556729}
-
Gyuyoung Kim authored
|WebMemoryCoordinator::OnPurgeMemory| has been only called in |RenderThreadImpl::OnPurgeMemory|, and |RenderThreadImpl::OnPurgeMemory| is only able to be called by |base::MemoryCoordinatorClient|. But, if there is no performance degradation, it would be good to purge used memory as much as possible. |RenderThreadImpl::ReleaseFreeMemory| has been more called than |RenderThreadImpl::OnPurgeMemory| from some callers. So we can free up more memory if we move |WebMemoryCoordinator::OnPurgeMemory| from |RenderThreadImpl::OnPurgeMemory| to |RenderThreadImpl::ReleaseFreeMemory|. TEST: Covered RenderThreadImplDiscardableMemoryBrowserTest.ReleaseFreeMemory Bug: None Change-Id: I225401bcd18f236265305aaf5eaf4b4219d9cde4 Reviewed-on: https://chromium-review.googlesource.com/1049414Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Gyuyoung Kim <gyuyoung.kim@lge.com> Cr-Commit-Position: refs/heads/master@{#556728}
-
Felix Ekblom authored
The new method will offer a crude way to query about the power state of connected HDMI CEC capable displays. BUG=b:78171027 Change-Id: Ic37abea8f300d1d22a80368acbd3f129cc5e0187 Reviewed-on: https://chromium-review.googlesource.com/1042575Reviewed-by:
Dan Erat <derat@chromium.org> Reviewed-by:
Toni Barzic <tbarzic@chromium.org> Commit-Queue: Felix Ekblom <felixe@chromium.org> Cr-Commit-Position: refs/heads/master@{#556727}
-
Tetsui Ohkubo authored
The handling of RequestToggleWifi and wifi toggle notification was done in TrayNetwork, which is a SystemTrayItem. We're going to get rid of SystemTrayItems as UnifiedSystemTray will replace them, so this should be a NotificationController. TEST=WifiToggleNotificationController BUG=837507 Change-Id: I6693db6df2c8ab2fcf8f7954adcd61a48e00f24d Reviewed-on: https://chromium-review.googlesource.com/1046289 Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by:
Yoshiki Iguchi <yoshiki@chromium.org> Cr-Commit-Position: refs/heads/master@{#556726}
-
Blake O'Hare authored
Re-arrange the return statement to fix this. Bug: 840646 Change-Id: I1c550de4433f59a70ab04b430d3d2c95e87e5d7c Reviewed-on: https://chromium-review.googlesource.com/1049411Reviewed-by:
Yuichiro Hanada <yhanada@chromium.org> Commit-Queue: Blake O'Hare <blakeo@chromium.org> Cr-Commit-Position: refs/heads/master@{#556725}
-
Noel Gordon authored
Enabled in RELEASE, now re-enable this test in DEBUG/MSAN/ASAN. Bug: 835626 Change-Id: I848da4a1571db55d1bbf4795eaa469ff564a55e2 Reviewed-on: https://chromium-review.googlesource.com/1049416Reviewed-by:
Naoki Fukino <fukino@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#556724}
-
Jochen Eisinger authored
R=mkwst@chromium.org,clamy@chromium.org Change-Id: I1d32699940108e69b116449309a50fae8e07c1bb Reviewed-on: https://chromium-review.googlesource.com/1049387Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#556723}
-
Noel Gordon authored
Enabled in RELEASE, now re-enable this test in DEBUG/MSAN/ASAN. Bug: 835626 Change-Id: I404db91904dab92834a7efca5a567b2ad66d84e4 Reviewed-on: https://chromium-review.googlesource.com/1049474Reviewed-by:
Naoki Fukino <fukino@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#556722}
-
Noel Gordon authored
Enabled in RELEASE, now re-enable this test in DEBUG/MSAN/ASAN. Bug: 835626 Change-Id: Ifd9d394f5cf2f4dd25146e0a0422b7bcd00b4103 Reviewed-on: https://chromium-review.googlesource.com/1049415Reviewed-by:
Naoki Fukino <fukino@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#556721}
-