- 02 Aug, 2018 25 commits
-
-
Reilly Grant authored
This change ports UserInfoFetcher from net::URLFetcher to SimpleURLLoader. This was the last user of the net::URLRequestContextGetter provided by policy::CloudPolicyClient and so this has been removed. Bug: 844951 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I752a16fbfcaa92419c79693d2379b918e04b7b9a Reviewed-on: https://chromium-review.googlesource.com/1156049 Commit-Queue: Reilly Grant <reillyg@chromium.org> Reviewed-by:
Ryan Hamilton <rch@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#580023}
-
Nasko Oskov authored
The WebNavigationApiTest.UserAction loads a page with has a subframe in a different site than the main frame. Since the main page is an extension URL, those will get isolated in separate processes. The test checks that a context menu in the subframe, which opens a link in a new tab results in the correct event for the webNavigation API. However, since it doesn't wait for the subframe navigation to complete, the context menu creation can incorrectly capture the parent process id instead of the subframe process id. This is timing dependent, which leads to flakiness in the test. This CL adds an explicit observer for the subframe navigation and updates the test expectation to reflect the fact that the subframe is in a separate process than the parent. Bug: 662160 Change-Id: I40efc406818f572243720d8feaff2dd597262a5d Reviewed-on: https://chromium-review.googlesource.com/1155255 Commit-Queue: Nasko Oskov <nasko@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#580022}
-
Evan Stade authored
Tested via --ash-debug-shortcuts and Ctrl+Alt+Shift+T Bug: 854704,760811 Change-Id: Ia3aadae89fcd50d95a84fdf7d275e97bc2e9e2e9 Reviewed-on: https://chromium-review.googlesource.com/1152230 Commit-Queue: Evan Stade <estade@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#580021}
-
angle-chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/a2d1d2dbd58a..cc73f241145d git log a2d1d2dbd58a..cc73f241145d --date=short --no-merges --format='%ad %ae %s' 2018-08-01 jmadill@chromium.org Micro-optimize some validation checks. 2018-08-01 jmadill@chromium.org Final cleanup to ValidateDrawAttribs. Created with: gclient setdep -r src/third_party/angle@cc73f241145d 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=fjhenigman@chromium.org Change-Id: I64516ed6f57faa643ef9e6b6a96cbb7d9e16f6bf Reviewed-on: https://chromium-review.googlesource.com/1159441Reviewed-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@{#580020}
-
Carlos Knippschild authored
With Offline Page being fully deployed there is no need anymore for the UI-exclusive flag that was introduced in its very beginnings, as a means to have its pipeline tested without any visual clues of its existence. This change removes that flag that and consolidates all UI related checks on the main feature flag that still remains in place. Bug: 850588 Change-Id: If30bbf36b8361a5af69bf59a748600d5fd1e71ea Reviewed-on: https://chromium-review.googlesource.com/1157526Reviewed-by:
Carlos Knippschild <carlosk@chromium.org> Reviewed-by:
Justin DeWitt <dewittj@chromium.org> Commit-Queue: Carlos Knippschild <carlosk@chromium.org> Cr-Commit-Position: refs/heads/master@{#580019}
-
iOS Autoroller authored
TBR=smut Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ifc1cef2d5690079e95fb7fda9c6ae5e81fbd7b57 Reviewed-on: https://chromium-review.googlesource.com/1159581 Commit-Queue: iOS Autoroller <ios-autoroller@chromium.org> Reviewed-by:
iOS Autoroller <ios-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#580018}
-
Mohammad Refaat authored
Tests are failing because clipping is disabled for iOS 12 by this cl: https://chromium-review.googlesource.com/c/chromium/src/+/1158566 Bug: 860790 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Icffec0a18d6775c24195093f900d00343ac22b6a Reviewed-on: https://chromium-review.googlesource.com/1159696 Commit-Queue: Mohammad Refaat <mrefaat@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#580017}
-
Johannes Henkel authored
Change-Id: I7fc667a8bfd72c8896848a9443f8ff6abc2cdf38 Reviewed-on: https://chromium-review.googlesource.com/1159240 Commit-Queue: Johannes Henkel <johannes@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#580016}
-
Eric Karl authored
Currently VizProcessContextProvider assumes no alpha. Android needs alpha in cases where it is rendering on top of a video overlay. Plumb the alpha request through via RendererSettings. Bug: 869131 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I6c51a07b141e61d95b2e25183156ebf15daeca38 Reviewed-on: https://chromium-review.googlesource.com/1155715Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
kylechar <kylechar@chromium.org> Commit-Queue: Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#580015}
-
Martin Kreichgauer authored
In 9aa7c545 AuthenticatorImpl::IsUserVerifyingPlatformAuthenticatorAvailable was changed to make Touch ID unavailable in Incognito and if the embedder does not provide a configuration for the Touch ID authenticator. The latter check dereferenced a nullptr in AuthenticatorImpl's request_delegate_. request_delegate_ only gets initialized for the duration of a MakeCredential or GetAssertion request. It is uninitialized during IsUVPAA. This works around the issue by adding a IsWebAuthenticationTouchIdAuthenticatorSupported instance method to ContentBrowserClient, so that the feature support can be checked without requiring a request delegate instance. Bug: 869651 Change-Id: Ib64a2278d63047d68876e224d44a9aab96f9cfea Reviewed-on: https://chromium-review.googlesource.com/1157605Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Kim Paulhamus <kpaulhamus@chromium.org> Commit-Queue: Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#580014}
-
Kyle Horimoto authored
This CL passes a MultiDeviceSetupClient to MultideviceHandler and utilizes it to pass data about the current multi-device host to the settings page. Additionally, this CL adds functionality to the TestWebUI class which allows it to store and respond to registered callbacks in WebUI. A follow-up CL will add functionality which provides the status of the various multi-device features (e.g. Instant Tethering) to the settings page as well. Bug: 824568 Change-Id: I5665e225122e7a2875150f61cef77aaf770298d9 Reviewed-on: https://chromium-review.googlesource.com/1157268Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#580013}
-
Sarah Hu authored
Bug: 864549 Change-Id: If79c0100067c62a9fedb3119f43813a87d5bd6c6 Reviewed-on: https://chromium-review.googlesource.com/1145864 Commit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org> Reviewed-by:
Jacob Dufault <jdufault@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#580012}
-
Joel Einbinder authored
This patch calls preventDefault on all events that we forward from extension panels into devtools. This would cancel too many events, so a list of devtools global keyboard shortcuts are passed into the extension. Bug: 682068 Change-Id: Ide557dafed759e20dfc6698d8cdc33f932acbe83 Reviewed-on: https://chromium-review.googlesource.com/1159467Reviewed-by:
Andrey Lushnikov <lushnikov@chromium.org> Commit-Queue: Joel Einbinder <einbinder@chromium.org> Cr-Commit-Position: refs/heads/master@{#580011}
-
Sergey Ulanov authored
Now a process that creates a WebContext must pass a service_directory handle. That directory is mounted as /svc in the context process. Also renamed dataDirectory->data_directory, which is a proper struct member name according to the FIDL style guide. Bug: 869216 Change-Id: I63af58a472edf0d45b79b0108082fffcac6eaa72 Reviewed-on: https://chromium-review.googlesource.com/1157196Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#580010}
-
Xiaoqian Dai authored
If a window has transient children, the transform that's applied during dragging should be adjusted for the transient children. See before https://drive.google.com/open?id=1BK_AlNAFGJR9eDZdgvf5MY_rsWaB0YXe and after https://drive.google.com/open?id=12Mg8VHaEhpTEC40VLHImZC9NroWMayDA Bug: 823769 Change-Id: Ic04739d8327b5febffcc9ff4430ce1db52ba3194 Reviewed-on: https://chromium-review.googlesource.com/1152505 Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#580009}
-
Adrienne Walker authored
Before this patch, there's no way to use --profiling-at-start or --profiling-flush to generate a profile of the gpu process, because these flags weren't being forwarded there. Change-Id: Idca9fd8daa9d67e91bb7270d6225b9255434e014 Reviewed-on: https://chromium-review.googlesource.com/1159393Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: enne <enne@chromium.org> Cr-Commit-Position: refs/heads/master@{#580008}
-
Mitsuru Oshima authored
Bug: 869429 Test: covered by unittest Change-Id: I192d74e0dd467238af2bf0db7e7e37b5df69ed81 Reviewed-on: https://chromium-review.googlesource.com/1156881Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#580007}
-
Muyuan Li authored
Bug: b/77916222 Change-Id: Ic2efb6e21d243bf5c19554dbdd35773ef082e12b Reviewed-on: https://chromium-review.googlesource.com/1132624 Commit-Queue: Muyuan Li <muyuanli@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#580006}
-
Anand K. Mistry authored
BUG=862892 Change-Id: Ib36cf0f8fdaadae0b9170ec3823d31e0fd398187 Reviewed-on: https://chromium-review.googlesource.com/1152638 Commit-Queue: Anand Mistry <amistry@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#580005}
-
Xiaoqian Dai authored
The dragging behavior is the same with dragging an app window from the top of the screen. As it shares some same logic with dragging an app window, these logic are extracted to TabletModeWindowDragDelegate. See recorded video: https://drive.google.com/open?id=1d6t7K-ZuxgpT57b-Mx5gMJ49IjMHX-9X Bug: 823769 Test: SplitViewTabDraggingTest.OnlyAllowDraggingOnCaptionArea Change-Id: I09d852ef9eefd1059f2be85fee52e1714b1968fd Reviewed-on: https://chromium-review.googlesource.com/1150860 Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#580004}
-
Lu Chen authored
Add recording of AnchorElementMetrics.*.HrefEngagementScoreExternal. Bug: 850624 Change-Id: I049b5847beb76a80a7c30077373f50079cee061c Reviewed-on: https://chromium-review.googlesource.com/1157551 Commit-Queue: Lu Chen <chelu@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#580003}
-
Renee Wright authored
Bug: 869734 Change-Id: Ie863c125d49c351b79f5b072bc20535a75dd8d6a Reviewed-on: https://chromium-review.googlesource.com/1157963Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Renée Wright <rjwright@chromium.org> Cr-Commit-Position: refs/heads/master@{#580002}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/68b5e7dbc095..134364159fa8 git log 68b5e7dbc095..134364159fa8 --date=short --no-merges --format='%ad %ae %s' 2018-08-01 jessehall@google.com [mali importer] Add Exynos Mali job event parser Created with: gclient setdep -r src/third_party/catapult@134364159fa8 The AutoRoll server is located here: https://catapult-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=sullivan@chromium.org Change-Id: I2dfb839946b066aa315b73b91f3be6d662200327 Reviewed-on: https://chromium-review.googlesource.com/1159421Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#580001}
-
Pedro Amaral authored
The new tab button doesn't have text next to it anymore. Change-Id: Ic489fe7f5c2a83cbf9a7a55497cd2065c715df21 Reviewed-on: https://chromium-review.googlesource.com/1153846Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Pedro Amaral <amaralp@chromium.org> Cr-Commit-Position: refs/heads/master@{#580000}
-
Krishna Govind authored
R=cmasso@chromium.org Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I3d30a20b118686d63080580579bd474029e12562 Reviewed-on: https://chromium-review.googlesource.com/1159605Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#579999}
-
- 01 Aug, 2018 15 commits
-
-
Evan Stade authored
This means Chrome will enter immersive when the user presses the fullscreen key (F4 on a normal keyboard --- although F4 on a normal keyboard doesn't actually work right now because event rewriters aren't functioning in OopAsh). The way this works in classic Ash is via BrowserWindowStateDelegate, which isn't used in OopAsh. Bug: 640365 Change-Id: I3b7e7a8911c5d8cba48f2d770e74104dba61e9e2 Reviewed-on: https://chromium-review.googlesource.com/1156995Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#579998}
-
Jorge Gil authored
This change adds support for multiple smart actions instead of only one and moves them below the copy/print section and groups them with extensions. Some SEPARATOR logic from https://chromium-review.googlesource.com/c/chromium/src/+/1087724 is reverted (due to the move from top to extensions) and some extra separator cleanup is needed after all actions are loaded. Bug: b/32781610 Change-Id: I7c825e9ef58cc21ed10a49b784e6258a704e2f8b Reviewed-on: https://chromium-review.googlesource.com/1147544Reviewed-by:
Luis Hector Chavez <lhchavez@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Greg Kerr <kerrnel@chromium.org> Reviewed-by:
David Jacobo <djacobo@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Jorge Gil <jorgegil@google.com> Cr-Commit-Position: refs/heads/master@{#579997}
-
Mathieu Perreault authored
Bug: 842014 Change-Id: Ie645e71c03c71fb7b7ddd7550128eb43bec3bc07 Reviewed-on: https://chromium-review.googlesource.com/1159190Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Mathieu Perreault <mathp@chromium.org> Cr-Commit-Position: refs/heads/master@{#579996}
-
Sammie Quon authored
1. Removes some unused headers. 2. Updates some comments. 3. Make functions in .cc match order in .h. 4. Removes some left behind headers. 5. Removes or updates some not up to date comments. Bug: none Test: ash_unittests Change-Id: Ia75eb9ece31a41c2bfa9a9d58aef5ec82cf05325 Reviewed-on: https://chromium-review.googlesource.com/1159329 Commit-Queue: Sammie Quon <sammiequon@chromium.org> Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#579995}
-
Chris Hamilton authored
This adds explicit lifetime management between URLRequestChromeJob and URLDataSourceImpl, by having the job own a shared_refptr to the data source. This allows the data source to be detached from the backend yet remain alive as long as the request job is still alive (which eventually wants to use its i18n template replacements map). BUG=863318 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I84caa70dff1e1c65ac430fca11656e85e4a6eebd Reviewed-on: https://chromium-review.googlesource.com/1156625 Commit-Queue: Chris Hamilton <chrisha@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#579994}
-
Takumi Fujimoto authored
The UI Service is not specific to WebUI, so c/b/ui/media_router/ is a more appropriate directory for it. TBR=bauerb@chromium.org Change-Id: I29b150969511a533bba1188f2466160537c28285 Reviewed-on: https://chromium-review.googlesource.com/1159381Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Commit-Queue: Takumi Fujimoto <takumif@chromium.org> Cr-Commit-Position: refs/heads/master@{#579993}
-
Darren Shen authored
KeyboardController has the concept of "locked". When the keyboard is locked (e.g. when it is launched from the accessibility menu), the work area of the screen is shrunk, so that only the part above the keyboard is considered 'useable'. Currently, this is implemented in the shelf by observing changes to the keyboard bounds. If the bounds change and the keyboard is locked, we update the work area with the given bounds. While this is correct, whether the keyboard is locked or not should be internal to the keyboard. Instead, shelf should be listening for the *displaced* bounds, which essentially tell observers how much of the screen is now unuseable. Thus, we replace the keyboard bounds in shelf to either the displaced bounds or the occluded bounds, depending on what the shelf actually wants. If the bounds are used for the work area, then it's the displaced bounds. Otherwise, it's the occluded bounds. Change-Id: I7c53d0b2fe8b3b6769ce7113b924def34db2785e Bug: 845780 Reviewed-on: https://chromium-review.googlesource.com/1146530Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Blake O'Hare <blakeo@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#579992}
-
Fredrik Hubinette authored
Frames have more granular information than the container, so let's use the color space from the frames rather than from the container. BUG=870050 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: I4dd972561dd85f8ddce3ca55a01e34dc3a8cf386 Reviewed-on: https://chromium-review.googlesource.com/1155785 Commit-Queue: Fredrik Hubinette <hubbe@chromium.org> Reviewed-by:
Tom Finegan <tomfinegan@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#579991}
-
Victor Costan authored
sql::Connection represents an opened SQLite database. Pedantically, the name is correct. At the same time, it is inconsistent with the naming convention of similar software -- IndexedDB, LevelDB, and WebSQL all use a derivative of "Database" (IDBDatabase, DB, WindowDatabase) to name this concept. Furthermore, most sql::Connection instances have names like "db" and "database". This CL renames the class to sql::Database, reducing the cognitive load caused by dissonances like "sql::Connection db_". TBR=mpearson Bug: none Change-Id: I8abf34016bca236b6e38a772f5848e7ea9bf1c59 Reviewed-on: https://chromium-review.googlesource.com/1157876 Commit-Queue: Victor Costan <pwnall@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#579990}
-
Kyle Horimoto authored
chrome.send() must always be enabled for dialogs, since dialogs rely on chrome.send() to notify their handlers that the dialog should be closed. Bug: 824568 Change-Id: I12325c243e2f2cf04d417ea71b03519bcb3d1159 Reviewed-on: https://chromium-review.googlesource.com/1145580Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#579989}
-
Zach Helfinstein authored
Rename "isInteresting" to "isActionable" for precision. Bug: None Change-Id: I40d6e2c74b80a0717f0c2b864010d5650bd69378 Reviewed-on: https://chromium-review.googlesource.com/1159179Reviewed-by:
David Tseng <dtseng@chromium.org> Commit-Queue: Zach Helfinstein <zhelfins@chromium.org> Cr-Commit-Position: refs/heads/master@{#579988}
-
Philip Rogers authored
There are several DidUpdateLayout functions but it is not obvious that these are only called when the main frame lays out (see the callsite in Document::LayoutUpdated). I confirmed this locally. This patch renames these to DidUpdateMainFrameLayout so it is more obvious that this is only called for main frame layouts. The comments indicate that DidUpdateMainFrameLayout should only be called with clean layout, but this is not true for the callsite in RenderViewImpl::OnEnablePreferredSizeChangedMode. A TODO has been added to clean up RenderViewImpl::OnEnablePreferredSizeChangedMode. Bug: 868983 Change-Id: I66de3e8c402e7edeff9a8b1e7991716f55b771a8 Reviewed-on: https://chromium-review.googlesource.com/1156991Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#579987}
-
Paul Dyson authored
Bug: 841671 Change-Id: Ieb86ea9e6636651edd0655c2721c11ea430f96cd Reviewed-on: https://chromium-review.googlesource.com/1148022Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Commit-Queue: Paul Dyson <pdyson@chromium.org> Cr-Commit-Position: refs/heads/master@{#579986}
-
Joshua Bell authored
The cookie RFC[1] does not define an encoding for cookie names/values; they are treated as a sequence of octets. The Cookie Store spec[2] mandates treating the octets as UTF-8 encoded. When decoding octet sequences into strings, the decode should be done without treating a leading U+FEFF as a BOM. Add a test to verify this. [1] https://tools.ietf.org/html/draft-ietf-httpbis-rfc6265bis-02 [2] https://wicg.github.io/cookie-store/ Bug: 729800 Change-Id: I23b7eb82b35862b8797a203ae6ea86cbd69001d2 Reviewed-on: https://chromium-review.googlesource.com/1159336Reviewed-by:
Victor Costan <pwnall@chromium.org> Commit-Queue: Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#579985}
-
Johannes Henkel authored
This one is real simple. I've also tweaked the constant for the native default to be of the same type as the fromNative that uses it, so it's slightly easier to read. Change-Id: Iaaa24d4ee9dbf195ec520a568006812d4549f72e Reviewed-on: https://chromium-review.googlesource.com/1158997Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Johannes Henkel <johannes@chromium.org> Cr-Commit-Position: refs/heads/master@{#579984}
-