- 21 Aug, 2018 40 commits
-
-
David Black authored
Per the mocks, in the mini view the Assistant icon should be in its wavy state. Bug: b:112904117 Change-Id: Ic29c447e23d39c629a4cd2856b3953e620af0192 Reviewed-on: https://chromium-review.googlesource.com/1183973 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#584909}
-
David Black authored
Decision was made to launch explore in a browser tab instead of within Assistant container. As such, we can deprecate the deep link and will instead use the OpenUrlResponse interaction flow. Bug: b:111704177 Change-Id: Iec68644f0b7c8df23c3926e89fae1283f3f6fd11 Reviewed-on: https://chromium-review.googlesource.com/1183965 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
-
Ned Nguyen authored
Bug:773084 Change-Id: I8ef6fe66ad10246e62e926f11d1d54ef209d300c NOTRY=true TBR=mythia@chromium.org, ulan@chromium.org Change-Id: I8ef6fe66ad10246e62e926f11d1d54ef209d300c Reviewed-on: https://chromium-review.googlesource.com/1183905Reviewed-by:
Ned Nguyen <nednguyen@google.com> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#584907}
-
Xiaoqian Dai authored
Bug: 874261 Change-Id: I5973ae1a7afd1a0ffd6f2dd61df679cc566e998a Reviewed-on: https://chromium-review.googlesource.com/1180253Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Mitsuru Oshima (Slow) <oshima@chromium.org> Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#584906}
-
Balazs Engedy authored
In case of long relying party IDs, we should not truncate them. Bug: 849323 Change-Id: Idbfb63866c54bd98b6a9006f47c59291a29bc4cc Reviewed-on: https://chromium-review.googlesource.com/1183494Reviewed-by:
Jun Choi <hongjunchoi@chromium.org> Commit-Queue: Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#584905}
-
Joel Einbinder authored
This patch uses Input.dispatchTouchEvent instead of Emulation.setEmitTouchEventsForMouse. This gives more predictable behavior and allows for each touch event to be awaited. Bug: chromedriver:2144 Change-Id: Ic243c216e2f832844a64f8f2af894eb6cc4234d1 Reviewed-on: https://chromium-review.googlesource.com/1182506 Commit-Queue: Joel Einbinder <einbinder@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#584904}
-
Kyle Horimoto authored
This CL adds a new page to the flow; the page utilizes the chrome.quickUnlockPrivate extension API to check the user's password. In a future CL, the auth token received from the API will be provided to the MultiDeviceSetup Mojo service when setting the host device. Bug: 870098, 824568 Change-Id: Ie10f6b2560a03c8879d8b7226d36e93c0311887c Reviewed-on: https://chromium-review.googlesource.com/1180482 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#584903}
-
Kyle Horimoto authored
This CL provides two new pieces of functionality: (1) UiPages can disable the forward button. (2) UiPages can prevent forward navigation, even though the button is enabled. This functionality is needed for a future CL which adds a password page. In that CL, the page's forward button needs to be disabled if no password has been entered yet, and forward navigation needs to be prevented if the password entered is incorrect. Bug: 870098, 824568 Change-Id: Id887c97d4b336d6b44f4aaa567d81e0ed73f9d5d Reviewed-on: https://chromium-review.googlesource.com/1179268 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#584902}
-
Geoff Lang authored
BUG=841655 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: I2d112cfc5262e02963d1151f19f721cd84fa756f Reviewed-on: https://chromium-review.googlesource.com/1181178Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Commit-Queue: Geoff Lang <geofflang@chromium.org> Cr-Commit-Position: refs/heads/master@{#584901}
-
Jimmy Tobin authored
MediaPipelineBackendManager's playing audio streams counts are now separated by ContentType. Added a IsPlaying method that can be called to check if there are playing streams for a specific ContentType. Bug: 112661891 Test: CQ and Checking with logs. Change-Id: I81f1268fd1c46db665b9fec5cf729049932bcc2d Reviewed-on: https://chromium-review.googlesource.com/1183952 Commit-Queue: Jimmy Tobin <jtobin@google.com> Reviewed-by:
Kenneth MacKay <kmackay@chromium.org> Cr-Commit-Position: refs/heads/master@{#584900}
-
Quan Nguyen authored
Bug: 870864 Change-Id: Iae47b89761e9fff42a3fe4a8cf52bef391900988 Reviewed-on: https://chromium-review.googlesource.com/1183964 Commit-Queue: Quan Nguyen <qnnguyen@chromium.org> Reviewed-by:
Jacob Dufault <jdufault@chromium.org> Cr-Commit-Position: refs/heads/master@{#584899}
-
Yue Li authored
Bug: b/112310573 Test: Manual Test Change-Id: I3e959a5c2eb4413a306fbc33fc930757d074aeeb Reviewed-on: https://chromium-review.googlesource.com/1182677Reviewed-by:
-
sczs authored
Bug: 876119 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Icf8628ea868bb76b27477022ee4068ed971a32ce Reviewed-on: https://chromium-review.googlesource.com/1182509 Commit-Queue: Sergio Collazos <sczs@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Gayane Petrosyan <gayane@chromium.org> Cr-Commit-Position: refs/heads/master@{#584897}
-
Martin Kreichgauer authored
This relaxes the access control restriction for keychain items created by the Touch ID authenticator to require biometric authentication *or* password entry. The effect is that the native Touch ID dialog will show a "use password" button next to the cancel button. Not that despite the name of the kSecAccessControlUserPresence attribute, the TouchIdAuthenticator is still user-*verifying* (passcode entry is a valid user verification method in the WebAuthN spec). Related Apple Developer documentation can be found here: https://developer.apple.com/documentation/security/secaccesscontrolcreateflags/ksecaccesscontroluserpresence?language=objc This change is somewhat backwards-incompatible: If a user tries to authenticate using a credential created *before* this change *and* actually chooses the "Use Password" fallback, they will afterwards be prompted with a second Touch ID dialog that does not have the password fallback button. This is acceptable since the feature hasn't launched yet. Bug: 678128 Change-Id: If4e3461ccd378bac286dbba68c3011fee2eb0fa3 Reviewed-on: https://chromium-review.googlesource.com/1183636 Commit-Queue: Martin Kreichgauer <martinkr@google.com> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#584896}
-
Philip Rogers authored
This reverts commit 7834ee21. Reason for revert: Suspected as causing failing tests in 874162 Original change's description: > Fix background painting on scrolling contents layer with foregrounds > > This patch removes a negative z-index children check from > PaintLayer::GetBackgroundPaintLocation. This lets backgrounds paint in > the scrolling contents layer in the presence of negative z-index > children, fixing the painting of background-attachment: local. This > patch also removes a DCHECK that was hit when a foreground layer and > kBackgroundPaintInScrollingContents coexisted. This DCHECK[1] could be > hit for root layers without the functional change in this patch. As the > test in this patch shows, kBackgroundPaintInScrollingContents and > foreground layers work together, so this patch makes non-root-layers use > both, and removes the DCHECK. > > What are foreground layers? A foreground layer is created if a stacked > PaintLayer has composited negative z-index children that should wedge > between the layer's background and the layer's contents. The foreground > layer will have the normal-flow contents in this case. See the comment > above foreground_layer_ in composited_layer_mapping.h. > > [1] DCHECK originally added in: https://crrev.com/9a2cdc8d96d. Reading > the comments, I think the DCHECK was intended to ensure the background > would not be painted into the foreground layer. > > Bug: 861948 > Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel > Change-Id: I7409cbe9ac61fed444856df342d0707d6babab09 > Reviewed-on: https://chromium-review.googlesource.com/1173532 > Reviewed-by: Chris Harrelson <chrishtr@chromium.org> > Commit-Queue: Philip Rogers <pdr@chromium.org> > Cr-Commit-Position: refs/heads/master@{#584802} TBR=pdr@chromium.org,chrishtr@chromium.org Change-Id: Ifd22a963dbd5ce288dfb31ab1e8ac41ebe63062d No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 861948 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Reviewed-on: https://chromium-review.googlesource.com/1184109Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#584895}
-
Gayane Petrosyan authored
These are histograms that have not been viewed in the last 6 months, but still report data on the Stable channel. Set these histograms to stop being recorded for Chrome releases after 2018-08-30. We will file bugs via Monorail (crbug) reminding owners about the approaching histogram expiry – initially 30 days prior to expiry, and also with follow-up alerts. The metrics dashboards will also display a warning for anyone viewing a soon-to-expire histogram. Owners and/or users of these histograms: Please comment on this CL if you think that any affected histogram should not have an expiry date set! This should be uncommon. Otherwise, there are a few options going forward: * If the histogram is no longer needed, please send a follow-up CL to delete the recording code and to mark the histogram as <obsolete>. (You can also remove the expiry date in the same CL.) * If you want to set a different expiry date, please send a follow-up CL to do so. * If neither of the above apply, then keep an eye out for Monorail bugs as the histogram expiry date approaches. For more info on how unused histograms are identified, see https://goto.google.com/uma-unused-histograms-cleanup Bug: 850539 Change-Id: I437ea5c000a3a4fe3a065dd51641de44a27a6fa9 Reviewed-on: https://chromium-review.googlesource.com/1126321 Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Reviewed-by:Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#584894}
-
Lei Zhang authored
Change-Id: I2ddcf856255fd4790ac0e810f41ddb44e0c7aaff Reviewed-on: https://chromium-review.googlesource.com/1182958Reviewed-by:
Wei Li <weili@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#584893}
-
Chris Harrelson authored
In particular, for cases when the PaintLayer direct compositing update fast-path applies. Bug: 872172 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: If059f2f76d9973c2a19d2f326f495e4891813d6e Reviewed-on: https://chromium-review.googlesource.com/1171797Reviewed-by:
vmpstr <vmpstr@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#584892}
-
Tim Zheng authored
This change add 3 histograms for measuring crosvm resource usage, namely CPU percentage, resident memory percentage, and number of crosvm processes. BUG=chromium:855762 TEST=manually tested on an eve device; unit tests included. Change-Id: I55f442505a340813db4eb6047844288e4801f325 Reviewed-on: https://chromium-review.googlesource.com/1175410 Commit-Queue: Tim Zheng <timzheng@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Dan Erat <derat@chromium.org> Cr-Commit-Position: refs/heads/master@{#584891}
-
Eugene But authored
-[CRWWebController setDocumentURL:] will have a DCHECK when new document URL does not match last committed URL in WKBasedNavigationManager. This DCHECK will help to diagnoze possible URL spoofing bugs in Slim Navigation feature. Deferring setDocumentURL: call to the point when the navigation is committed will make sure that WKBasedNavigationManager is in correct state and it is safe to compare these URLs. Without this change WKBasedNavigationManagerImpl::GetLastCommittedItemIndex will return last_committed_item_index_ ivar, which is not correct and URLs will not match. WebStateImpl::OnNavigationFinished is still called after setDocumentURL: to make sure that WebStateObserver callback is called after all WC internals were updated. Bug: 1182320 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I2e4067fc6220c3af5a28881cdca8f36f948c503e Reviewed-on: https://chromium-review.googlesource.com/1182414Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#584890}
-
Will Harris authored
https://helpx.adobe.com/security/products/flash-player/apsb18-25.html BUG=412078 TBR=jschuh@chromium.org Change-Id: I7e9dc9f7ee7b8e99cb524c9dd40ed5e4a8a28634 Reviewed-on: https://chromium-review.googlesource.com/1184027 Commit-Queue: Will Harris <wfh@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Cr-Commit-Position: refs/heads/master@{#584889}
-
Fredrik Söderquist authored
In preparation for making base-val <-> attribute synchronization, refactor the SVG*TearOff hierarchy so that "mutation notifications" are funneled through the SVGAnimatedProperty(Base) if the tear-off is associated with an attribute. Rather than storing a SVGElement* and QualifiedName, store a reference to the underlying SVGAnimatedProperty(Base) instead of the latter. This is then used to send notification through. BUG=873470 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ib6e5c25cf97b806ff613a39478dc4f10f85dcaf5 Reviewed-on: https://chromium-review.googlesource.com/1181342Reviewed-by:
-
Sky Malice authored
This reverts commit 66ce35f8. Reason for revert: Speculative revert to fix MainIntentBehaviorMetricsIntegrationTest#testLaunch_From_InAppActivities, see https://crbug.com/876463 Bug: 876463 Original change's description: > Remove data use updates from page load metrics > > Now that per-resource updates are sent to page load metrics, we no > longer need to send totals from the renderer process. Remove all > piping/callbacks and convert existing consumers to > OnResourceDataUseObserved. > > Per-resource updates: > https://chromium-review.googlesource.com/c/chromium/src/+/1164222 > > Change-Id: I1cb03df5e6579fc0ff767900336308a6bbad244a > Reviewed-on: https://chromium-review.googlesource.com/1178324 > Reviewed-by: Charlie Harrison <csharrison@chromium.org> > Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> > Commit-Queue: John Delaney <johnidel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#584744} TBR=kinuko@chromium.org,csharrison@chromium.org,rajendrant@chromium.org,johnidel@chromium.org Change-Id: I4b750a31c1f0a99b5edd18d7104d1b948913e1c8 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/1184062Reviewed-by:
Sky Malice <skym@chromium.org> Commit-Queue: Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#584887}
-
Mohsen Izadi authored
The connection error handler set on the GpuService in GpuProcessHost might arrive either before or after GpuProcessHost::OnProcessCrashed(). If it arrives before, it might end up running a new GPU process before OnProcessCrashed() has a chance to fallback to the appropriate GpuMode. This CL essentially reverts the use of connection error handler introduced in r584455 so that we ensure that the new GPU process is not launched before the fallback GPU mode is determined. BUG=876336 Change-Id: Ie83b0824b45b25c46c2baad3aa55e7f30c8a9113 Reviewed-on: https://chromium-review.googlesource.com/1183696Reviewed-by:
-
Alexander Timin authored
R=rkaplow@chromium.org BUG=853771 Change-Id: I9f569103758c4f174b921925f1f0f2f2d7503553 Reviewed-on: https://chromium-review.googlesource.com/1183677Reviewed-by:
Robert Kaplow (slow) <rkaplow@chromium.org> Commit-Queue: Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#584885}
-
Yi Gu authored
Upon calculating the relative position between a composited sticky element and its scroll container, we use EnclosingIntRect instead of RoundedIntRect which causes a scroll offset mismatch between main and cc if the relative position is non integer, e.g. float prepadding. Bug: 693412 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ideab875d2d087b8fe97264d0bddbd7e2393f8e7f Reviewed-on: https://chromium-review.googlesource.com/1183622Reviewed-by:
Stephen McGruer <smcgruer@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#584884}
-
Gayane Petrosyan authored
These are histograms that have not been viewed in the last 6 months, but still report data on the Stable channel. Set these histograms to stop being recorded for Chrome releases after 2018-08-30. We will file bugs via Monorail (crbug) reminding owners about the approaching histogram expiry – initially 30 days prior to expiry, and also with follow-up alerts. The metrics dashboards will also display a warning for anyone viewing a soon-to-expire histogram. Owners and/or users of these histograms: Please comment on this CL if you think that any affected histogram should not have an expiry date set! This should be uncommon. Otherwise, there are a few options going forward: * If the histogram is no longer needed, please send a follow-up CL to delete the recording code and to mark the histogram as <obsolete>. (You can also remove the expiry date in the same CL.) * If you want to set a different expiry date, please send a follow-up CL to do so. * If neither of the above apply, then keep an eye out for Monorail bugs as the histogram expiry date approaches. For more info on how unused histograms are identified, see https://goto.google.com/uma-unused-histograms-cleanup Bug: 850539 Change-Id: Ia7d065d7786ece8cdb0dbad78de13c0f9a699290 Reviewed-on: https://chromium-review.googlesource.com/1111473Reviewed-by: -
Mike Wasserman authored
Bug: 613657, 622480 Change-Id: Ief1471c2f5830a99f9e650d7447b821c155e4a49 Reviewed-on: https://chromium-review.googlesource.com/1183900Reviewed-by:
James Cook <jamescook@chromium.org> Commit-Queue: Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#584882}
-
Tommy Steimel authored
This CL adds a check in IsMouseEventOnInternalButton to make sure there is a layout view before using it. This prevents a crash in MediaControlOverlayPlayButtonElement. Bug: 870490 Change-Id: I085333c8ed6704d22b2fe8d33ef9a6e45332288e Reviewed-on: https://chromium-review.googlesource.com/1182788 Commit-Queue: Tommy Steimel <steimel@chromium.org> Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#584881}
-
Kenneth Russell authored
Neglected to use --additional-driver-flag command line arg. Bug: 875172 Change-Id: I3603e0b3659a2e6309673aad66b76a3dcb01fcc0 Reviewed-on: https://chromium-review.googlesource.com/1183884Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#584880}
-
Sky Malice authored
This reverts commit c0af3bf1. Reason for revert: Suspected cause of ios_chrome_ui_egtests failures, see https://crbug.com/876425. Original change's description: > Replace polling with callbacks in translate_ios.js. > > This should alleviate flaky errors when translating on iOS. > > When translate is tapped, translate_ios.js polls translate.js to wait > for translate element ready and then later polls again to wait for the > translation result. Polling can often time out due to network slowness. > > Bug: 869729 > Change-Id: I7e0e46fe279571f21b3aa0dd18f241dc83afee90 > Reviewed-on: https://chromium-review.googlesource.com/1178691 > Reviewed-by: David Roger <droger@chromium.org> > Reviewed-by: Hiroshi Ichikawa <ichikawa@chromium.org> > Reviewed-by: Eugene But <eugenebut@chromium.org> > Commit-Queue: John Wu <jzw@chromium.org> > Cr-Commit-Position: refs/heads/master@{#584549} TBR=droger@chromium.org,eugenebut@chromium.org,jzw@chromium.org,ichikawa@chromium.org Change-Id: Id0903e753c70dd43db8d4b84ba0e3bb868a65518 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 869729 Reviewed-on: https://chromium-review.googlesource.com/1183826Reviewed-by:
-
Mohsen Izadi authored
During shut down, HostGpuMemoryBufferManager singleton instance in content is destroyed before winding down the IO thread. Probably after that and before IO thread is shut down completely, there are GPU clients that might try to create GPU memory buffers. Ignore those requests. BUG=876327 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: I53854f451129693a5167fe0bb21cd9d72925b585 Reviewed-on: https://chromium-review.googlesource.com/1182282 Commit-Queue: Mohsen Izadi <mohsen@chromium.org> Reviewed-by:
kylechar <kylechar@chromium.org> Cr-Commit-Position: refs/heads/master@{#584878}
-
Sergio Villar Senin authored
Both are currently using URLFetcher for timezone and geolocation requests. They should use SimpleURLLoader instead to make them eventually work with the network service. Instead of having different CLs for them, we're migrating them at the same time as they're entangled, there is a dependency of geolocation in the timezone code. Bug: 872890 Change-Id: I8180a774f9fd8eaca8406328f404fb216b13180e Reviewed-on: https://chromium-review.googlesource.com/1177399Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Sergio Villar <svillar@igalia.com> Cr-Commit-Position: refs/heads/master@{#584877}
-
Lei Zhang authored
The WebContents for the Print Preview dialog does not perform PDF compositing. Only web pages renderers do. Also clean up print_manager_utils.h. Change-Id: I2875858d6c9513a6d920ab8ad9b603e06751ea77 Reviewed-on: https://chromium-review.googlesource.com/1182864 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
-
Li Lin authored
Bug: b:110107505 Change-Id: I8d392135d05864eefef230ce388bcf2a1e2aefd1 Reviewed-on: https://chromium-review.googlesource.com/1182619Reviewed-by:
-
Jérôme Lebel authored
Issue introduced with crrev.com/c/1174543 The table view now contains only one section. So the "Add Account…" cell is now part of the first section. The difference between the "Add Account…" cell and the identity cells should be make with item class. Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I3fabec71aa2caadc29c16bde819df712da56f0b1 Reviewed-on: https://chromium-review.googlesource.com/1183498Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#584874}
-
angle-chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/4bcaf9987acf..de8d4cb0d496 git log 4bcaf9987acf..de8d4cb0d496 --date=short --no-merges --format='%ad %ae %s' 2018-08-21 jmadill@chromium.org Optimize BindingsBenchmark::drawBenchmark. 2018-08-21 tobine@google.com Expect pass on pointcoord test on Pixel 2 Created with: gclient setdep -r src/third_party/angle@de8d4cb0d496 The AutoRoll server is located here: https://angle-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. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=cwallez@chromium.org Change-Id: I1705b2f61d163ae8aeac52ea82516d1407c283d1 Reviewed-on: https://chromium-review.googlesource.com/1183764Reviewed-by:
angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#584873}
-
Erik Chen authored
Iteration through observer list has been observed to add significant overhead to task processing. See https://bugs.chromium.org/p/chromium/issues/detail?id=859155#c12 for more details. Switching to a std::vector should minimize this overhead. Bug: 859155 Change-Id: I8c8851bfbb404afb948072984d399ea22882f125 Reviewed-on: https://chromium-review.googlesource.com/1182298Reviewed-by:
Gabriel Charette <gab@chromium.org> Commit-Queue: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#584872}
-
Xiangjun Zhang authored
Add an interface to allow the Cast Media Router Provider to connect to the Mirroring Service to start a mirroring session. The extension change will be submit separately. Before then, there is no behavior change. Bug: 734672 Change-Id: I41a865313be0a96c684e519bb25d665e197e9ebe Reviewed-on: https://chromium-review.googlesource.com/1171786 Commit-Queue: Xiangjun Zhang <xjz@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Scott Chen <scottchen@chromium.org> Reviewed-by:
Yuri Wiitala <miu@chromium.org> Reviewed-by:
Derek Cheng <imcheng@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#584871}
-
Mike Wasserman authored
Using ConvertRectToPixel yields unexpected enclosing rect bounds. (changes in dip locations can alter pixel bounds unintentionally) Instead, convert the rect origin and size pixel values separately. Bug: 871582 Test: No KSV (Ctrl-Alt-/) DCHECKs with non-1 ui/dsf scale [tablet mode]. Change-Id: Ia22ea75bb34382e323753bb173ce81b2c4b143b9 Reviewed-on: https://chromium-review.googlesource.com/1180265Reviewed-by:
-
