- 25 Sep, 2018 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/74c179bb9bb9..b36a4816a546 git log 74c179bb9bb9..b36a4816a546 --date=short --no-merges --format='%ad %ae %s' 2018-09-25 jmadill@chromium.org Vulkan: Add OpenGL line segment rasterization. 2018-09-25 fjhenigman@chromium.org Vulkan: work around vkCmdCopyBuffer bug. Created with: gclient setdep -r src/third_party/angle@b36a4816a546 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll 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=syoussefi@chromium.org Change-Id: I3e9bd5449538329cc8b497355d67757c0c5ab837 Reviewed-on: https://chromium-review.googlesource.com/1243543Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#594017}
-
Mikel Astiz authored
Now that sessions have their own KeyedService, we can migrate away from ProxyModelTypeControllerDelegate, which was needed before for the subtle reason of avoiding a dependency cycle during initialization of ProfileSyncService. This removes a transitional state and simplifies reasoning about the datatype's state (controller and bridge are always in a consistent state). Bug: 883199 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: Ia52284c8eb61314486807a626580dbe132401051 Reviewed-on: https://chromium-review.googlesource.com/1243086 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#594016}
-
rbpotter authored
Fix this test to pass with Polymer 2. Bug: 875452 Change-Id: Ic2f9c38409f3659b0b4cfee7abe51e2ea562903f Reviewed-on: https://chromium-review.googlesource.com/1239717Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#594015}
-
Christian Fremerey authored
Add an API to the video capture service that allows observers to register and receive events whenever virtual devices are added or removed. Design Doc: https://docs.google.com/document/d/1CW6wslQFvV9FMIMkpyCzh2TcASr7LLXrawAtlgSecg8/edit?usp=sharing Test: services_unittests --gtest_filter=VideoCaptureServiceDeviceFactoryProviderTest* Bug: 885256 Change-Id: I49dfe298bc4d19d9b4b417ec658b899b4898ce1f Reviewed-on: https://chromium-review.googlesource.com/1237378Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Emircan Uysaler <emircan@chromium.org> Commit-Queue: Christian Fremerey <chfremer@chromium.org> Cr-Commit-Position: refs/heads/master@{#594014}
-
rbpotter authored
- Make cr-input full height of radio button so there is no border area where the hand pointer appears. - Ignore pointer events on link container when disabled. - Fix bug where "----" was considered valid input in pages. Bug: 884603, 887823, 888493 Change-Id: Ie9b7f2c42b12d7e2a7026f405b1a61b774578589 Reviewed-on: https://chromium-review.googlesource.com/1242037Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#594013}
-
Xi Cheng authored
1) Change DetermineImageDirectory() from being a private class member to being in the anonymous space; 2) Make RegisterTemporaryImage() early return when there is no image data. 3) Clean up the headers. Bug: 888276 Change-Id: Ib7006121d279bfc8147ad6ae71382a95b94d05be Reviewed-on: https://chromium-review.googlesource.com/1239728 Commit-Queue: Xi Cheng <chengx@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Cr-Commit-Position: refs/heads/master@{#594012}
-
Francois Doray authored
This method is invoked when all the frames in a process are frozen. In an upcoming CL, we will empty the working set of a process when all its frames are frozen. Bug: 885293 Change-Id: I3f3d15ccb605d47f71cbfb30dd8e88121c70dbe2 Reviewed-on: https://chromium-review.googlesource.com/1236461Reviewed-by:
Sigurður Ásgeirsson <siggi@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#594011}
-
Christian Biesinger authored
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: Iabb07283ec32b93bd9ec0a4ace317645410c8a64 Reviewed-on: https://chromium-review.googlesource.com/1236932 Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#594010}
-
Jérôme Lebel authored
Adding item/cell to display a cell with a image on the left, a multiline title and an optional multiline detail text. This item/cell is used in crrev.com/c/1231333 to show sync error from: https://docs.google.com/presentation/d/1cZfr5FGWGSy0PNaQ8uzik0alLAH-5glh1vsb030vha8/edit?ts=5aba5455#slide=id.g3e63bc1501_10_15 See: https://drive.google.com/open?id=1i0FEUwRI4RsRIYYHjxoqFyFxk3q7JfMZ Collection cell catalog: https://drive.google.com/open?id=16G_97-5oZZV1TKrxM3JYqXXISTXn-hTV Bug: 849754 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: Ieb3c5f0fcb814d5d8808a096e2a01d280aafa151 Reviewed-on: https://chromium-review.googlesource.com/1236338 Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#594009}
-
Avi Drissman authored
BUG=887925 TEST=WebContentsImplBrowserTest.JavaScriptDialogsNormalizeText Change-Id: Ibea163e5cc8822c427d2278717b5d1c8316d6be4 Reviewed-on: https://chromium-review.googlesource.com/1243226Reviewed-by:
Leonard Grey <lgrey@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#594008}
-
Daniel Rubery authored
Currently, the values are all offset by one. The "SUCCESS" bucket in UMA corresponds to the "UPLOAD_SUCCESS" enum value, which makes the top-level metric look okay. But the "UPLOAD_SUCCESS" bucket in UMA is actually the "UPLOAD_CANCELLED" enum value, which is very misleading. Change-Id: I0732ebb0070b064c27d23c1d81c4072220df7725 Reviewed-on: https://chromium-review.googlesource.com/1243667Reviewed-by:
Nathan Parker <nparker@chromium.org> Commit-Queue: Daniel Rubery <drubery@chromium.org> Cr-Commit-Position: refs/heads/master@{#594007}
-
Kevin Strohbehn authored
Bug: 876940 Change-Id: I3838f52b4f2501e488d4c4a9efeb08f1d26b81b4 Reviewed-on: https://chromium-review.googlesource.com/1242455Reviewed-by:
Weidong Guo <weidongg@chromium.org> Commit-Queue: Kevin Strohbehn <ginko@google.com> Cr-Commit-Position: refs/heads/master@{#594006}
-
Aaron Leventhal authored
Bug: 889089 Change-Id: I8dccdccb7de9f742cf6657106739039b59fcd824 Reviewed-on: https://chromium-review.googlesource.com/1243584 Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#594005}
-
Fabrice de Gans-Riberi authored
The sized_buffer mode was renamed to buffer in Garnet. This change keeps the service implementation in sync. See: https://fuchsia-review.googlesource.com/c/garnet/+/201201 https://fuchsia-review.googlesource.com/c/garnet/+/201203 Bug: 874155 Change-Id: I74061afd6e92d0397c286391efbd36f02373008c Reviewed-on: https://chromium-review.googlesource.com/1243569 Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#594004}
-
Ben Kelly authored
The ashmem_create_region() API documentation indicates the size should be specified in "page-aligned bytes". Bug: 887357 Change-Id: I874914f57408663a380d39d61fddbaf986e6ff24 Reviewed-on: https://chromium-review.googlesource.com/1238673Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Reviewed-by:
Alexandr Ilin <alexilin@chromium.org> Commit-Queue: Ben Kelly <wanderview@chromium.org> Cr-Commit-Position: refs/heads/master@{#594003}
-
Peter K. Lee authored
This CL rolls MDC components to its origin/stable branch and fixes the BUILD.gn file to contain new files. This CL replaces http://crrev/c/1238472 Bug: 883921 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I421525fd4194cd292004952767753b241cd38da0 Reviewed-on: https://chromium-review.googlesource.com/1241975 Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#594002}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/15052410fda2..297caafc5805 git log 15052410fda2..297caafc5805 --date=short --no-merges --format='%ad %ae %s' 2018-09-25 benjaminwagner@google.com [infra] Enable QuadroP400 bots on rack:2 Created with: gclient setdep -r src/third_party/skia@297caafc5805 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll 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-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel TBR=bsalomon@chromium.org Change-Id: I718f199165bcd9d006146790b22ba693dfd951af Reviewed-on: https://chromium-review.googlesource.com/1243604Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#594001}
-
danakj authored
So don't try to handle both anymore. DCHECK this is the case. R=dcheng@chromium.org, piman@chromium.org Bug: 720116, 419087 Change-Id: Ia10e77f6c81da37eacb1000ca679d1055b2ea372 Reviewed-on: https://chromium-review.googlesource.com/1239004Reviewed-by:
Antoine Labour <piman@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#594000}
-
Orin Jaworski authored
This feature param prepares the way for experimentation with new omnibox pedal development. Two pedal suggestion modes are already implemented, ready to be gated by this feature flag. A subsequent CL will hook this feature into chrome://flags . Change-Id: I04ec30cb01308f1e98be87c3bbe18592ee6eb30b Reviewed-on: https://chromium-review.googlesource.com/1235200 Commit-Queue: Orin Jaworski <orinj@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#593999}
-
Tanya Gupta authored
R=ramyan@chromium.org Bug: 882822 Change-Id: I1a70d25d39a927d0a07a2a77420f0b696513c788 Reviewed-on: https://chromium-review.googlesource.com/1241795Reviewed-by:
Ramya Nagarajan <ramyan@chromium.org> Reviewed-by:
Kristi Park <kristipark@chromium.org> Commit-Queue: Tanya Gupta <tgupta@chromium.org> Cr-Commit-Position: refs/heads/master@{#593998}
-
David Black authored
Per UX guidance, mic should open for both launcher entry points when in tablet mode. Otherwise, we will use keyboard input modality. Later we will also factor in user preference. Bug: b:116509014 Change-Id: I5961f48aa2c8c808dde1a55307e8fd149fabe22c Reviewed-on: https://chromium-review.googlesource.com/1241760Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#593997}
-
Xi Cheng authored
While "Double Pointer" is commonly used in C, let's use (implicit) move semantics in the modern C++ era. Bug: 886957 Change-Id: I15693413a8a2474fb8200fc1778d4f9fe4656905 Reviewed-on: https://chromium-review.googlesource.com/1242298 Commit-Queue: Xi Cheng <chengx@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#593996}
-
Clark DuVall authored
The error handling in WebRequestProxyingURLLoaderFactory was causing some objects not to be destroyed. This now uses a pattern similar to signin::ProxyingURLLoaderFactory. Bug: 888707 Change-Id: I033bd11c4281b238cebeb7924cc334a0264ff3b4 Reviewed-on: https://chromium-review.googlesource.com/1242297Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#593995}
-
Robert Sesek authored
Bug: 832676 Change-Id: Ibeba925bbd23e5eb33aeaf748bc2f48161db6a96 Reviewed-on: https://chromium-review.googlesource.com/1243727Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#593994}
-
Ryan Harrison authored
Currently if a search term in the PDF text has been broken over two lines by a soft hyphen, find will not correctly identify it as a match. This is rooted in the fact that the result of FPDF_GetText includes a marker for soft-hyphens, 0xFFFE, which causes the match to fail. This CL adds in filtering this character from the text being searched over, so that these matches can pass. This requires changes in the SearchUsingICU method to strip ignorable characters from the string before searching, and correctly converting the results back into the non-stripped index space. Ranges also have had filtering for 0xFFFE added in, so that the highlights created by searching are properly placed. BUG=chromium:788799 Change-Id: I06c8181358cdebe6454c36437065592820637808 Reviewed-on: https://chromium-review.googlesource.com/1234998 Commit-Queue: Ryan Harrison <rharrison@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Henrique Nakashima <hnakashima@chromium.org> Cr-Commit-Position: refs/heads/master@{#593993}
-
Eugene But authored
Bug: 887273 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I99e171f64a20c7dba9f8984d05cb509b00d01ea5 Reviewed-on: https://chromium-review.googlesource.com/1243166Reviewed-by:
Danyao Wang <danyao@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#593992}
-
sczs authored
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I5775e79501baf6787e86c275ea7dbdbff5f6a5f6 Reviewed-on: https://chromium-review.googlesource.com/1239545 Commit-Queue: Sergio Collazos <sczs@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#593991}
-
Jordan Demeulenaere authored
Bug: 806868 Change-Id: I4d72bf67afb654ee6752c7a3c02a03f9b3c7dc80 Reviewed-on: https://chromium-review.googlesource.com/1242459Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org> Cr-Commit-Position: refs/heads/master@{#593990}
-
John Chen authored
This CL is one step in making InitSession command W3C compliant. It expands Capabilities object to include all capabilities that are specified in the W3C spec. Changes include: * Re-ordered fields in Capabilities into W3C section and ChromeDriver specific section. Entries in W3C section are listed in the same order as they appear in W3C spec. * Added browserName, browserVersion, and platformName capabilities. They will be needed for proper firstMatches handling. * Added fields needed for timeouts capability. Parsing and using of these fields will be added in an upcoming CL. * Replaced legacy name unexpected_alert_behaviour with W3C-compliant name unhandled_prompt_behavior. * Modified Capabilities::Parse to accept all capability values, including those not yet supported by ChromeDriver. Added another method Capabilities::IsSupported to check for unsupported options. This is needed for upcoming W3C-compliant capability validation. Bug: chromedriver:1997 Change-Id: I5f53341cf47f77a1853f40028b1efe6f0b69c611 Reviewed-on: https://chromium-review.googlesource.com/1241213 Commit-Queue: John Chen <johnchen@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Cr-Commit-Position: refs/heads/master@{#593989}
-
Ahmed Fakhry authored
This CL is a preparation for turning the feature on by default. It fixes the following issues: - Editable elements focus change notifications are broadcasted to all listeners from all render view hosts. Handle it only for the active webcontents. - The value of DoBrowserControlsShrinkBlinkSize() should never change while scrolling is in progress. It should also return false when the feature is disabled regardless of the current shown ratio. - A gray patch that used to show at the bottom of the page. We should never set the shown ratio of the browser immediately without asking the renderer to do it first and call us when done. For that we need to defer switching the status to disabled until the renderer calls us with the final shown ratio value, only then we know both the browser and the renderer are synchronized and we can layout the browser view. - The active webcontents can be null in unit tests. - Expose top-controls related fields from behind OS_ANDROID. BUG=867063, 885223, 884453 TEST=Manual, passes existing tests, passes all tests with the feature enabled by default. Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I3bc7747a745619dd6f789d25a2a1609bbe8f15db Reviewed-on: https://chromium-review.googlesource.com/1217754 Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:Daniel Cheng <dcheng@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#593988}
-
Maria Tarasova authored
If browser_tests is built with tsan enabled, Thread Sanitizer reports data races in browsing_data_remover_test_util. There are two threads to concurrently access the shared data, the test’s main thread and a thread controlled by TaskScheduler singleton. Both access browsing_data_remover_done_ and/or flush_for_testing_complete_ fields to break run loop and exit test after some cleanup occurs. In TaskScheduler’s thread, the write access is done by the lambda function. The proposed solution is to use the lambda call only to report the asynchronous event to main thread, which becomes the only one to access the fields implementing run loop break logic. R=sky@chromium.org Bug: 885188 Change-Id: I706edb24ed362444cf46e558b1d54389179884da Reviewed-on: https://chromium-review.googlesource.com/1233613Reviewed-by:
Robert Liao <robliao@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#593987}
-
Nico Weber authored
Bug: 101600 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I39cc247a432861bf8e7ac815dc2d1a2834a2cf08 Reviewed-on: https://chromium-review.googlesource.com/1240975Reviewed-by:
danakj <danakj@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#593986}
-
Andreas Haas authored
Bug: v8:7295 v8:8015 Change-Id: I0b6a6c6348d446b318329db9a91ac1a148eee7ca Reviewed-on: https://chromium-review.googlesource.com/1238176Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#593985}
-
Robert Sesek authored
Bug: 845389 Change-Id: I33de543963096de0a4473252cc4017aa52b4a981 Reviewed-on: https://chromium-review.googlesource.com/1243623Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#593984}
-
Adithya Srinivasan authored
- Adds blink.mojom.FileSystemManager to the frame and worker registries, allowing it to be accessed directly from ExecutionContext::GetInterfaceProvider()->GetInterface() - Changes ownership of blink::FileSystemDispatcher: now owned by ExecutionContext - Omits render_frame_id_ from ChooseEntry - Removes WebFileSystem and content::FileSystemDispatcher Bug: 787281, 873661 Change-Id: Ie41d48c5a206b5d1f387fcee3bd47bc30a708f0f Reviewed-on: https://chromium-review.googlesource.com/1217506 Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Cr-Commit-Position: refs/heads/master@{#593983}
-
Jordan Demeulenaere authored
This flag was removed in the backend, so it is safe to remove here as well. Bug: 806868 Change-Id: Ibe9415dcb0f8415591fe59110fb99e1bcbfd3d5a Reviewed-on: https://chromium-review.googlesource.com/1243284Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org> Cr-Commit-Position: refs/heads/master@{#593982}
-
Lukasz Anforowicz authored
Bug: 887645 Change-Id: I0f05856b0473bb6c197d53184322faa1042b2f23 Reviewed-on: https://chromium-review.googlesource.com/1237253Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#593981}
-
Theresa authored
BUG=888980 Change-Id: Ic1699abfe2b1b911857557ccfdc04f49543c80d5 Reviewed-on: https://chromium-review.googlesource.com/1243046Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#593980}
-
Michael Lippautz authored
Removes a test that was checking that a JS wrapper was collected when doing a V8 GC. This relies on the fact that wrapper tracing does not see the back references in LiveNodeList::owner_. No-try: true Bug: 843903 Change-Id: I0586ba63f15d686a6e33c152d9eb9a8f08247cd5 Reviewed-on: https://chromium-review.googlesource.com/1243066Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#593979}
-
Justin DeWitt authored
This patch does the following: * Adds a method to determine the variation of ExploreSites that is enabled * Inflates the ExploreSites ViewStub with the layout appropriate for the variation * Renames several dimensions to make them specific for the Experimental variation of EoS. * Adds a TileGridLayout for ExploreSites to match the Suggestions TileGridLayout * Adds two ExploreSitesCategory TileView classes (regular and condensed) * Adds an ExploreSitesSection that asks for the catalog and inflates the category tiles. This does not properly hook up: * Icon generation/fetching * exact layout parameters. Bug: 888176 Change-Id: Ie7e1b9f66defeac745dce98853e06f10af88330a Reviewed-on: https://chromium-review.googlesource.com/1239658 Commit-Queue: Justin DeWitt <dewittj@chromium.org> Reviewed-by:
Cathy Li <chili@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#593978}
-