- 16 Dec, 2017 28 commits
-
-
Eric Caruso authored
This is a variant of CopyDirectory which refuses to overwrite files that already exist in the destination directory. Bug: 768003 Test: existing + new unit tests Change-Id: I29ba308158d13b90f21928b76104eb671ab8a728 Reviewed-on: https://chromium-review.googlesource.com/754266 Commit-Queue: Eric Caruso <ejcaruso@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#524556}
-
Will Chen authored
This creates an ApplicationTestRunner.resetState() method to reset storage state and is added to the beginning of every test that relies on a storage API. This makes ClearDataForOrigin more robust by making it explicit that it's asynchronous and (mostly) fixing the race condition. The callback doesn't guarantee the storage has actually been deleted but at least it has been scheduled (see storage_partition.h), whereas before it returned with a success response immediately. We could update third_party/WebKit/Source/devtools/front_end/resources/ClearStorageView.js to actually wait for this callback to be called to update the UI when the user clicks on the clear storage button rather than rely on a 500ms timeout, but let's defer this for a follow-up patch. Bug: 667560 Change-Id: Ia7b70aebabd970957ae3581e47ee31a10992c657 Reviewed-on: https://chromium-review.googlesource.com/792534 Commit-Queue: Will Chen <chenwilliam@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#524555}
-
Dmitry Gozman authored
This unblocks moving DevToolsAgent to blink. Bug: 776009 Change-Id: Ie366481cc3913d864f773aa07007dd90aec0dc8d Reviewed-on: https://chromium-review.googlesource.com/828145 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Reviewed-by:
Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#524554}
-
Kenneth Russell authored
These will run side-by-side the non-Swarmed ones until they're green, at which point they'll be renamed to replace the old ones. BUG=754875 TBR=dpranke@chromium.org NOTRY=true 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: I1b84a56a6242085c8f11510515aee2b0acbd2d02 Reviewed-on: https://chromium-review.googlesource.com/831153Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#524553}
-
Joel Einbinder authored
In 2014, CodeMirror had poor support for auto-closing quotes. In response to bug 411238, a custom implementation of auto-closing quotes was added. This assumed a US keyboard layout, and broke some international keyboards. CodeMirror has much better support for auto-closing quotes now, so this patch removes the legacy code. Bug: 578852 Change-Id: I9b80e6f628adadc7b37c39dab7be5eaa0c76d7a9 Reviewed-on: https://chromium-review.googlesource.com/830650Reviewed-by:
Andrey Lushnikov <lushnikov@chromium.org> Commit-Queue: Joel Einbinder <einbinder@chromium.org> Cr-Commit-Position: refs/heads/master@{#524552}
-
Sergey Ulanov authored
Bug: 762125 Change-Id: I010c4608ecd2fd86149bcbd17a86c991575d01f2 Reviewed-on: https://chromium-review.googlesource.com/827673Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#524551}
-
depot-tools-roller@chromium.org authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/41d9d87e9653..d3f769a71026 $ git log 41d9d87e9..d3f769a71 --date=short --no-merges --format='%ad %ae %s' 2017-12-15 thestig Change "ldap" to "username". Created with: roll-dep src/third_party/depot_tools The AutoRoll server is located here: https://depot-tools-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=phajdan.jr@chromium.org Change-Id: I28546c9d25183ccffb8e96240aabc3635a037050 Reviewed-on: https://chromium-review.googlesource.com/831172Reviewed-by:
depot-tools-roller . <depot-tools-roller@chromium.org> Commit-Queue: depot-tools-roller . <depot-tools-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#524550}
-
Miguel Casas authored
This CL removes the superfluous PIXEL_FORMAT_I422, synonym of the older PIXEL_FORMAT_YV16. This format was landed in crrev.com/2571163002 with a comment that "Apart from how the buffer is allocated the format is basically idential to I422 (a.k.a YU16).", but I can't see any difference from the code, i.e. both PIXEL_FORMAT_YV16 and PIXEL_FORMAT_I422 are treated the same everywhere. Note that usually we should not roll back VideoPixelFormat's PIXEL_FORMAT_MAX but PIXEL_FORMAT_I422 was never added to enums.xml, so UMA knows nothing about it [1], and we can just pretend it was never there and was never logged [2] (it'd be logged as some hash name) [1] https://cs.chromium.org/chromium/src/tools/metrics/histograms/enums.xml?type=cs&q=histograms+enums.xml+y16&sq=package:chromium&l=43271 [2] https://uma.googleplex.com/timeline_v2?sid=deaff528e0268c7d0fdaa405130a075f TBR=reveman@chromium.org for the autochange in video_resource_updater.cc Bug: 795424 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;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: I9e5d52d8013bc4f6d237c8274ceb63327d78fa30 Reviewed-on: https://chromium-review.googlesource.com/830769 Commit-Queue: Miguel Casas <mcasas@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#524549}
-
dpapad authored
Convert the "Extension options" row to a cr-link-row and leverage the existing styling for the disabled case. Bug: 794756 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I7139e20a17d5e0ac5c85f0e1da7a5ad3bd2ab3d9 Reviewed-on: https://chromium-review.googlesource.com/826484 Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Dave Schuyler <dschuyler@chromium.org> Cr-Commit-Position: refs/heads/master@{#524548}
-
Scott Chen authored
Bug: 795012 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ifab4bd719569504821dc18db8ac785fadfaa71b3 Reviewed-on: https://chromium-review.googlesource.com/828461 Commit-Queue: Scott Chen <scottchen@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#524547}
-
Chromium WPT Sync authored
Using wpt-import in Chromium b0c1909e. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/7815 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 Directory owners for changes in this CL: jsbell@chromium.org: external/wpt/encoding TBR=foolip No-Export: true Change-Id: If3d920c7f2e742fa5d422359590d6f0a863e88a6 Reviewed-on: https://chromium-review.googlesource.com/830929 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@{#524546}
-
David Tseng authored
- introduce a scoped setter which enables the keyboard during spoken feedback's lifetime - remove restriction/message from ChromeVox disclaimer asking user to enable a11y vk - re-map some Search-like-function keys 1. Search+Left/Right -> Home/End 2. Search+Shift+Left/Right -> Shift Home/End 3. Search+Ctrl+Left/Right -> Ctrl+Home/End 4. Search+Shift+Ctrl+Left/Right -> Shift+Ctrl+Home/End - re-map braille commands to edit key commands as follows: 1. dot 3/6 chord -> previous/nextCharacter -> Left/Right 2. dot 2/5 chord -> previous/nextWord -> Ctrl+Left/Right 3. dot 1/4 chord -> previous/nextObject -> Up/Down 4. dot 2-3/5-6 chord -> previous/nextGroup -> Ctrl+Up/Down (jumpToTop/Bottom remain dots 1-2-3/4-5-6 and trigger ordinary edit key command mappings). notes: we use the vk codepath because it handles compatibility down the stack for us and potentially triggers other side effects such as auto complete/IME Test: with a braille display, ensure a11y vk off. Press incrementally Ctrl + t; verify new tab shows up. In Google Docs, press Search+Left/Right (and all other combos enumerated above). Verify proper caret/selection afterwards. Turn on sticky mode. Verify Search nav performs ChromeVox commands rather than send keys. Do all of the same with a braille display inside of Docs. Verify ability to edit and navigate directly from display. Bug: Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I4878c8a930cd643c406294b7f302c9ad2c07ee9f Reviewed-on: https://chromium-review.googlesource.com/823578 Commit-Queue: David Tseng <dtseng@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#524545}
-
Kenneth Russell authored
Disabling this RenderFrameHostImplBrowserTest on this configuration as it's frequently hanging and blocking the CQ. BUG=795326 TBR=avi@chromium.org Change-Id: I88e5b4158de3c92113b767b00b64d88e11ae09ca Reviewed-on: https://chromium-review.googlesource.com/831127Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#524544}
-
Maria Khomenko authored
This reverts commit 0a2853de. Reason for revert: Fixed the underlying problem on O bot. Original change's description: > Disable ChromeHomeAppMenuTest#testPromoAppMenuHeader on O > > TBR=mdjones@chromium.org > BUG=786454 > > Change-Id: Ia87e8cbba360cc98ae5bf70a68d757fe147b3c90 > Reviewed-on: https://chromium-review.googlesource.com/825306 > Reviewed-by: Theresa <twellington@chromium.org> > Reviewed-by: Matthew Jones <mdjones@chromium.org> > Commit-Queue: Theresa <twellington@chromium.org> > Cr-Commit-Position: refs/heads/master@{#523849} TBR=twellington@chromium.org,mdjones@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 786454 Change-Id: I9c1f6ce1992573fb894f5ee203995fe0ccb5481f Reviewed-on: https://chromium-review.googlesource.com/831227Reviewed-by:
Maria Khomenko <mariakhomenko@chromium.org> Commit-Queue: Maria Khomenko <mariakhomenko@chromium.org> Cr-Commit-Position: refs/heads/master@{#524543}
-
Hiroshige Hayashizaki authored
https://chromium-review.googlesource.com/752006 added an IsolatedCopy() call but the SecurityOrigin can be null when WebLocalFrameImpl::SetIsolatedWorldSecurityOrigin() is called from TestRunnerForSpecificView::SetIsolatedWorldSecurityOrigin(). Bug: 793649, 779730 Change-Id: I017eec29cc6c8cc9a753181c9b6f335569cccb9f Reviewed-on: https://chromium-review.googlesource.com/825653 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#524542}
-
Dmitry Gozman authored
It was introduced to resume other pages in the same render process when the page paused on breakpoint is closed. This now works due to Mojo connection error handler triggering detach automatically and unpausing as a result. Removing content-specific code from DevToolsAgent unblocks migrating it to blink. Bug: 776009 Change-Id: I20113df99a947eeaeabfb59886a07fc3f866ea27 Reviewed-on: https://chromium-review.googlesource.com/830977Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#524541}
-
Hiroshige Hayashizaki authored
This CL creates CachedMetadataHandler when a response is received, instead of creating when a Resource is constructed. This is to - Prepare for removing the reference from CachedMetadataHandler to Resource in [1]. The data needed for CachedMetadataHandler is available when a response is received, but not available when the Resource is created. Also, this CL makes the lifetime of CachedMetadataHandler match with the period where the corresponding ResourceResponse is set in Resource::response_. - Enable CachedMetadataHandler even after revalidation. Previously, CachedMetadataHandler was cleared on failed revalidation and was never re-created after that. This CL creates a new CachedMetadataHandler everytime a ResourceResponse is received, and thus re-creates a CachedMetadataHandler on failed revalidation. [1] https://chromium-review.googlesource.com/791494 Bug: 784875 Change-Id: Ia32f5772cbd5970734f67098f235c2a1428b46ea Reviewed-on: https://chromium-review.googlesource.com/786410 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Tsuyoshi Horo <horo@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Reviewed-by:
Nate Chapin <japhet@chromium.org> Cr-Commit-Position: refs/heads/master@{#524540}
-
Elad Alon authored
WebRtcEventLogManager can now handle locally-bound WebRTC event logs. Once we add handling of remote-bound logs, it would be good to have clear separation of the logic that handles locally-bound and remote- bound logs. This CL does just that. Bug: 775415 Change-Id: If7113d2fdbb7f8f240cceb4c309f6032da86341f Reviewed-on: https://chromium-review.googlesource.com/822938Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Elad Alon <eladalon@chromium.org> Cr-Commit-Position: refs/heads/master@{#524539}
-
Aleks Totic authored
After CL 827548, margins are always copied to Legacy. Additional copy in LayoutNGBlockFlow is no longer needed. Ian says so. Bug: 740993 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: If1aee4b15a966d58bd62f66190fdf00f035ed0b4 Reviewed-on: https://chromium-review.googlesource.com/830474Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Aleks Totic <atotic@chromium.org> Cr-Commit-Position: refs/heads/master@{#524538}
-
Ovidio Henriquez authored
This change updates a few characteristic removed tests to use the new test API. This will allow the tests to be migrated into the wpt/ directory. BUG=509038 Change-Id: I7ee5735d4052168f380437e9389546cd4d29a72f Reviewed-on: https://chromium-review.googlesource.com/822522Reviewed-by:
Conley Owens <cco3@chromium.org> Reviewed-by:
Vincent Scheib <scheib@chromium.org> Commit-Queue: Vincent Scheib <scheib@chromium.org> Cr-Commit-Position: refs/heads/master@{#524537}
-
Brett Wilson authored
Replace with calls to SetLayoutManager(make_unique...); This is for consistency with other layout manager types now that we have a safer scoped_ptr way to set the layout manager. GridLayout's members have been updated to use C++11 initialization. Change-Id: Ief98beea9e473cec1550642c4d2b6e8d40c9e14f Reviewed-on: https://chromium-review.googlesource.com/830675 Commit-Queue: Brett Wilson <brettw@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#524536}
-
Tom Anderson authored
The freetype unbundle build file is out-of-date, and trying to use it will result in errors like this: ERROR Unresolved dependencies. //third_party:freetype_harfbuzz(//build/toolchain/linux/unbundle:default) needs //third_party/freetype:freetype_source(//build/toolchain/linux/unbundle:default) This CL changes the unbundled build file to simply toggle the use_system_freetype flag. R=thestig@chromium.org CC=phajdan.jr@chromium.org Change-Id: If38ccc51098857cd479707879562b23f8f32e85f Reviewed-on: https://chromium-review.googlesource.com/830974Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#524535}
-
Jay Civelli authored
Moving the PdfToEmfConverter interface to the printing service. As a result, simplified the client of that interface in chrome/browser/printing/pdf_to_emf_converter.cc: - merging PdfConverterUtilityProcessHostClient and PdfConverterImpl. - calls on the service can now happen on the UI thread instead of the IO thread Renamed the mojom::PdfToEmfConverter implementation from PdfToEmfConverterImpl to PdfToEmfConverter as is now preferred naming style for Mojo interface implementation. Bug: 766451 Change-Id: Ic748547f56bb193a558210f97558640ef8d6e44b Reviewed-on: https://chromium-review.googlesource.com/826308Reviewed-by:
Will Harris <wfh@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Jay Civelli <jcivelli@chromium.org> Cr-Commit-Position: refs/heads/master@{#524534}
-
Will Chen authored
Now that all the tests have been migrated to the new integration test framework, we can remove the remaining bits of the legacy framework. There were a few references to inspector files in a handful of network tests, which have been updated to their respective devtools' resources files. Also found a (disabled) virtual test that slipped through the cracks: virtual/threaded/http/tests/devtools/tracing/idle-callback.js Bug: 667560 Change-Id: I2fddfee0af766c7f672bd118ff3fc75d23418d89 Reviewed-on: https://chromium-review.googlesource.com/828268Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Will Chen <chenwilliam@chromium.org> Cr-Commit-Position: refs/heads/master@{#524533}
-
Pavel Feldman authored
Bug: 795333 Change-Id: Iec3bfa4f6ed0f10813f20fbc93e3422cd094334a Reviewed-on: https://chromium-review.googlesource.com/830606Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#524532}
-
Dirk Pranke authored
This CL adds the blink unit tests (blink_heap_unittests, blink_platform_unittests, webkit_unit_tests, webkit_python_unittests, wtf_unittests) to a bunch of mac, linux, win, chromeos, and fyi bots where they should've been running but weren't, by removing a bunch of test_suite_exceptions. All of these tests are fast so there should be no significant impact on capacity or cycle time. There are still a bunch of exceptions for android, memory, and clang bots that we'll address in a follow-up CL. R=jbudorick@chromium.org, foolip@chromium.org BUG=560435, 795306 Change-Id: I38c5b72b49e47014017cf671dd22c4bf52b5869d Reviewed-on: https://chromium-review.googlesource.com/829874 Commit-Queue: Dirk Pranke <dpranke@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#524531}
-
Will Chen authored
* reattach-after-editing-styles.html - migrated into a protocol test. * shadow-dom-rules-restart.html - migrated into a protocol test. * injected-script-discard.html - migrated into a protocol test. * tabbed-pane-closeable-persistence-restore.html - removed. It required an unusual testing API and has adequate coverage from tabbed-pane-closeable-persistence.js and browser test "DevToolsSanityTest.TestSettings" * open-close-open.html - tried migrating into a browser test, but found a race condition with quickly opening, closing, and re-opening DevTools window. This test covered a very specific regression and had a very specific implementation fix which doesn't seem applicable anymore: https://bugs.webkit.org/show_bug.cgi?id=56747 Bug: 667560 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I41b0ea3e7837dee5049cc973354c431d09d9ae7f Reviewed-on: https://chromium-review.googlesource.com/827647Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Andrey Lushnikov <lushnikov@chromium.org> Commit-Queue: Will Chen <chenwilliam@chromium.org> Cr-Commit-Position: refs/heads/master@{#524530}
-
Alex Moshchuk authored
Bug: 739418 Change-Id: Idc340923293ac257c15045c047da08c3c18281a6 Reviewed-on: https://chromium-review.googlesource.com/831147Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Commit-Queue: Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#524529}
-
- 15 Dec, 2017 12 commits
-
-
Matt Falkenhagen authored
The "service worker context" is attached to a StoragePartition. When this context tries to start a service worker, it asks SiteInstance for a process to start the worker in. This process must be in the same StoragePartition as the context. In some cases like the <webview> tag, a non-default StoragePartition is created for guests, but the service worker was trying to start up in a process for the default StoragePartition since it created a SiteInstance using the script URL of the worker (e.g., https://www.example.com) and got a process from that. The solution is to stash the site URL on the StoragePartition when it's for guests (e.g., chrome-guest://someapp/somepath), and have service worker use that site URL when creating a SiteInstance. Bug: 781313, 752667 Change-Id: I8bf7ee8fd78a42e6c4e23684160c18da4e5189e5 Reviewed-on: https://chromium-review.googlesource.com/807973Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Tsuyoshi Horo <horo@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#524528}
-
Elliot Glaysher authored
I only saw these comments after the patch landed. Bug: 769308 Change-Id: Ia401bb2ec83a714e89a0cc16d8262a341f354e99 TBR: sky@chromium.org Reviewed-on: https://chromium-review.googlesource.com/831059Reviewed-by:
Elliot Glaysher <erg@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Elliot Glaysher <erg@chromium.org> Cr-Commit-Position: refs/heads/master@{#524527}
-
Yuly Novikov authored
BUG=793970 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: I44b9c33f7d7dc2bfde4e0cf4093e91dd80adc391 Reviewed-on: https://chromium-review.googlesource.com/821470Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#524526}
-
Ivan Sandrk authored
In locked fullscreen mode we want to disable most keyboard shortcuts to prevent circumventing the mode. More details on the design doc: go/locked-fullscreen-mode/?disco=AAAABc8Ml1k Bug: chromium:755696 Change-Id: I9ad28cab5cc5dae77ff00295b470ef268bff4d9d Reviewed-on: https://chromium-review.googlesource.com/785952 Commit-Queue: Ivan Šandrk <isandrk@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#524525}
-
Joel Einbinder authored
Some dark theme colors were being unintentionally overwritten by the light theme. Some other colors were never defined in the dark theme. Bug: 738029 Change-Id: I161a23dbf296980af0c4f4a9fab49558e9941fd1 Reviewed-on: https://chromium-review.googlesource.com/826037Reviewed-by:
Andrey Lushnikov <lushnikov@chromium.org> Commit-Queue: Joel Einbinder <einbinder@chromium.org> Cr-Commit-Position: refs/heads/master@{#524524}
-
Yves Arrouye authored
Act directly on changes to policy data. Do not store a CrOS setting for when an enrollment identifier is needed. Do not convert the current policy in ChromeDeviceSettingsProto to anything (it will be removed). BUG=chromium:778535 TEST=unit_tests Change-Id: I49fb2537556f9e3781267f9eb2ede1b2d5dee964 Reviewed-on: https://chromium-review.googlesource.com/827575Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#524523}
-
xlou authored
TestEng needs some APIs to help with testing. This API returns the list of printers which includes kConfigured, kEnterprise, and kAutomatic printers that are available through CupsPrintersManager. R=achuith@chromium.org, isherman@chromium.org, skau@chromium.org Bug: 780998 Change-Id: Ib5411ba5a3a094a2772ce5ecbbf636f66e9597dd Reviewed-on: https://chromium-review.googlesource.com/818394 Commit-Queue: Shirleen Lou <xlou@chromium.org> Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by:
Sean Kau <skau@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Cr-Commit-Position: refs/heads/master@{#524522}
-
Ovidio Henriquez authored
This change updates the prefixes used for loading Chromium specific resources to use the aliases used by the Apache HTTP server for the http bluetooth test. As a result, the cross origin iframe test now uses the new test API so that it can be migrated to the wpt/ directory. BUG=509038 Change-Id: Ib946c912913d7a67b887fff4d74e432ea915ddb9 Reviewed-on: https://chromium-review.googlesource.com/827531Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Conley Owens <cco3@chromium.org> Commit-Queue: Ovidio Henriquez <odejesush@chromium.org> Cr-Commit-Position: refs/heads/master@{#524521}
-
Dmitry Skiba authored
This CL adds more trace events to functions called during renderer startup. See crbug.com/794646#c1 for before/after results. Bug: 794646 Change-Id: I2bd0d7f46f128c112c9da9fac9b643ca203d6237 Reviewed-on: https://chromium-review.googlesource.com/825623 Commit-Queue: Dmitry Skiba <dskiba@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#524520}
-
rnephew authored
Bug: 781409 Change-Id: I214af9ee0173795c5909d3a037c944af8f5ae456 Reviewed-on: https://chromium-review.googlesource.com/830683Reviewed-by:
Ned Nguyen <nednguyen@google.com> Commit-Queue: rnephew <rnephew@chromium.org> Cr-Commit-Position: refs/heads/master@{#524519}
-
Kurt Horimoto authored
In addition to adding keyboard notification observation, this CL also moves VoiceOver fullscreen disabling into a shared object. Bug: 793129 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia8f8501ff7f5e97ec9d794e6c3d33687e90f4d9f Reviewed-on: https://chromium-review.googlesource.com/816356 Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#524518}
-
Maria Khomenko authored
When tests use a rule that enables features and the test enables yet another feature separately only one set of features will end up getting applied because the enable-feature/disable-feature flag is duplicated. Ensure we aggregate all requests before applying. BUG=786454 Change-Id: I79208e13f25de7c725d71f241481a454265d83c4 Reviewed-on: https://chromium-review.googlesource.com/830259Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Maria Khomenko <mariakhomenko@chromium.org> Cr-Commit-Position: refs/heads/master@{#524517}
-