- 12 Dec, 2017 9 commits
-
-
Andrey Lushnikov authored
This patch changes StylesSidebarPane autocomplete for property names in the following maner: - for regular nodes, only regular CSS properties are autocompleted - for SVG nodes, both CSS and SVG properties are autocompleted BUG=721998 R=dgozman Change-Id: Ie06307d62a1b9685f0bf998b5b2714c47e3d5ee8 Reviewed-on: https://chromium-review.googlesource.com/820647Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#523269}
-
Jaebaek Seo authored
As shown in crbug.com/786852/#c9, the current select popup in Android WebView is not correctly located. The main reason is that the left margin for the anchor rectangle of the popup is added by x value of scroll manager. This CL adds y value of scroll manager to the left margin instead. Bug: 786852 Change-Id: I8ebf983de64eea3c89d8fc7d2063962a7f1c7bb1 Reviewed-on: https://chromium-review.googlesource.com/818679Reviewed-by:
Tao Bai <michaelbai@chromium.org> Reviewed-by:
Jinsuk Kim <jinsukkim@chromium.org> Commit-Queue: Jaebaek Seo <jaebaek@chromium.org> Cr-Commit-Position: refs/heads/master@{#523268}
-
Xiyuan Xia authored
- No "Servers" and "Authorities" tab; - No "Import" as in previous non-MD impl; Bug: 719907 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ie6fc8e7b257937b4e0607e426d9b0550dc2ddd4b Reviewed-on: https://chromium-review.googlesource.com/820638Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#523267}
-
Mitsuru Oshima authored
Modified ClientControlledStateSurface so that - State change are first sent to client (Android) except for PIN/TRUSTED_PIN - PIN/TRUSTED_PIN are still handled on server side per security requirement. It may leave the the state inconsistent and the same thing can happen in the old implmenetation. - the bounds change request from chrome side is ignored for now and only the geometry change from client is applied. * Emulate the state change by client in ShellSurface unit tests. Animation isn't enabled yet in this CL. It'll be addressed in a separate CL. BUG=762816 TEST=manually tested on device (maximize/fullscreen/minimix/pin/unpin etc) Existing test should pass with new implementation (with test impl). Change-Id: Idc8001876f4b1c50d375b1090ce63516f3998ab0 Reviewed-on: https://chromium-review.googlesource.com/809488 Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#523266}
-
Devlin Cronin authored
An (illegal) include was commented out. Remove it. Bug: None Change-Id: Ief2da1cb5a3f381aeff1589f055fd12158bda1fc Reviewed-on: https://chromium-review.googlesource.com/821110Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#523265}
-
John Abd-El-Malek authored
When there's an ongoing navigation, if the app loada a JavaScript URL we create a temporary NavigationRequest in NavigatorImpl::RequestNavigation. However the call to RenderFrameHostManager::GetFrameHostForNavigation was resetting the speculative RFH of the original NavigationRequest since the site instance of a JavaScript load is always the same as the existing frame. Bug: 793432 Change-Id: I5a25eea9a8733cd5cebe71f700b30ebb4c0b1710 Reviewed-on: https://chromium-review.googlesource.com/820453 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#523264}
-
Yoichi Osato authored
This patch is a part of crrev.com/c/813336 ([LayoutNG] Paint text selection on load.) LayoutSelection.cpp: - This converts legacy offset to NG offset if LayoutObject has enclosing NGBlockFlowlayout before paint phase. - LayoutSelectionStartEndForNG() returns start/end offsets that the NGPhysicalTextFragment is selected. LayoutSelectionTest.cpp: - This patch adds NGLayoutSelectionTest to confirm/demonstrate how we use the function. Also you can refer my following painting patch(crrev.com/c/818884). Bug: 708452 Change-Id: Ic9fc6822f83e0821c5f2f4917d4ac316543702e2 Reviewed-on: https://chromium-review.googlesource.com/818864Reviewed-by:
Koji Ishii <kojii@chromium.org> Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Commit-Queue: Yoichi Osato <yoichio@chromium.org> Cr-Commit-Position: refs/heads/master@{#523263}
-
Ilya Sherman authored
R=asvitkine@chromium.org Bug: 727984 Change-Id: Ic1993cce55beed9e00512f6fe4b001fc5ec4c591 Reviewed-on: https://chromium-review.googlesource.com/818746 Commit-Queue: Ilya Sherman <isherman@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#523262}
-
Hector Carmona authored
Bug: 789897 Change-Id: Id16a994fda31d9a294444cf56ceab646548cb29d Reviewed-on: https://chromium-review.googlesource.com/803623 Commit-Queue: Hector Carmona <hcarmona@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#523261}
-
- 11 Dec, 2017 31 commits
-
-
Andrey Kosyakov authored
Bug: 726381 Change-Id: I76fb27d4eb50730ff85fdf39895859db4e343bc8 Reviewed-on: https://chromium-review.googlesource.com/811526 Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#523260}
-
Allen Vicencio authored
Add ReadDirectory and GetMetadataEntry methods for SmbProviderClient. This allows read-only navigation of the share, excluding reading files. Bug:chromium:757625 CQ-DEPEND=CL:794892 Change-Id: I55d4baa6dec95392774cdea3c07c13989b31e782 Reviewed-on: https://chromium-review.googlesource.com/749897 Commit-Queue: Allen Vicencio <allenvic@chromium.org> Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Reviewed-by:
Dan Erat <derat@chromium.org> Cr-Commit-Position: refs/heads/master@{#523259}
-
Joy Ming authored
This CL is part of the greater effort to build out functionality to allow users to change their default download storage location. This is the first step, surfacing the option to change download location in the preferences pane. This is not fully functional (the directory selector is not yet implemented) and is hidden behind a flag. Bug: 792775 Change-Id: Ic57aa2e4f61c83861e5711c2fe0b48737217904e Reviewed-on: https://chromium-review.googlesource.com/818205 Commit-Queue: Joy Ming <jming@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#523258}
-
Lei Zhang authored
OS_CHROMEOS already implies OS_LINUX. Change-Id: I194767e151ec59134682db0a276e60f2ef9469ee Reviewed-on: https://chromium-review.googlesource.com/820932Reviewed-by:
Ehsan Karamad <ekaramad@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#523257}
-
Steven Holte authored
Coverage on iOS left for a follow up. Bug: 793470 Change-Id: I4844e667c421f7df3c3412362d50ad74078e8223 Reviewed-on: https://chromium-review.googlesource.com/818202Reviewed-by:
Brian White <bcwhite@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#523256}
-
Tom Sepez authored
Network service requires this in some situtations. Additionally, alphabetize methods and consolidate some logic into helper methods along the way. As part of the consolidation, need to mask open() flags before checking "if allowed" and test for "EFAULT" earlier in some situations. Require "Create" permission for rename() for consistency. Use initializer-list for vectors where possible. Clean up hooks and tests to use bitset creation helper function. Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: Ide22e759692baf4350b643400619885f1676db15 Reviewed-on: https://chromium-review.googlesource.com/812452 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#523255}
-
Petr Hosek authored
We will still produce an error if --fail-on-unused-args is used. Bug: 684096 Change-Id: I4a0640ad83072750d555c8de9e6391f76942ae38 Reviewed-on: https://chromium-review.googlesource.com/792606 Commit-Queue: Petr Hosek <phosek@chromium.org> Reviewed-by:
Scott Graham <scottmg@chromium.org> Reviewed-by:
Brett Wilson <brettw@chromium.org> Cr-Commit-Position: refs/heads/master@{#523254}
-
spqchan authored
Highlight the URLs on the sidebar according to the Security Panel MD specs. Screenshot: https://screenshot.googleplex.com/baLGTsSAe8B Updated the sidebar tests so it will test for the new changes. Bug: 523589, 617311 Change-Id: I35c14aded5530cfad2b58f18e9270250fa2e7ed1 Reviewed-on: https://chromium-review.googlesource.com/795171 Commit-Queue: Sarah Chan <spqchan@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Eric Lawrence <elawrence@chromium.org> Cr-Commit-Position: refs/heads/master@{#523253}
-
Yutaka Hirano authored
This CL consists of bunch of refactoring efforts: - Add explanatory comments. - Rename scheduler() to Scheduler() - Rename WasRan() to WasRun() - Add Release() and ReleaseAndSchedule() to test fixtures as the existing usage in tests are too verbose. - InvalidInstance() now returns a value instead of a mutable reference. Bug: 785770 Change-Id: I80c963a3509cce74aaba28c2b5f62eca47ebe84d Reviewed-on: https://chromium-review.googlesource.com/816276Reviewed-by:
Takashi Toyoshima <toyoshim@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#523252}
-
Ryan Landay authored
The select_insert_text param in this method is only ever called with false after https://chromium-review.googlesource.com/c/chromium/src/+/801979. Bug: 784039 Change-Id: I3c8d0176bfb6600605d42b83676341a61fd83aac Reviewed-on: https://chromium-review.googlesource.com/820352 Commit-Queue: Ryan Landay <rlanday@chromium.org> Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#523251}
-
Xing Liu authored
Bug: 793158 Change-Id: I32c0567bb5e1c44ccd21e898e1022593121b187e Reviewed-on: https://chromium-review.googlesource.com/817359 Commit-Queue: Xing Liu <xingliu@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#523250}
-
Peiyong Lin authored
It's very important to understand how much IPC volume GRC needs to handle. This patch adds an IPCVolumeReporter to report IPC count for frame, page and process every minute. The histogram uses a maximum count of 100 with 10 buckets. BUG=776118, 790680 Change-Id: Id11ffa4480f7d066f140cf6daba1dedc744e311e Reviewed-on: https://chromium-review.googlesource.com/815280Reviewed-by:
Zhen Wang <zhenw@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: lpy <lpy@chromium.org> Cr-Commit-Position: refs/heads/master@{#523249}
-
Ned Nguyen authored
Bug:793609 NOTRY=true TBR=dpranke@chromium.org Change-Id: I770559876a6dd6b7de0a3575e02a1b854858a414 Reviewed-on: https://chromium-review.googlesource.com/820939Reviewed-by:
Ned Nguyen <nednguyen@google.com> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#523248}
-
Erik Luo authored
Console will remember - User text typed in the input filter - Whether the sidebar was open, consistent with Sources and the drawer - Which top level sidebar filter was selected Bug: 787620, 734088 Change-Id: Iab109fbf1f459b0c64974c7414dcf3fd24739f11 Reviewed-on: https://chromium-review.googlesource.com/810090 Commit-Queue: Erik Luo <luoe@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#523247}
-
Xiaohan Wang authored
Today we always write and read from the DataPipe asynchronously. This might introduce unnecessary latency since the DataPipe might already be readable/writable when we schedule the read/write. This also indirectly contributes to out-of-order dispatch of Decode() and Reset() calls in some MojoDecoderBufferConverter clients, e.g. MojoAudioDecoderService, MojoVideoDecoderService, MojoDecryptorService. See BUG for more details. Unit tests are added to cover this issue. Note that this CL does not completely fix the BUG. A follow up CL will come next, with more tests added. BUG=792281 TEST=More tests added. Change-Id: Ia6dc25f8621cca2a041b6299b104de2074c06a02 Reviewed-on: https://chromium-review.googlesource.com/818329 Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Dan Sanders <sandersd@chromium.org> Reviewed-by:
Xiangjun Zhang <xjz@chromium.org> Cr-Commit-Position: refs/heads/master@{#523246}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 9e68ea0b. With Chromium commits locally applied on WPT: d857d94a "Handle display:contents while creating counter nodes." Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/7457 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=raphael.kubo.da.costa@intel.com No-Export: true Change-Id: I2c7430b2fc8ddeeab76dac9f922f70cacccff341 Reviewed-on: https://chromium-review.googlesource.com/820193 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@{#523245}
-
Ryan Landay authored
Gboard's voice input feature repeatedly calls InputConnection#setComposingText() with the in-progress text composition. If you say "new paragraph", it inserts a pair of newlines into the text. We currently don't handle this case properly because InputMethodController::SetComposition() calls TypingCommand::InsertText() (through InsertTextDuringCompositionWithEvents()) with the TypingCommand::kSelectInsertedText flag (which causes InsertText() to leave the newly-inserted text selected). InsertText() doesn't actually implement this behavior properly if the text being composed has a newline in it. It processes the text in multiple "runs," inserting line breaks in between. Currently, it inserts the first run, leaving it selected, then inserts a line break, replacing the first run of text (and then proceeds similarly for subsequent runs of text). There's currently a (quite old) FIXME in the code to fix this behavior. I'm taking a slightly different approach from the one suggested there, and selecting the inserted text all at once in TypingCommand::InsertText() instead of trying to modify InsertTextCommand and InsertParagraphSeparatorCommand so they can extend the current selection. I was inspired by the implementation of AdjustSelectionAfterIncrementalInsertion() to use a plain text offset from the start of the editable region to determine the start of the final selection. This approach is robust to whitespace normalization (this was a problem I ran into trying to use a DOM range). Note that with this fix, we no longer need the fix in https://chromium-review.googlesource.com/c/chromium/src/+/783770 to change the ending selection in InputMethodController::SetComposition(), as we're no longer setting the problematic ending selection in TypingCommand::InsertText(). Bug: 784039 Change-Id: I4b0bb95d1ecf8a979f15b3494f57a0b23e6e2459 Reviewed-on: https://chromium-review.googlesource.com/801979Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Changwan Ryu <changwan@chromium.org> Commit-Queue: Ryan Landay <rlanday@chromium.org> Cr-Commit-Position: refs/heads/master@{#523244}
-
Scott Graham authored
Bug: 793956 Change-Id: I8c1fe8a4e77b754a03e0d257f944c0ce5037fb72 Reviewed-on: https://chromium-review.googlesource.com/820853 Commit-Queue: Scott Graham <scottmg@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#523243}
-
Chris Cunningham authored
The VideoSampleEntry box includes a scoped_refptr<BitstreamConverter> regardless always, but we were only building the BitstreamConverter cc/h files when proprietary_codecs=true. This CL includes the BitstreamConverter interface defintion in all builds. Proprietary bitsream conversion logic (e.g. AVCBitstreamConvter) is still only included if proprietary_codecs = true. BUG: 793702 Change-Id: I8720b6547ca2e0af47d64ae594e86850da0e84b5 Reviewed-on: https://chromium-review.googlesource.com/819978 Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#523242}
-
Koji Ishii authored
The CL[1] disabled skipping inks for characters of IsCJKIdeographOrSymbol(). This does not cover all characters in CJK. This patch adds other CJK characters from Unicode blocks, in the same way as WebKit does. [1] https://codereview.chromium.org/2598393002 Bug: 793762 Change-Id: I20e10588adc988399c900b336d0c22ad984bb011 Reviewed-on: https://chromium-review.googlesource.com/820210 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/master@{#523241}
-
Charles Harrison authored
The upper %iles of SB2.Delay can be high (especially on Android). This is most concerning for main frame resources, since it is in the critical path of PLT. Bug: None Change-Id: Ia7df3082d5d34bc04e6696f900084e2e8c31acab Reviewed-on: https://chromium-review.googlesource.com/803951Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Nathan Parker <nparker@chromium.org> Commit-Queue: Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#523240}
-
Marti Wong authored
1. Prevent the folder icons from showing on a bookmark cells when loading favicons. 2. When creating new folder, add the new table cell to the tableView manually using insertRowsAtIndexPaths (so to skip reloadData) to shorten the icon blinking time. (before this change, it could take ~0.1s to load favicons after reloadData because icon loading, table scrolling and keyboard appearing all happen at the same time after tapping on 'New Folder'.) Bug: 792409 Change-Id: I6675dc4ea332ac2c095d5da4abec8c63e55de1c3 Reviewed-on: https://chromium-review.googlesource.com/817454Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Marti Wong <martiw@chromium.org> Cr-Commit-Position: refs/heads/master@{#523239}
-
Krishna Govind authored
R=abdulsyed@chromium.org Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia2d24601f998d384deb9bd0474ee00f19570cfac Reviewed-on: https://chromium-review.googlesource.com/820716Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#523238}
-
Tom Anderson authored
The tests are flaky when run in random order. Example failures: https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65460 https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65456 https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65449 https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65433 https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65431 https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65424 https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65404 https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65373 https://ci.chromium.org/buildbot/chromium.linux/Linux%20Tests/65279 BUG=664858 TBR=qyearsley@chromium.org Change-Id: I9871947f69860d308c4b2765f9e23dabbae3c3d4 Reviewed-on: https://chromium-review.googlesource.com/820100 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#523237}
-
Reilly Grant authored
This new tests fails with the Network Service enabled just as other PDFExtensionTests do. Lumping this in with issue 705114. Bug: 705114 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I6da8f21354e7f809e8b86a3f57b3f6723e57e386 Tbr: jam@chromium.org No-Try: true No-Tree-Checks: true No-Presubmit: true Reviewed-on: https://chromium-review.googlesource.com/820810Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#523236}
-
Yuzhu Shen authored
Bug: 793480 Change-Id: I02c9ae70759be31af025e5787a5683cd76bffc57 Reviewed-on: https://chromium-review.googlesource.com/818452Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Commit-Queue: Yuzhu Shen <yzshen@chromium.org> Cr-Commit-Position: refs/heads/master@{#523235}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/bc4dbea81177..92c8b1a07b01 $ git log bc4dbea81..92c8b1a07 --date=short --no-merges --format='%ad %ae %s' 2017-12-11 thestig Initialize FDE_TEXTEDITPIECE. 2017-12-11 thestig Get rid of default ctors for CPDF_Action and CPDF_AAction. 2017-12-11 thestig Clarify some function names in CPDF_NameTree. Created with: roll-dep src/third_party/pdfium The AutoRoll server is located here: https://pdfium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: I2e5b2499388ec4f71f8cdf2dd8df713a6fe7186f Reviewed-on: https://chromium-review.googlesource.com/820099 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#523234}
-
Mitsuru Oshima authored
Separated window state change methods into ShellSurface and ClientControlledShellSurface Bug: 809488 Change-Id: Ifec29e7a7ce2df0dd3b3c4a606479682d9f60dc2 Reviewed-on: https://chromium-review.googlesource.com/814835 Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Cr-Commit-Position: refs/heads/master@{#523233}
-
Eugene But authored
This method will be user for reporting Download.IOSDownloadPassKitResult UMA metric. Bug: 789735 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ic39d5ea1869dfb4e7f45e0ffa2902829a2e09a6d Reviewed-on: https://chromium-review.googlesource.com/817950 Commit-Queue: Eugene But <eugenebut@chromium.org> Reviewed-by:
Gregory Chatzinoff <gchatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#523232}
-
Brandon Jones authored
Bug: 670510 Change-Id: I4f135d13ac44ac6214b74eb03eb7734d6786a737 Reviewed-on: https://chromium-review.googlesource.com/817976Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
David Dorwin <ddorwin@chromium.org> Commit-Queue: Brandon Jones <bajones@chromium.org> Cr-Commit-Position: refs/heads/master@{#523231}
-
Tim Brown authored
For supported distros, group the versions together so that we can more easily see them together (include some spare ID range for each distro too). Also add in a couple of future versions so that we don't have to keep on updating the metrics. Downside is that the old histogram values are deprecated and will take a while to settle on the new values as newer versions roll out. Bug: 724244 Change-Id: I74f1c704529856ccb946178f035e8baed5212ecc Reviewed-on: https://chromium-review.googlesource.com/818203 Commit-Queue: Tim Brown <timbrown@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#523230}
-