- 14 Jun, 2018 40 commits
-
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/7d4d6b816c28..b57a861f183b git log 7d4d6b816c28..b57a861f183b --date=short --no-merges --format='%ad %ae %s' 2018-06-14 angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/angle2 4cc753e01054..d668be9ce66f (1 commits) Created with: gclient setdep -r src/third_party/skia@b57a861f183b The AutoRoll server is located here: https://autoroll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=csmartdalton@chromium.org Change-Id: I749d5e987c923bf960ff3ee33cf3da6cd587b372 Reviewed-on: https://chromium-review.googlesource.com/1100696Reviewed-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@{#567218}
-
Yuichiro Hanada authored
This CL also removes obsolete entries: - bshe@ (no longer working on VK) - bryeung@ and kevers@ (no longer in Chrome team) - sadrul@ (already the owner of the parent directory) Bug: None Change-Id: Ia49203ff7fd65a22fe1fa9aff0b38e8bf5257337 Reviewed-on: https://chromium-review.googlesource.com/1098433 Commit-Queue: Yuichiro Hanada <yhanada@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#567217}
-
David Roger authored
Site isolation for Gaia is now launched at 100%, and we can remove the codepath that was used when site isolation was disabled. In particular, after https://chromium-review.googlesource.com/c/chromium/src/+/1096664 the GaiaOriginIsolatedCallback always returns true, and can be removed. TBR=blundell Change-Id: If41007a3fc0b94c58a56f8f58a35f034861ae015 Reviewed-on: https://chromium-review.googlesource.com/1100466 Commit-Queue: David Roger <droger@chromium.org> Reviewed-by:
Thomas Tangl <tangltom@chromium.org> Cr-Commit-Position: refs/heads/master@{#567216}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/b4ddfe4f..d791a8f5 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: I40d2a424c71adcc34382fbf35ce543949f150a33 Reviewed-on: https://chromium-review.googlesource.com/1100681Reviewed-by:
V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Cr-Commit-Position: refs/heads/master@{#567215}
-
Stefan Zager authored
Conversion between 'frame' and 'contents' coordinate spaces is now a no-op. This CL settles on 'frame' as the term d'art for the coordinate space of the frame, so ViewportToContents becomes ViewportToFrame, and vice versa. Also, this adds a couple of missing float overrides for still-useful conversion methods. Arbitrarily converting between LayoutUnit and float can be a source of bugs. BUG=823365 R=skobes@chromium.org,bokan@chromium.org Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Id687f88988e6e27ea75027026377d9464bee3406 Reviewed-on: https://chromium-review.googlesource.com/1099996 Commit-Queue: Stefan Zager <szager@chromium.org> Reviewed-by:
Steve Kobes <skobes@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#567214}
-
Mikel Astiz authored
Prior to this patch, TabContentsSyncedTabDelegate was itself a tab helper (WebContentsUserData) that was created on all platforms, with the whole purpose to reuse a partial implementation of sync_sessions::SyncedTabDelegate. This was a bit weird because a) The life cycle diverges across platforms: created lazily on Android, early otherwise. b) On Android there was no real need to register a tab helper. c) On Android, it's weird to reason about two tab delegates coexisting, one of which is a proxy but also overrides some logic. Instead, let's make TabContentsSyncedTabDelegate a base class that is *NOT* a WebContentsUserData, and let platform-specific subclasses decide whether they want to register as WebContentsUserData, as well as which logic they need to override. This also removes code that is dead on some platforms, because the base class is abtract and doesn't implements all methods (as opposed to a proxy object which cannot be abstract). Bug: 851905 Change-Id: I305ad85e2392fdd2ce428c0464b7750ac599babf Reviewed-on: https://chromium-review.googlesource.com/1097405 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#567213}
-
Tatiana Gornak authored
TBR=dtrainor@chromium.org Bug: 852073 Change-Id: I59017409e79bcddce615de5a820f1424f34eedbc Reviewed-on: https://chromium-review.googlesource.com/1098917Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Tatiana Gornak <melandory@chromium.org> Commit-Queue: Tatiana Gornak <melandory@chromium.org> Cr-Commit-Position: refs/heads/master@{#567212}
-
Sylvain Defresne authored
Those fields are never accessed after initialization (except for destruction). Remove the fields also track preferences that are unused on iOS (they are related to managed accounts which is not supported on iOS). Bug: none Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I13f86b143d34fd0ba07d4372ff0e37133e0e2213 Reviewed-on: https://chromium-review.googlesource.com/1097334 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#567211}
-
Findit authored
This reverts commit 6bc98f91. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 566961 as the culprit for failures in the build cycles as shown on: https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzZiYzk4ZjkxODRkYjQzZmFlYWE5N2YwODZhMTFlYmEyNGZkMWMxZjkM Sample Failed Build: https://ci.chromium.org/buildbot/chromium.memory/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/27907 Sample Failed Step: unit_tests Original change's description: > Add server token revocation in AccountManager > > Bug: 820046 > Test: chromeos_unittests --gtest_filter="*AccountManager*" > Test: unit_tests --gtest_filter="*CrOSOAuthDelegateTest*" > Change-Id: I017bb9e9f9ed789d77235336be8fa465fd4289b8 > Reviewed-on: https://chromium-review.googlesource.com/1087455 > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Reviewed-by: Lutz Justen <ljusten@chromium.org> > Reviewed-by: David Roger <droger@chromium.org> > Commit-Queue: Kush Sinha <sinhak@chromium.org> > Cr-Commit-Position: refs/heads/master@{#566961} Change-Id: Ie71a2d159ed50aa3196bf43b488877747b1f88b8 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 820046 Reviewed-on: https://chromium-review.googlesource.com/1100515 Cr-Commit-Position: refs/heads/master@{#567210}
-
David Roger authored
The guard was present in the .h, but not in the .cc. This CL adds the misssing guard in the .cc. Change-Id: I8716cbad52dd2f578cf730b41118ebbb554fc480 Reviewed-on: https://chromium-review.googlesource.com/1100470Reviewed-by:
-
Kush Sinha authored
This reverts commit 94c42a4f. Reason for revert: To unblock https://chromium-review.googlesource.com/c/chromium/src/+/1100515 Original change's description: > Code Cleanup: Dedup usages of AccountManager's URL Context > > |InlineLoginHandlerChromeOS| and |ChromeOSOAuth2TokenServiceDelegate| > use the system URL context because they 'know' that AccountManager is > going to use the same. This is a leaky abstraction. > > Add a method in AccountManager that returns its URL Request Context. > This request context happens to be the system URL context for now. > > Bug: 820046 > Change-Id: I668769167893c995f73df63acc575eb410c0f986 > Reviewed-on: https://chromium-review.googlesource.com/1088617 > Commit-Queue: Kush Sinha <sinhak@chromium.org> > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Reviewed-by: Lutz Justen <ljusten@chromium.org> > Cr-Commit-Position: refs/heads/master@{#566998} TBR=xiyuan@chromium.org,ljusten@chromium.org,sinhak@chromium.org Change-Id: I57c56e1cde479c6544f93c42061cfc6eec08254e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 820046 Reviewed-on: https://chromium-review.googlesource.com/1100775Reviewed-by:
Kush Sinha <sinhak@chromium.org> Commit-Queue: Kush Sinha <sinhak@chromium.org> Cr-Commit-Position: refs/heads/master@{#567208}
-
Michael Lippautz authored
Bug: chromium:847679 Change-Id: I36061d52a69fb89ce3d4a224904be39c82c40d59 Reviewed-on: https://chromium-review.googlesource.com/1098746 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#567207}
-
Hajime Hoshi authored
This CL also marks WebThreadScheduler::DefaultTaskRunner main-thread only. Change-Id: Ia2b8d962c78a4cf426c1ecc1010d728b58c45e4a Reviewed-on: https://chromium-review.googlesource.com/1100603Reviewed-by:
Yuta Kitamura <yutak@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#567206}
-
Makoto Shimazu authored
Bug: 852202, 669772, 806658 Change-Id: Ife9c41fbe08c1f414bb5459cba29d98d7e1fb892 Reviewed-on: https://chromium-review.googlesource.com/1098650 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#567205}
-
Adam Rice authored
The Blink rename changed the WebSocket handshake files from WebSocketHandshake* to web_socket_handshake*. Since WebSocket is one word, these should properly be websocket_handshake*. Fix them. BUG=834596 Change-Id: I2009a679c948ea460713890ed548e42d797b290c Reviewed-on: https://chromium-review.googlesource.com/1098743Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
-
Maksim Ivanov authored
Add plumbing that allows to skip the "manual password entry" logic when the smart card SAML sign-in is used. Without this, the user would be asked to type their password at the end of their HTTPS-client-certificate-based SAML sign-in. For this, the gaia sign-in screen JS code will ask the C++ handler whether the user is expected to have a password. In this CL, the handler is just a stub. It will be replaced later with an actual smart card sign-in logic, once it's implemented. No changes in behavior are expected from this CL alone. BUG=chromium:826417 TEST=none (existing tests) Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I44012f02fade8823556bcf1200a122b0791d8292 Reviewed-on: https://chromium-review.googlesource.com/1097406Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#567203}
-
Ramin Halavati authored
LoginPromptBrowserTest.CancelLoginInterstitialOnRedirect fails occasionally on Mac 10.12. Disabling it. Bug: 852703 Change-Id: I3ac9fac42fdb1da52a21ccf27049889693136e20 Reviewed-on: https://chromium-review.googlesource.com/1100467Reviewed-by:
Vaclav Brozek <vabr@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#567202}
-
Ramin Halavati authored
BrowserTest.ReattachDevToolsWindow occasionally fails on Mac 10.11. Bug: 852697 Change-Id: Ibd94a9b94d7963825c42dbe50fb8c72097409da0 TBR: ellyjones@chromium.org Reviewed-on: https://chromium-review.googlesource.com/1100465Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#567201}
-
Sylvain Defresne authored
Bug: 798408 Change-Id: I6983b0468743607e8dd2a8722885a4861aaba8b2 Reviewed-on: https://chromium-review.googlesource.com/1096772Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#567200}
-
Alexander Hendrich authored
Fix1: If either AllowOnlyPolicyNetworksToConnect or AllowOnlyPolicyNetworksToAutoconnect is enabled, this CL will additionally remove the network configurations (AllowOnlyPolicyNetworksToConnect) or disable auto-connect (for AllowOnlyPolicyNetworksToAutoconnect) for the unmanaged networks to prevent Shill from re-connecting when searching for a best service to connect. Fix2: Fixed missing disconnect, when AllowOnlyPolicyNetworksToConnect gets enabled. This disconnect was incorrectly prevented by the |applied_autoconnect_policy_| boolean before, which ensures that the disconnect on AllowOnlyPolicyNetworksToAutoconnect only happens once. Bug: 850550 Change-Id: I869ed49d58adb82707e8856e896c5a141f6f6625 Reviewed-on: https://chromium-review.googlesource.com/1091051Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Alexander Hendrich <hendrich@chromium.org> Cr-Commit-Position: refs/heads/master@{#567199}
-
Yuichiro Hanada authored
To make writing unit-tests easy, this CL separates mojo interfaces from ArcInputMethodManager. This CL does the following: - Rename ArcInputMethodManagerBridge to ArcInputMethodManagerService. - Introduce new ArcInputMethodManagerBridge interface for injection point. Bug: 845079 Test: unit tests Change-Id: I2e16b0bb9eba2c6bf9717813dff1640bd1f54b1d Reviewed-on: https://chromium-review.googlesource.com/1098738 Commit-Queue: Yuichiro Hanada <yhanada@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Cr-Commit-Position: refs/heads/master@{#567198}
-
Morten Stenshorne authored
Apart from making it more human neck friendly, those which were in vertical-lr would expose bug 850504 (once support for anonymous NG table cells is enabled), which really isn't an interesting thing to test in these tests. To be consistent, rewrite all tests to use horizontal-tb, not just those that previously used vertical-lr. viewport-y-vertical-XX-rtl additionally need a fixed horizontal offset for the absolutely positioned box in the ref. Not sure what went wrong, but both NG and legacy decided to act up in identical ways. Bug: 850504 Change-Id: I888db0680c5d543c9ac19a1ae8ab9ca791d8b259 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Reviewed-on: https://chromium-review.googlesource.com/1099384Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#567197}
-
Friedrich Horschig authored
This CL connects the TabLayout int the keyboard accessory with the ViewPager in the accessory sheet. This means the visibility of keyboard and tabs can be controlled by the ManualFillingCoordinator and is decoupled from the accessory. Bug: 828832, 811747 Change-Id: I01f225458d068e75fde19398fc352672779d23c0 Reviewed-on: https://chromium-review.googlesource.com/1091190 Commit-Queue: Friedrich Horschig <fhorschig@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Cr-Commit-Position: refs/heads/master@{#567196}
-
Roman Sorokin authored
Fixes regression introduced in CL:1068184 Also added a browsertest to verify behaviour on user kerberos files changed D-Bus signal BUG=chromium:845829 TEST=ExistingUserControllerActiveDirectoryTest.UserKerberosFilesChangedSignalTriggersFileUpdate Change-Id: Ieeb800f85210c094f34c93a71b0e7502ceae3114 Reviewed-on: https://chromium-review.googlesource.com/1089060Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Roman Sorokin <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#567195}
-
Mark Cogan authored
This CL updates the tab grid selector control assets, and removes the programatic scaling effect now that the assets are scaled. Tinting is also now done in code. Bug: 849645 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I48e9a3e72cad8609ad14a2940733a02140d89005 Reviewed-on: https://chromium-review.googlesource.com/1099070 Commit-Queue: Mark Cogan <marq@chromium.org> Reviewed-by:
edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#567194}
-
Joel Hockey authored
Bug: 845075 Change-Id: Id6667e02fcdd7be54d11e6ccec0074a475948754 Reviewed-on: https://chromium-review.googlesource.com/1100360Reviewed-by:
Stuart Langley <slangley@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#567193}
-
Mark Cogan authored
Per the attached bug, this CL changes the tab grid to not show the changed selection state when leaving the tab grid via a tap on a cell. Cell selection actually happened twice when a cell was tapped; once directly from the collection view, and a second time after the selection had round-tripped through the mediator. This CL cleans that up, and has the collection view decline all selection events, allowing them only to happen as a result of consumer calls. This CL adds a |showSelectionState| property to the grid view controller; when set to NO, the grid view controller will not update the selection state of the cells when the -selectWithItemID: consumer call is received. The ID of the selected item is maintained, however. This Cl also doesn't show the selection ring on the proxy cells animated for the transition. Bug: 849642 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ibf8d2c91dc80c42dea55c7e457efe7232d234267 Reviewed-on: https://chromium-review.googlesource.com/1096763Reviewed-by:
-
Dominic Battre authored
This CL enables the spec fetching if you enable the ExperimentalUi. If you have to manually enable the feature, this probably means that you did not get a proper finch config, so a good default is set that exercises the new code. Bug: 846694 Change-Id: I072e7c3b1977bd654949e2244873a48fb5a551ee Reviewed-on: https://chromium-review.googlesource.com/1100458Reviewed-by:
Dominic Battré <battre@chromium.org> Reviewed-by:
-
Colin Blundell authored
The identity_test_utils.h methods that deal with refresh tokens promise that they block until the relevant operation is complete (as do their wrapper IdentityTestEnvironment methods). However, they don't actually do that. And that's really the most important part. This CL makes reality match the contracts. As part of this change, OneShotIdentityManagerObserver needs to be extended to have the information of which event it's waiting on, as a single signin can generate both a token updated and a primary account set event, and the one that's received first might not be the one that the client of the observer is actually intending to wait on. Note: The current production impl doesn't actually require this blocking as it is synchronous. However, it is a goal to change those interactions back to being asynchronous, which this blocking in the test code future-proofs for. Bug: 798699 Change-Id: Ib8b098b65411ef22a8a3ceed4a4f58dfcad1e931 Reviewed-on: https://chromium-review.googlesource.com/1098966 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/28252da5cf0a..7d4d6b816c28 git log 28252da5cf0a..7d4d6b816c28 --date=short --no-merges --format='%ad %ae %s' 2018-06-14 skia-bookmaker@skia-swarming-bots.iam.gserviceaccount.com Update markdown files Created with: gclient setdep -r src/third_party/skia@7d4d6b816c28 The AutoRoll server is located here: https://autoroll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=csmartdalton@chromium.org Change-Id: If22f28c901c95110abdb4cfcb66fca18adfb4b86 Reviewed-on: https://chromium-review.googlesource.com/1100695Reviewed-by:
-
Nasko Oskov authored
Bug: none. Change-Id: I803b731a6418f02b2326b4e9a8cc9da7a572c92b Reviewed-on: https://chromium-review.googlesource.com/1100549Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#567188}
-
Sergiy Byelozyorov authored
TBR=dpranke@chromium.org Bug: 831171 Change-Id: Ida71a6072a3b25910bdda8fa4f15901349c7c8a0 Reviewed-on: https://chromium-review.googlesource.com/1100461Reviewed-by:
Sergiy Byelozyorov <sergiyb@chromium.org> Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Cr-Commit-Position: refs/heads/master@{#567187}
-
Tetsui Ohkubo authored
This CL changes icon color and sub header text color in UnifiedSystemTray's detailed views. Screenshot: http://screen/224qsUVChRx TEST=manual BUG=850396 Change-Id: I48b512bd3d0e0cb4b7b42b3f6b2557fb637239af Reviewed-on: https://chromium-review.googlesource.com/1098857Reviewed-by:
Yoshiki Iguchi <yoshiki@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#567186}
-
Roman Sorokin authored
Shows network error message instead of an unknown error message BUG=842619 TEST=manual Change-Id: I5149fbed8282c44e9cf1960d3b4a6f013095f983 Reviewed-on: https://chromium-review.googlesource.com/1098693Reviewed-by:
-
Tetsui Ohkubo authored
This CL adds audio detailed view (audio settings) to UnifiedSystemTray. Audio settings is only accessible when there are multiple audio inputs or outputs. TEST=manual BUG=847770 Change-Id: Ifee6be242c9c89bdb8012f3a6c905fca042aa1c6 Reviewed-on: https://chromium-review.googlesource.com/1098871Reviewed-by:
-
Friedrich Horschig authored
This CL replaces the duplicated @+id/keyboard_accessory used for the bottom sheet with @+id/keyboard_accessory_sheet. Furthermore, it adds some spacing, colors and dimensions that bring the whole appearance of the keyboard accessory and its bottom sheet closer to the intended outcome. Bug: 828832, 811747 Change-Id: I7e949b8e0f3f5825344f672a933e7b3e6bcd685a Reviewed-on: https://chromium-review.googlesource.com/1090921Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Friedrich Horschig <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#567183}
-
Patti authored
Origins with only embargoed content settings are currently not included in the All Sites list. Fix. Bug: 835712 Change-Id: Ic413db891753e729f618097cca28c554cc02f1fc Reviewed-on: https://chromium-review.googlesource.com/1090596 Commit-Queue: Patti <patricialor@chromium.org> Reviewed-by:
Dave Schuyler <dschuyler@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#567182}
-
Yoichi Osato authored
This patch rebases image expectations for - paint/markers/inline-spelling-markers-hidpi-composited.html - paint/markers/inline-spelling-markers-hidpi.html since they are scaled and only have 1px diff. Bug: 850448 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: Ib5e68393d577a7ee493b31406d65dab60efb36ce Reviewed-on: https://chromium-review.googlesource.com/1100595Reviewed-by:
-
Mitsuru Oshima authored
BUG=b/74845106 TEST=covered by unittests Change-Id: I1dde9c2ab19d08945f7ad4d9d2038dc1d0ff3e54 Reviewed-on: https://chromium-review.googlesource.com/1093518Reviewed-by:
Malay Keshav <malaykeshav@chromium.org> Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#567180}
-
Hiroshige Hayashizaki authored
Instead of redirecting ModuleMap->Modulator->ModuleScriptLoader, this CL makes ModuleMap::FetchSingleModuleScript() to invoke ModuleScriptLoader directly, in order to: - Reduce indirection around Modulator, and - Remove Modulator::FetchNewSingleModule(). This CL moves ModuleScriptLoaderRegistry from ModulatorImplBase to ModuleMap, to make it accessible from ModuleMap. This shouldn't affect lifetime and semantics, because Modulator and ModuleMap correspond one-to-one. Previously, unit tests have overridden Modulator::FetchNewSingleModule() and put fake implementation there. After this CL, unit tests creates a fake ModuleScriptFetcher. Bug: 845285 Change-Id: If94d958c57e0e3e18622efd032863b2fbd8cc5c2 Reviewed-on: https://chromium-review.googlesource.com/1079963 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#567179}
-
