- 02 Oct, 2017 40 commits
-
-
Brett Wilson authored
The documentation only gave examples of calling base::Passed with a pointer. Change-Id: I86c516546b55e6dcd62f8c83cdff859cba64578d Reviewed-on: https://chromium-review.googlesource.com/688685 Commit-Queue: Albert J. Wong <ajwong@chromium.org> Reviewed-by:
Albert J. Wong <ajwong@chromium.org> Cr-Commit-Position: refs/heads/master@{#505808}
-
Corentin Wallez authored
Forcing the structure of the handlers to be like this will help avoid TOCTOU errors. BUG=602688 Change-Id: I50626770f3f123c40f45680661d97ca9e1416129 Reviewed-on: https://chromium-review.googlesource.com/695842Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Cr-Commit-Position: refs/heads/master@{#505807}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/1cfdb58521aa..951b1119d4c7 $ git log 1cfdb5852..951b1119d --date=short --no-merges --format='%ad %ae %s' 2017-10-02 dsinclair More define cleanup 2017-10-02 dsinclair Remove extraneous ;s 2017-10-02 art-snake Remove useless code. Created with: roll-dep src/third_party/pdfium Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls TBR=dsinclair@chromium.org Change-Id: I6f3289e09d69054e1f95d318c61a1010cf32b472 Reviewed-on: https://chromium-review.googlesource.com/695816 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#505806}
-
Maxim Kolosovskiy authored
This CL changes AutofillQueryResponseContents proto in order to support multiple field predictions. A field may have several server-side predictions. For example, for the same field we can have two different classifications: |EMAIL_ADDRESS| and |USERNAME| and both are important. This CL also updates how the response proto is parsed and the process of choosing the autofill predictions which are useful for password manager. The CL was almost done by Madalina (https://chromium-review.googlesource.com/c/chromium/src/+/664704). Here I applied a couple of comments from reviewer when Madalina's internship finished. Bug: 699530 Change-Id: I65b3ca8683e03b5d85049f3180971bcdeb1a1309 Reviewed-on: https://chromium-review.googlesource.com/677298Reviewed-by:
Maxim Kolosovskiy <kolos@chromium.org> Reviewed-by:
Roger McFarlane <rogerm@chromium.org> Reviewed-by:
Mathieu Perreault <mathp@chromium.org> Commit-Queue: Maxim Kolosovskiy <kolos@chromium.org> Cr-Commit-Position: refs/heads/master@{#505805}
-
Dmitry Gozman authored
BUG=761655 Change-Id: Ib3bde429ea024b97a8713daa1d870f10de69daff Reviewed-on: https://chromium-review.googlesource.com/692852Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#505804}
-
Tom Sepez authored
This seems to have been missed when these sources moved to the service_manager directory tree. Change-Id: I4443527ca72086bfacfa15b1593c514e669c7567 Reviewed-on: https://chromium-review.googlesource.com/695823Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#505803}
-
Will Chen authored
This is a very old testing tool that's no longer used by DevTools TBR=pfeldman@chromium.org Bug: 667560 Change-Id: Id25542b2fbf6a213546943a2ef86e1270e05e025 Reviewed-on: https://chromium-review.googlesource.com/692597Reviewed-by:
Will Chen <chenwilliam@chromium.org> Commit-Queue: Will Chen <chenwilliam@chromium.org> Cr-Commit-Position: refs/heads/master@{#505802}
-
Mostyn Bramley-Moore authored
Bug: 746953 Change-Id: Id2bf0dbb4c4c67a5905586c3cee28290dfc00da2 Reviewed-on: https://chromium-review.googlesource.com/696082Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Cr-Commit-Position: refs/heads/master@{#505801}
-
Dave Tu authored
Bug: 770009 Change-Id: I898ea3017e5a2bfff311026dee773810e92e9281 Reviewed-on: https://chromium-review.googlesource.com/691043Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: David Tu <dtu@chromium.org> Cr-Commit-Position: refs/heads/master@{#505800}
-
Christopher Grant authored
A mock call warning is interesting despite the fact that the test output says it's uninteresting. BUG= Change-Id: I34e2b2ca98685886f08f3adf411ea3252cd025f7 Reviewed-on: https://chromium-review.googlesource.com/695820Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Commit-Queue: Christopher Grant <cjgrant@chromium.org> Cr-Commit-Position: refs/heads/master@{#505799}
-
Adithya Srinivasan authored
The attribute check is the slowest check in that if statement, and it's likely for some of the other checks to fail first. Bug: 764618 Change-Id: I6dd694f9ace8fc5e9461a51f6eab8abdec32580b Reviewed-on: https://chromium-review.googlesource.com/695982Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#505798}
-
Devlin authored
This reverts commit b243824e. Reason for revert: Looks like this really is responsible. Once it landed in https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/63607, both BrowserTest.WindowOpenClose and ExtensionLoadingTest.RuntimeValidWhileDevToolsOpen started failing immediately. Original change's description: > Field trial testing config for StabilityDebugging > > This enables testing for the "record, collect on crash, process > postmortem without sending postmortem reports" configuration. > > This is a reland attempt. The previous attempt was reverted, but I don't > think it was the true culprit. See: > https://chromium-review.googlesource.com/q/Iba7b79d153a52fca73e4e943697ebcdf5332a500 > > Bug: 620813 > Change-Id: I5d9d38fdf4e97fb95d73e1db06c4f6d40d18ce17 > Reviewed-on: https://chromium-review.googlesource.com/695445 > Reviewed-by: Jesse Doherty <jwd@chromium.org> > Commit-Queue: Pierre-Antoine Manzagol (departed) <manzagop@chromium.org> > Cr-Commit-Position: refs/heads/master@{#505714} TBR=jwd@chromium.org,manzagop@chromium.org Change-Id: I62b586df4361de3f7e20cfb6ede180fff665cf5e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 620813 Reviewed-on: https://chromium-review.googlesource.com/696126Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#505797}
-
Matthew Jones authored
This change replaces the use of ChromeAnimation in ToolbarSwipeLayout with a CompositorAnimator. BUG=750381 Change-Id: I8c2f5cdfc9ca8f8e13960219cf39abd4e5d2218f Reviewed-on: https://chromium-review.googlesource.com/691282 Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#505796}
-
Max Moroz authored
It's a temporary solution while pending integration on LLVM side. An example of usage: ./coverage.py --command "out/coverage/zlib_uncompress_fuzzer \ -runs=0 $HOME/corpus/zlib_uncompress_fuzzer" --source "$PWD" \ --output "cov" <...> Done. The next steps would be: 1. cd cov && python -m SimpleHTTPServer 2. open http://127.0.0.1:8000/report.html Examples of reports: - small test target: http://go/cov-sample-1 - v8_script_parser_fuzzer: http://go/cov-sample-2 Bug: 759794 Change-Id: I57b8ffc7ff70d2415dc0d3796508e85f7c1f048b Reviewed-on: https://chromium-review.googlesource.com/677138Reviewed-by:Max Moroz <mmoroz@chromium.org> Reviewed-by:
Oliver Chang <ochang@chromium.org> Cr-Commit-Position: refs/heads/master@{#505795}
-
Krishna Govind authored
TBR=abdulsyed@chromium.org Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ifdf02cedc56bfe25736200e58de9aa0be7bf367f Reviewed-on: https://chromium-review.googlesource.com/695915Reviewed-by:
Krishna Govind <govind@chromium.org> Commit-Queue: Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#505794}
-
Daniel Cheng authored
This reverts commit 8adb9990. Reason for revert: breaking PolyGerrit Original change's description: > offsetParent fails to consider continuations when searching for parent. > > Block-level objects can have inline parents, but this triggers the > insertion of anonymous boxes and continuations, which offsetParent() > currently doesn't understand. > > This change lets offsetParent() understand continuations better. > If an anonymous continuation is encountered when finding ancestors, > it will use the split inline as the ancestor, and continue searching > from there. > > BUG=638177 > > Change-Id: I8850f9b5dacaffaa26b5f1c414f4f351709d7088 > Reviewed-on: https://chromium-review.googlesource.com/674875 > Reviewed-by: Emil A Eklund <eae@chromium.org> > Commit-Queue: Karl Anders Øygard <karlo@opera.com> > Cr-Commit-Position: refs/heads/master@{#503704} TBR=kochi@chromium.org,eae@chromium.org,karlo@opera.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 638177 Change-Id: I6e59c060ca0f290fbfabb7662f233424baab6564 Reviewed-on: https://chromium-review.googlesource.com/695843Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#505793}
-
Yuke Liao authored
This CL uses os.extsep to append extensions for the sake of cross platform compatability. Bug: Change-Id: Ic12d98df9c19d5114029dba3dde35b14216d6005 Reviewed-on: https://chromium-review.googlesource.com/695925Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#505792}
-
Carlos IL authored
Added an UploadLogToURL function to NetMetricsLogUploader, this contains all the logic that was in the existing UploadLog function, with the exception that it takes a URL as a parameter and uploads the log to that URL instead of the one passed in the constructor. The UploadLog function was modified so it now calls UploadLogToURL with the URL set to server_url (So UploadLog keeps the same functionality it had before this change). R=estark@chromium.org Bug: 769142 Change-Id: I0d1b40d76c1d2fcb8528f301fe488d5d10e268b7 Reviewed-on: https://chromium-review.googlesource.com/692510Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Emily Stark <estark@chromium.org> Commit-Queue: Carlos IL <carlosil@chromium.org> Cr-Commit-Position: refs/heads/master@{#505791}
-
Devlin Cronin authored
ExtensionService unittests construct a bunch of ExternalInstallInfos, but for some reason choose smart pointers over stack allocation (even though they aren't passed anywhere). Use stack allocation instead. Bug: 770007 Change-Id: Ic5a2ee425581b7f887b95a49d9409a5238d2394d Reviewed-on: https://chromium-review.googlesource.com/693396Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#505790}
-
yoshiki iguchi authored
This patch removed the cache (notification_views_map) from MessageCenterView. Instead, it refers MessageListView directly. As the investigation (see crbug.com/737858#3), the cache and the list view are sometimes inconsistent. To remove the inconsistency, this patch removes the cache. Then, the Issue 737858 should be fixed by this patch. Bug: 737858 Test: manual Change-Id: I7740e282018c2e964947296b53539ce8d91c432d Reviewed-on: https://chromium-review.googlesource.com/686214 Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Eliot Courtney <edcourtney@chromium.org> Reviewed-by:
Naoki Fukino <fukino@chromium.org> Cr-Commit-Position: refs/heads/master@{#505789}
-
Evan Stade authored
Bug: 766846, 769353 Change-Id: I48a8b72f787c4e96f517ee2ed60be32bc977f550 Reviewed-on: https://chromium-review.googlesource.com/689743 Commit-Queue: Evan Stade <estade@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#505788}
-
Lukasz Anforowicz authored
WebContents::GetRenderProcessHost is an API that doesn't behave as expected with out-of-process iframes, as multiple processes can be associated with a single WebContents. This CL replaces callers of this API, so that they go through an equivalent API that explicitly selects which frame's process is needed. This CL was started by using an ad-hoc clang-plugin that replaced existing callsites of WebContents::GetRenderProcessHost() with a call to something like wc->GetMainFrame()->GetProcess(). This was followed-up by manually adding an include of render_frame_host.h and git cl format and git cl lint and then a self-review with small tweaks (e.g. sometimes using a wc->GetRenderViewHost()->GetProcess() is more appropriate). This CL was uploaded by git cl split. R=wjmaclean@chromium.org Bug: 666525 Change-Id: If468bececcb1ea017a75b363c0208b3b6b182e97 Reviewed-on: https://chromium-review.googlesource.com/689108Reviewed-by:
James MacLean <wjmaclean@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#505787}
-
David Tseng authored
Bug: 709109 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I5a7f10b3cc0d62ad7baab56e380e77a860ca2a5a Reviewed-on: https://chromium-review.googlesource.com/677095Reviewed-by:
Oliver Chang <ochang@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Yuki Awano <yawano@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#505786}
-
Chromium WPT Sync authored
Using wpt-import in Chromium bef716c8. Build: https://build.chromium.org/p/chromium.infra.cron/builders/wpt-importer/builds/2537 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 Directory owners for changes in this CL: ksakamoto@chromium.org: external/wpt/resource-timing rouslan@chromium.org, mathp@chromium.org: external/wpt/payment-request TBR=qyearsley@chromium.org No-Export: true Change-Id: If89359128aa57606890bb52787d24e5a0ff141a1 Reviewed-on: https://chromium-review.googlesource.com/694982 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@{#505785}
-
W. James MacLean authored
If we route the GesturePinch events sent from RenderWidgetHostViewMac, then wheel event listeners in OOPIF subframes, and the PDF viewer, can intercept the pinch events in order to implement native pinch zoom behaviour. Bug: 739213 Change-Id: I50db39ef53a988b491b3b7e79fa66a392e86c9b7 Reviewed-on: https://chromium-review.googlesource.com/559650 Commit-Queue: James MacLean <wjmaclean@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Cr-Commit-Position: refs/heads/master@{#505784}
-
Mostyn Bramley-Moore authored
There is another FontMessageFilter in pepper_browser_font_singleton_host.cc Bug: 746953 Change-Id: I2ee0188ac38637b94f9e9aa792dccd42be7d8e9a Reviewed-on: https://chromium-review.googlesource.com/696101Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Cr-Commit-Position: refs/heads/master@{#505783}
-
David Reveman authored
The new version includes build fix for where math.h was not included correctly. TBR=adlr@chromium.org Bug: Change-Id: I8d7715c20e38dc18aed86be77b4036cf99f2f761 Reviewed-on: https://chromium-review.googlesource.com/693474 Commit-Queue: David Reveman <reveman@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Cr-Commit-Position: refs/heads/master@{#505782}
-
Christopher Grant authored
BUG= R=tiborg Change-Id: Id4180e06908fdb4e07bc73cec25d15204d119bdf Reviewed-on: https://chromium-review.googlesource.com/693222Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Commit-Queue: Christopher Grant <cjgrant@chromium.org> Cr-Commit-Position: refs/heads/master@{#505781}
-
Bin Zhao authored
We are trying to not call chrome.cast.channel.open / chrome.cast.channel.close if in-browser cast discovery is enabled. Currently message observer is only added in open function, if we skip opening, observer wont be added. This patch registered cast socket message observer in send function as well. Extension side change: cl/169416458 Bug: 752310 Change-Id: I96231c98bfbcf7163eee1f90869e4b5ae0941260 Reviewed-on: https://chromium-review.googlesource.com/664085Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Bin Zhao <zhaobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#505780}
-
Donn Denman authored
Disables the Smart Select Text Classifier for the Omnibox. This prevents a longpress gesture in the Omnibox on Android from suggesting "BROWSER" as an action in the Action Bar. BUG=768594 Change-Id: I2dccb6b7443d9891e4243d7288fd65beb9a9da0f Reviewed-on: https://chromium-review.googlesource.com/686060Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Donn Denman <donnd@chromium.org> Cr-Commit-Position: refs/heads/master@{#505779}
-
Gregory Chatzinoff authored
Removes unnecessary view controller parameters from APIs in preparation for boxing. These view Controllers are unnecessary since the |presentingViewController_| variable can be used instead. Bug: none Change-Id: I93805365a20438bfd40a5f6b9dd7b229f700df73 Reviewed-on: https://chromium-review.googlesource.com/691517Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Reviewed-by:
Rohit Rao (ping after 24h) <rohitrao@chromium.org> Commit-Queue: Gregory Chatzinoff <gchatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#505778}
-
Steven Holte authored
Bug: 755630 Change-Id: I7b6c1c890a68870311411b5f6c7e084052d80f7c Reviewed-on: https://chromium-review.googlesource.com/609588 Commit-Queue: Steven Holte <holte@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#505777}
-
Peter Boström authored
BUG=chromium:654115 R=bsep@chromium.org Change-Id: Iead94366cb68df77f396d92d1f54bc83677551be Reviewed-on: https://chromium-review.googlesource.com/691178 Commit-Queue: Peter Boström <pbos@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#505776}
-
Kevin McNee authored
In order to prevent a child frame from scrolling during an overscroll gesture in the root, we give the root a chance to consume the child's GestureScrollUpdate events before the child. With the new scroll latching behaviour, this is unnecessary, as the child would not consume scroll in this case. We now only filter the child's GestureScrollUpdates when scroll latching is disabled. Bug: 751782 Change-Id: I8aa7d652dda4d8bc5768d659019dd750bf44ad33 Reviewed-on: https://chromium-review.googlesource.com/675167Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Commit-Queue: Kevin McNee <mcnee@chromium.org> Cr-Commit-Position: refs/heads/master@{#505775}
-
Roger McFarlane authored
AngularJS can prefix the value in select elements with a type prefix. Autofill already supports teh "number:" prefix. This CL adds support for "string:". Bug: 770781 Change-Id: I5c2b272ca20a3f3a68e3445166c0df36ad5aa058 Reviewed-on: https://chromium-review.googlesource.com/695941 Commit-Queue: Roger McFarlane <rogerm@chromium.org> Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org> Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#505774}
-
Stephen McGruer authored
This tests the DCHECK fix that was applied in http://crrev.com/1c6d2c6 Bug: 467721, 766140 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I96a2f0e608f4bb22a03e6e6845219de63fba2423 Reviewed-on: https://chromium-review.googlesource.com/668957Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Robert Flack <flackr@chromium.org> Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#505773}
-
Qiang Xu authored
click changes: Regression comes from https://chromium-review.googlesource.com/c/chromium/src/+/603547 Tray bubble view is default not active, install pre target event handler (RerouteEventHandler) only when TrayBubbleView is opened by click on the corresponding tray, which must be triggered by PerformAction(). Bug: 765473 Test: device test and added test coverage Change-Id: Icca804f2d134f588d9901c38aaebb7a8d87f6f8e Reviewed-on: https://chromium-review.googlesource.com/677231 Commit-Queue: Qiang(Joe) Xu <warx@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#505772}
-
Vladislav Kaznacheev authored
Make MetalayerMode access Shell::Get()->highlighter_controller() directly (not via PaletteDelegate as it used to do). Move the code responsible for auto-exiting the metalayer mode from PaletteDelegateChromeOS to HighlighterController. This is the first step towards shrinking and possibly removing PaletteDelegate in favor of feature-specific mojo connections. Bug: 769996 Test: ash_unittests --gtest_filter=Palette* Change-Id: Ic2f9f79e5f45b30127c20152b36552c2c6118217 Reviewed-on: https://chromium-review.googlesource.com/692748 Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#505771}
-
Matthew Jones authored
BUG= Change-Id: I3af2eab75b5498dad128c792992b43dfed735745 Reviewed-on: https://chromium-review.googlesource.com/692986Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#505770}
-
Albert J. Wong authored
Some accidental uses of ADL for resolving PartitionAlloc API calls accidentally snuck in. This removes them. Bug: 766882 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I5998cfa3a13eafc444fe97ff447fbb8f35ef7b4e Reviewed-on: https://chromium-review.googlesource.com/686175Reviewed-by:
Albert J. Wong <ajwong@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Commit-Queue: Albert J. Wong <ajwong@chromium.org> Cr-Commit-Position: refs/heads/master@{#505769}
-