- 13 Dec, 2017 40 commits
-
-
xzhan96 authored
This CL creates blink.mojom.RequestContextFrameType enum type and uses it to replace existing native enums content::RequestContextFrameType and blink::WebURLRequest::FrameType. This is the preparation to impl the blink.mojom.ServiceWorkerClientInfo, and finally to remove content::ExtendableMessageEventSource, which is an incremental step toward Onion Soup: having less dependence on content types for service worker. BUG=782528 Change-Id: I6f7787575f1583297a27a0213b1440d01e94e2dd Reviewed-on: https://chromium-review.googlesource.com/791916 Commit-Queue: Xiaofeng Zhang <xiaofeng.zhang@intel.com> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#523699}
-
dpapad authored
Bug: 794003 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I4dc96f9a899ccc9b0b208f2f326772208b47c187 Reviewed-on: https://chromium-review.googlesource.com/822974Reviewed-by:
Dave Schuyler <dschuyler@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#523698}
-
Scott Violet authored
Creating BluezDBusManager is unnecessary for --mus, as WindowManagerService (in ash) takes care of it. BUG=776514 TEST=test only change Change-Id: Ie3b7e39a507a9263779ce8f1045141e51d81192f Reviewed-on: https://chromium-review.googlesource.com/823526Reviewed-by:
James Cook <jamescook@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#523697}
-
Tetsui Ohkubo authored
Notification center used fixed font sizes for UI labels. This is not the right way of settings font size. This CL - removes ash::message_center_style::GetFontListForSizeAndWeight(). - replaces all fixed font sizes with delta. TEST=manual - compared screenshots and verified it cause no visual change on actual device (chell) BUG=793207 Change-Id: Ia2c50965e6a132520d63921252e78a76a644734f Reviewed-on: https://chromium-review.googlesource.com/817456Reviewed-by:
Evan Stade <estade@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#523696}
-
Victor Miura authored
This reverts commit 3b36628a. Reason for revert: Suspect this broke the build. Unexpected Failures: * compositing/geometry/repaint-foreground-layer.html * compositing/gestures/gesture-tapHighlight-1-iframe-composited-scrolled-clipped.html * compositing/gestures/gesture-tapHighlight-1-iframe-composited-scrolled-late-noncomposite.html * compositing/gestures/gesture-tapHighlight-1-iframe-composited-scrolled.html * compositing/gestures/gesture-tapHighlight-1-iframe-composited.html * compositing/gestures/gesture-tapHighlight-1-overflow-div-composited-scroll-clip.html * compositing/gestures/gesture-tapHighlight-1-overflow-div-composited-scrolled.html * compositing/gestures/gesture-tapHighlight-1-overflow-div-composited.html * compositing/gestures/gesture-tapHighlight-1-overflow-div-scrolled-late-noncomposite.html * compositing/gestures/gesture-tapHighlight-2-iframe-composited-inner.html * compositing/gestures/gesture-tapHighlight-2-iframe-composited-outer.html * compositing/gestures/gesture-tapHighlight-2-iframe-scrolled-inner-late-composite.html * compositing/gestures/gesture-tapHighlight-2-overflow-div-composited-inner-scroll-inner.html * compositing/gestures/gesture-tapHighlight-2-overflow-div-composited-inner-scroll-outer.html * compositing/gestures/gesture-tapHighlight-2-overflow-div-composited-inner.html * compositing/gestures/gesture-tapHighlight-2-overflow-div-composited-outer-scroll-inner.html * compositing/gestures/gesture-tapHighlight-2-overflow-div-composited-outer-scroll-outer.html * compositing/gestures/gesture-tapHighlight-2-overflow-div-composited-outer.html * compositing/gestures/gesture-tapHighlight-img-and-text-2.html * compositing/gestures/gesture-tapHighlight-img-and-text.html * compositing/gestures/gesture-tapHighlight-pixel-rotated-link.html * compositing/gestures/gesture-tapHighlight-simple-cancel2.html * compositing/gestures/gesture-tapHighlight-simple-longPress.html * compositing/gestures/gesture-tapHighlight-simple-margin.html * compositing/gestures/gesture-tapHighlight-simple-multi-line.html * compositing/gestures/gesture-tapHighlight-simple-nested.html * compositing/gestures/gesture-tapHighlight-simple-scaledX.html * compositing/gestures/gesture-tapHighlight-simple-scaledY.html * compositing/gestures/gesture-tapHighlight-simple-window-scroll.html * compositing/gestures/gesture-tapHighlight-simple.html Original change's description: > [PE] Don't ignore clip change not across transform boundary > > If the clip change is caused by the object having transform, we should > not ignore the clip change because it actually affects descendants. > > Bug: 790560 > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 > Change-Id: Ifc05835e406eb265717128129c8095f3e52bb864 > Reviewed-on: https://chromium-review.googlesource.com/822919 > Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> > Reviewed-by: Chris Harrelson <chrishtr@chromium.org> > Cr-Commit-Position: refs/heads/master@{#523653} TBR=wangxianzhu@chromium.org,trchen@chromium.org,chrishtr@chromium.org Change-Id: Id760d5784db00715f5f16cd0eeed0cb4e2812688 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 790560 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Reviewed-on: https://chromium-review.googlesource.com/822138Reviewed-by:
Victor Miura <vmiura@chromium.org> Commit-Queue: Victor Miura <vmiura@chromium.org> Cr-Commit-Position: refs/heads/master@{#523695}
-
junweifu authored
Create Windows bitmap from detected SkBitmap that is Rgba8/Bgra8 color type, then convert the pixel format SoftwareBitmap to Gray8/Nv12 that is supported in current version. Detect faces asynchronously with the created Windows bitmap, and pause incoming method call to prevent the Detect function from being called before the AsyncOperation completes. Add unittest to test the result of detecting faces. Split original large CL[1] up in smaller subpatches including this CL. Link Face Detection demo[2] here. [1] https://chromium-review.googlesource.com/c/chromium/src/+/708336 [2] https://codepen.io/miguelao/pen/PmJWro BUG=767021 Cq-Include-Trybots: master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win10_chromium_x64_rel_ng Change-Id: If2f93ede374bdebc6921cee4b0c9a09df879b693 Reviewed-on: https://chromium-review.googlesource.com/787790Reviewed-by:
Miguel Casas <mcasas@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Commit-Queue: Junwei Fu <junwei.fu@intel.com> Cr-Commit-Position: refs/heads/master@{#523694}
-
Devlin Cronin authored
Update the include rules for //extensions. There are many test-only exceptions (including quite a few that violate layering). These rules are currently applied for all tests in //extensions, but most only need to be applied to //extensions/browser. Trim the //extensions/DEPS include list to only have includes that are included from multiple subdirectories, and move the rest into //extensions/browser/DEPS and //extensions/renderer/DEPS. Also remove a few includes that were no longer necessary, or didn't exist. Bug: 773004 Change-Id: Ib6e705faae4472369858102e8a9ebe555e2a8d66 Reviewed-on: https://chromium-review.googlesource.com/820937 Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#523693}
-
cm.sanchi authored
From discussion on chromium-dev, it was decided to remove 'chrome' namespace. See the discussion about this that has happened in https://groups.google.com/a/chromium.org/d/topic/chromium-dev/pcgYLSJsii8/discussion TBR=sky@chromium.org Bug: 289619 Change-Id: Ic03e825bf2a65ec1edff0bd2c9e36b2a8c24c39b Reviewed-on: https://chromium-review.googlesource.com/821831 Commit-Queue: Chandramouli Sanchi <cm.sanchi@samsung.com> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#523692}
-
Patti authored
ShouldShowPermission() in page_info.cc always returns true for geolocation on Android because Android does additional filtering to check if geolocation should be shown due to DSE settings. The test PageInfoTest.NonFactoryDefaultAndRecentlyChangedPermissionsShown touches upon this in testing, but was coincidentally passing on Android because it later expects geolocation to be shown regardless by changing it to a non-default setting. Fix by changing the vector of the expected permissions to a std: :set instead. Change-Id: I9841563ddb6add41ea6f4d2edea4569d16b879a9 Reviewed-on: https://chromium-review.googlesource.com/813439Reviewed-by:
Raymes Khoury <raymes@chromium.org> Commit-Queue: Patti <patricialor@chromium.org> Cr-Commit-Position: refs/heads/master@{#523691}
-
webrtc-autoroll@chromium.org authored
https://webrtc.googlesource.com/src.git/+log/199942f3e6d4..3fe1b15413a6 $ git log 199942f3e..3fe1b1541 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc The AutoRoll server is located here: https://webrtc-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=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I19fea3155f2ad4323614ad88e27b87af0b607f07 Reviewed-on: https://chromium-review.googlesource.com/823682Reviewed-by:
WebRTC Roll Bot <webrtc-autoroll@chromium.org> Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#523690}
-
chaopeng authored
Currently the receiving of WM_MOUSEWHEEL messages lags when we open a URL on a new window until we resize. The issue was introduced by r384698. That CL change wait for MOUSE_EVENT to wait for SENDMESSAGE in MessagePumpForUI::WaitForWork to optimize the GPU thread. But the GPU thread does not use MessagePumpForUI anymore so it should be OK to just remove that code. The updated version of Chrome was tested to make sure that the hangs that were occasionally seen when using chrome://tracing are still gone. After this CL chaopeng@ need to monitor to make sure that the crash rate not increase. Bug: 713907, 596190 Change-Id: I7545a867de4851acd4e57c9b8c7a3dd05def1dd8 Reviewed-on: https://chromium-review.googlesource.com/809829 Commit-Queue: Jianpeng Chao <chaopeng@chromium.org> Reviewed-by:
Bruce Dawson <brucedawson@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#523689}
-
Xiaohan Wang authored
These tests were flaky and disabled. But we have not been able to repro the issue locally, nor understand the root cause of the issue. Part of the problem is that the top part of the test log is truncated, which has been fixed in a previous CL. Now we need to reproduce the test failure to be able to examine the logs for investigation. This CL reenables those flaky tests by reverting two CLs that disabled those tests: This reverts commit 9f4bee97. Revert "Disable ECKEncryptedMediaTest.Renewal on linux asan" This reverts commit b023b4e8. Revert "Disable EncryptedMediaTest.Playback_VideoClearAudio_WebM on Linux asan" TBR=jrummell@chromium.org,cfroussios@chromium.org BUG=793426 Change-Id: I7441ccf1d89ea81d396ba0ab84a77eb41c9bf700 Reviewed-on: https://chromium-review.googlesource.com/823572Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#523688}
-
Chris Watkins authored
This CL only touches base headers (the headers were accidentally excluded from the previous change to cc files). This change was done using clang-tidy as described here: https://chromium.googlesource.com/chromium/src/+/lkcr/docs/clang_tidy.md BUG=778959,778957 Change-Id: I4d3b4ca1218b835a671c2e0c8f31c2b528e25a0c Reviewed-on: https://chromium-review.googlesource.com/813316Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Chris Watkins <watk@chromium.org> Cr-Commit-Position: refs/heads/master@{#523687}
-
Chromium WPT Sync authored
Using wpt-import in Chromium fe678147. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/7534 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: I2f32e481b487712f35e114f73a349bfa8abe03c7 Reviewed-on: https://chromium-review.googlesource.com/822304 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@{#523686}
-
kylechar authored
Always turn on surface synchronization when the --enable-viz flag is present. Disable a few extra tests which now fail in the new configuration. The --enable-surface-synchronization flag was added to an FYI trybot to check for failures. No failures were observed over the weekend. Remove the now unneeded flag from the FYI trybot. Bug: 793302 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I6bc0aff21d968ba5ee38f7453fbd7b5779efb34a Reviewed-on: https://chromium-review.googlesource.com/812409 Commit-Queue: kylechar <kylechar@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Cr-Commit-Position: refs/heads/master@{#523685}
-
Katie Dektar authored
These simplify recording a boolean value and are more readable. Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I77165d0f6aaa760c7126d7c29083f139600fcf5d Reviewed-on: https://chromium-review.googlesource.com/820631 Commit-Queue: Katie D <katie@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#523684}
-
Ted Meyer authored
Basically we always scale up, rather than scale width, which is what the spec was changed to require about a year ago. BUG=606039 Change-Id: Iebfed41771a5b11bad6daf3655de750cce4c6cf8 Reviewed-on: https://chromium-review.googlesource.com/813201Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Ted Meyer <tmathmeyer@chromium.org> Cr-Commit-Position: refs/heads/master@{#523683}
-
Nicholas Verne authored
the renderer. As a result of this, several intermediate classes that passed MojoPipeHandle over IPC now pass the URLLoaderFactoryPtr over mojo. Also, AppCacheDispatcherHost no longer needs to be an IPC::Sender. Further simplifications of the AppCache class hierarchy should now be possible, but are out of scope for this CL. Bug: 611938 Change-Id: I5c05f66fbbab0dc4a2afa1aa6979b2ba0ff4f1f4 Reviewed-on: https://chromium-review.googlesource.com/810345Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Joshua Bell <jsbell@chromium.org> Reviewed-by:
Sam McNally <sammc@chromium.org> Commit-Queue: Nicholas Verne <nverne@chromium.org> Cr-Commit-Position: refs/heads/master@{#523682}
-
chrome-release-bot authored
TBR=anantha@chromium.org Change-Id: Ia6333c6b1e8851ea565b82fc7a826865c2cbfc44 Reviewed-on: https://chromium-review.googlesource.com/822873Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#523681}
-
Dmitry Gozman authored
Fragment is a piece of DOM generated from a template string, to be used instead of manual dom construction. - Allows functions ("binds") to be executed during instantiation: <div ${(fragment, node) => node._owner = fragment} />. - Exposes selected elements via $: <div $=myName />. - Toggling states which affect arbitrary attributes: <div s-selected-attr=value /> changes the value of "attr" to "value". - Inserting elements into markup: <div>Some ${myDiv} text</div>. - Inserting text into markup: <div>Some ${myText} text</div>. - Composed attribute names: <div ${foo ? 'foo' : 'bar'}-name=value />. - Transforms <x-shadow> to shadow root: <div><x-shadow>dark dom here</x-shadow></div>. - Caching preprocessed template if needed. Bug: none Change-Id: Ib1eb7c10b0f3f4aaebf98c022697111f38d55f9f Reviewed-on: https://chromium-review.googlesource.com/809952Reviewed-by:
Andrey Lushnikov <lushnikov@chromium.org> Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#523680}
-
Noel Gordon authored
Speculative fix: back off from using -O3 on Android for the inflate chunk copy code to see if -O3 caused the perf bot regression on the Android One bot, http://bit.ly/2AfaxhG Bug: 772870 Change-Id: I06bd941224dacbc5d7024a65934faaa53e4a4ce5 Reviewed-on: https://chromium-review.googlesource.com/823503 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Mike Klein <mtklein@chromium.org> Reviewed-by:
Chris Blume <cblume@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#523679}
-
dpapad authored
Bug: 790755 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I853f3765273c34fbd057708c745b5bc807d8dc3b Reviewed-on: https://chromium-review.googlesource.com/822736Reviewed-by:
Dave Schuyler <dschuyler@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#523678}
-
Makoto Shimazu authored
This patch implements a part of idle timer's logic. Fetch events coming through mojom::ControllerServiceWorker after the worker has requested termination to the browser should be queued until the worker receives the next event or StopWorker message from the browser. If the worker receives the next event from the browser, the queued events should run before the event from the browser, otherwise the order of the events would be messed up. If the worker receives StopWorker message instead, the worker shouldn't invoke the callback for the fetch event, and silently should disconnect the pipe to the clients. Bug: 774374 Change-Id: I6f3c90405ff037fe07df0310e2234cb81cb2ead9 Reviewed-on: https://chromium-review.googlesource.com/816458 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#523677}
-
Alexei Filippov authored
Change-Id: I77f35482e9a4dc6468bab404ba3534a1ca301487 Reviewed-on: https://chromium-review.googlesource.com/823573 Commit-Queue: Alexei Filippov <alph@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#523676}
-
Peter Collingbourne authored
We cannot find library dependencies to add to a target's ordered libraries list by searching the build directory because the build directory may contain libraries that appear to be dependencies but are in fact unrelated libraries that share a name with a system library that the target depends on. For example, we may be building a target that depends on the system's libGLESv2.so, and the build directory may contain a library named libGLESv2.so as a result of building ANGLE. The resulting APK would try to load libGLESv2.so using the crazy linker, which might fail to load the system's library. The solution is to use GN's write_runtime_deps feature to export the runtime dependency list from the build system and create an ordered libraries list from that. Bug: 742655 Change-Id: Ifdbf3de6b78b56af22dbfe0d12d80db5a48fe9db Reviewed-on: https://chromium-review.googlesource.com/821600Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: Peter Collingbourne <pcc@chromium.org> Cr-Commit-Position: refs/heads/master@{#523675}
-
Pavel Feldman authored
This change enables remote users to define and style printing headers and footers. Proposed API exposes following element classes: 'date', 'title', 'url', 'pageNumber', 'totalPages' ...that can be used to tag template elements. Printing subsystem will substitute text content of these elements with the actual values. Header and footer templates will be provided over the remote debugging protocol and injected into the preview later. Change-Id: Ia3d1737d532b7f951007a0260c4936fcac0276fc Reviewed-on: https://chromium-review.googlesource.com/813177 Commit-Queue: Pavel Feldman <pfeldman@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#523674}
-
Peter Boström authored
Copies the strings and string descriptions verbatim but makes the use_titlecase entry In Title Case. The Restore label is the same in both but an additional copy is made to make sure that other languages could title case it if appropriate. Bug: chromium:698785 Change-Id: I923457767c155cdf4c147402a0ca7bc8376319fd Reviewed-on: https://chromium-review.googlesource.com/822136Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#523673}
-
Tetsui Ohkubo authored
Deprecated Shortcut notification was using cogwheel icon. This CL changes the notification to use more specific keyboard icon. TEST=manual (Press Shift-Esc) BUG=792589 Change-Id: I62ed098ade7f3fd3d03bd140553190fed5293bb4 Reviewed-on: https://chromium-review.googlesource.com/818866 Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#523672}
-
Will Chen authored
This is a part-automated / part-manual process to migrate these /startup/ tests to the new integration test framework: 1. extract the precondition activities into an html file for the inspected window. 2. add TestRunner.setupStartupTest(...) as the first line of the js test to navigate to the inspected window before the DevTools session is started. 3. add testRunner.navigateSecondaryWindow(...) once the inspected page is done w/ the precondition activities. Bug: 667560 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I708c0719aa8ccdb38161e558fd535de12e7ccf9f Reviewed-on: https://chromium-review.googlesource.com/821517 Commit-Queue: Will Chen <chenwilliam@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#523671}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/1228ae247f21..94ed4456bfe6 $ git log 1228ae247..94ed4456b --date=short --no-merges --format='%ad %ae %s' 2017-12-13 dsinclair Move remaining Script_ methods to CJX_Object 2017-12-13 dsinclair Move Script_Som_* methods to CJX_Object 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: I133e00c2d76181726aab23e87b4a7bcfee3a2d67 Reviewed-on: https://chromium-review.googlesource.com/822763 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#523670}
-
chromeos-commit-bot@chromium.org authored
TBR=chrome-os-gardeners@google.com Change-Id: I1bde77f69c41fae4e05b65b5be723462f7bb66fd Reviewed-on: https://chromium-review.googlesource.com/823683Reviewed-by:
ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#523669}
-
John Budorick authored
This reshards the suites w/ 90th percentile task times > 10 minutes on linux_chromium_asan_rel_ng. Bug: 793993 Change-Id: Id269a3a2466af43956e11b00e25637e02ba5f410 Reviewed-on: https://chromium-review.googlesource.com/822183 Commit-Queue: John Budorick <jbudorick@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#523668}
-
John Abd-El-Malek authored
Bug: 793432 Change-Id: Ic88a3aeedcb5c3d0f06f0e7da8edb3ef6433b155 Reviewed-on: https://chromium-review.googlesource.com/820672 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#523667}
-
Tetsui Ohkubo authored
Previously, if both the title and the message for a progress bar notification is long enough, the implementation showed the full text of the title and elided the message by following Android behavior. However, in Download notification, it elides "n mins left" text (See screenshot in the bug), which is not preferable. This CL changes the behavior to prioritize the message and elide the title if there is not enough space. TEST=manual BUG=789924 Change-Id: I9e8da6c9c53ce1409b6d2d7aa05ae2bd62ca854b Reviewed-on: https://chromium-review.googlesource.com/804982Reviewed-by:
Yoshiki Iguchi <yoshiki@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#523666}
-
Brian White authored
This is debug code to help locate the source of a problem that is causing files to not be processed. Bug: 760317 Change-Id: I53d950204d76dfd34dfee7b1d9db184f70c40a8e Reviewed-on: https://chromium-review.googlesource.com/823099 Commit-Queue: Brian White <bcwhite@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#523665}
-
Eric Karl authored
ClientDiscardableTextureManager and ClientTransferCache are called from special Threadsafe* functions on ContextSupport without the context lock held. We need locking in these cases to prevent data races which are currently possible. Bug: 794293 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: I70c5a23dc2bd55cb77fb29c3c9057e8c37211e8c Reviewed-on: https://chromium-review.googlesource.com/823232 Commit-Queue: Eric Karl <ericrk@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#523664}
-
Bill Budge authored
- V8 is moving to a read/write read/execute system. We won't need read/write/execute and should remove it from Chrome now. Bug: chromium:756050 Change-Id: Icc8a3ec334535c460e74a413cb265de2872db9f5 Reviewed-on: https://chromium-review.googlesource.com/822514Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#523663}
-
Jared Saul authored
Bug: Change-Id: I2f3ba3920dcb92a093359f979a40888f36840b64 Reviewed-on: https://chromium-review.googlesource.com/822184Reviewed-by:
Mathieu Perreault <mathp@chromium.org> Commit-Queue: Mathieu Perreault <mathp@chromium.org> Cr-Commit-Position: refs/heads/master@{#523662}
-
Sam McNally authored
Move registration of the BackgroundFetchService interface from RenderProcessHost to RendererInterfaceBinders so its implementation can receive a browser-tracked origin for its renderer-process client. Add support for move-only types to base::AutoReset. Use it in the BackgroundFetchServiceImpl test to swap in a service instance with a different origin for the part of the test that previously passed a different origin to the mojo interface. Bug: 779444 Change-Id: I68737625cb5cd0a2843eeb46643aad3b4f852d8c Reviewed-on: https://chromium-review.googlesource.com/788641Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Sam McNally <sammc@chromium.org> Cr-Commit-Position: refs/heads/master@{#523661}
-
Tetsui Ohkubo authored
As new style notification is launched in M62, and IsNewStyleNotificationEnabled() returns true by default on Chrome OS, we no longer have to check IsNewStyleNotificaitonEnabled(). This CL makes following changes. - Remove IsNewStyleNotificationEnabled checks under chrome/browser. (*.cc) - Remove icons that are no longer used. (*.png, *.icon) Icons that are not removed are referenced in somewhere. Bug: 757703 Change-Id: Ibfd26747c05d516140026718c093f18bac443cfe Reviewed-on: https://chromium-review.googlesource.com/810348Reviewed-by:
Evan Stade <estade@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Reviewed-by:
Dan Erat <derat@chromium.org> Reviewed-by:
Pavel Yatsuk <pavely@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#523660}
-