- 12 Apr, 2018 40 commits
-
-
Kenneth Russell authored
https://chromium.googlesource.com/external/khronosgroup/webgl.git/+log/aef0b3a..7c0541d Bug: 804118, 818336, 828262, angleproject:2381 Tbr: zmo@chromium.org, kainino@chromium.org, jdarpinian@chromium.org Cq-Include-Trybots: luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_angle_rel_ng;luci.chromium.try:win_angle_rel_ng Change-Id: I28b3d55e43f5633ffe4c8cd6e7e375dd2fba2b9b Reviewed-on: https://chromium-review.googlesource.com/1010910Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#550408}
-
David Jacobo authored
Authentication process for a variety of apps require ARC to send some of the traffic to Chrome-side, where we can validate the user. Most likely after validating the credentials the tab will try to comunicate back to the app, using a url that makes no sense on Chrome side (e.g. copito_app://some_token) and thus it would be useful to: 1) Bypass the intent picker when there is only one potential app to handle the url and we can guarante this navigation was originated within ARC. 2) Close the current tab, used only for authentication. This CL addresses 1) by creating and attaching ArcWebContentsData to a tab generated via ChromeShellDelegate so we can uniquely label this kind of navigations. Bug: b/67656508 Test: Build, also added new unit tests for the added functionality. Change-Id: I8e5e7537937b6f1e03f896a9f8893cb67044120f Reviewed-on: https://chromium-review.googlesource.com/890160Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Reviewed-by:
Jorge Lucangeli Obes <jorgelo@chromium.org> Commit-Queue: David Jacobo <djacobo@chromium.org> Cr-Commit-Position: refs/heads/master@{#550407}
-
John Abd-El-Malek authored
This matches the permissions used by the old mime_types API that the new API replaces. Bug: 819761 Change-Id: I7b9cb7150762d54a375bf1fa69cd10d4b715dfb0 Reviewed-on: https://chromium-review.googlesource.com/1007636Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#550406}
-
Becky Zhou authored
+ Disable showing app menu for hardware menu button + Hide app menu on tab modal dialog about to be shown Bug: 812452 Change-Id: I924a03ca0c2a5c13d8425a91b2f235e216efd186 Reviewed-on: https://chromium-review.googlesource.com/1011235Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Becky Zhou <huayinz@chromium.org> Cr-Commit-Position: refs/heads/master@{#550405}
-
Filip Gorski authored
* Tracks all the tabs for non empty paint * Triggers a fetch when the visible tab is 10 seconds past the non empty paint. Bug: 824188 Change-Id: I7a26353eafe5efa39c5406040278159b9c3a3334 Reviewed-on: https://chromium-review.googlesource.com/1008276 Commit-Queue: Filip Gorski <fgorski@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#550404}
-
Kenneth Russell authored
Remove the "Rats!" WebGL infobar and associated content/public API. Blocking of WebGL is now handled entirely within content/. Allow a particular domain to use WebGL again after a lost context via any user-initiated or browser-initiated navigation (reload, explicit navigation via the Omnibox, DevTools / Telemetry), but not reloads initiated from JavaScript. Bug: 575305 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: I9d9f65ba5f8ff6afe3da58d358ad88fb85c999c8 Reviewed-on: https://chromium-review.googlesource.com/522938Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#550403}
-
Steven Holte authored
Follow up to https://chromium-review.googlesource.com/c/chromium/src/+/978942 which added it to the build, but did not include the result in the archive. Change-Id: I0b5d1168397cace23c56206bd59f571e89111bfa Reviewed-on: https://chromium-review.googlesource.com/996469Reviewed-by:
Michael Moss <mmoss@chromium.org> Commit-Queue: Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#550402}
-
Jun Mukai authored
Otherwise linker fails for the undefined reference to the vtable of AssistantSettingsManager (not -Impl). Bug: None Test: build succeeds Change-Id: I5b9bcd7ae67c8fcf8ac51f2b6bf53807ed180f8f Reviewed-on: https://chromium-review.googlesource.com/1011384Reviewed-by:
Muyuan Li <muyuanli@chromium.org> Commit-Queue: Jun Mukai <mukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#550401}
-
John Z Wu authored
1. Get all address autofill profiles. 2. Update an address autofill profile. 3. Delete an address autofill profile. Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: If433ea349119dda0e61fb3b5963a561b234984c8 Reviewed-on: https://chromium-review.googlesource.com/1008403 Commit-Queue: John Wu <jzw@chromium.org> Reviewed-by:
Hiroshi Ichikawa <ichikawa@chromium.org> Cr-Commit-Position: refs/heads/master@{#550400}
-
Kevin Marshall authored
See the bug for more details. TBR=fukino@chromium.org Bug: 832192 Change-Id: I24052278924c70b2fb4de9b55777c393724022d3 Reviewed-on: https://chromium-review.googlesource.com/1011230 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#550399}
-
W. James MacLean authored
The fix for https://crbug.com/784932 seems to have broken the touch selection handles for PDF; sfter scrolling they're in the wrong place. Going back to the original solution, but this time making sure that on scroll we don't update the selected text, thus avoiding the DCHECK that was at the root of 784932. Bug: 829406 Change-Id: I387d120d105c351d80c337ba1646a3cb1eaf40dc Reviewed-on: https://chromium-review.googlesource.com/998073Reviewed-by:
dsinclair <dsinclair@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
-
Demetrios Papadopoulos authored
Bug: None Change-Id: I551037fe39f016e7fad38cea21c63ac58a4859ee Reviewed-on: https://chromium-review.googlesource.com/1010475Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#550397}
-
Mugdha Lakhani authored
And follow up on review comments. Parent review: https://chromium-review.googlesource.com/c/chromium/src/+/968865 Bug: 813564 Change-Id: Ie2033f8f20ccd064d9e57f28958e52c87962b217 Reviewed-on: https://chromium-review.googlesource.com/1002896Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Commit-Queue: Mugdha Lakhani <nator@chromium.org> Cr-Commit-Position: refs/heads/master@{#550396}
-
Ryan Sleevi authored
The CT Log URL was initially included as part of the log metadata, as it was anticipated that clients may fetch STHs directly from the log. However, in further design review, it was realized this would disproportionately add to log load, and unnecessarily. Instead, STHs are distributed to clients via the component updater, and it's not necessary to track the log URL in the client (nor has it ever been used). Remove it and save a few bytes - and delete dead code. BUG=832224 Change-Id: I61f207b2583774b3d65cd586e58d51f8e50db77a Reviewed-on: https://chromium-review.googlesource.com/1010519Reviewed-by:
David Benjamin <davidben@chromium.org> Commit-Queue: Ryan Sleevi <rsleevi@chromium.org> Cr-Commit-Position: refs/heads/master@{#550395}
-
Patrick Monette authored
Bug: 717696 Change-Id: Ieb4a125227180f741afc1f3f51721888688a9d64 Reviewed-on: https://chromium-review.googlesource.com/1011184 Commit-Queue: Patrick Monette <pmonette@chromium.org> Reviewed-by:
Chris Hamilton <chrisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#550394}
-
Matt Mueller authored
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia6395ada4498d61fda0eb258f8196bb93e1dc002 Reviewed-on: https://chromium-review.googlesource.com/1009105Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Matt Mueller <mattm@chromium.org> Cr-Commit-Position: refs/heads/master@{#550393}
-
Kush Sinha authored
Add a metric to track the number of forced logouts for child accounts on an account reconciliation error. Bug: 831635 Change-Id: Ie109c3e3117e379db87e8da3c044530a35939ba9 Reviewed-on: https://chromium-review.googlesource.com/1007136Reviewed-by:
Brian White <bcwhite@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Kush Sinha <sinhak@chromium.org> Cr-Commit-Position: refs/heads/master@{#550392}
-
Patrick Monette authored
This was caused by comparing a normalized subject name to a non- normalized subject. The trailing null character of the non-normalized was causing the comparison to fail for the same subject name. That normalization is now done as part of GetCertificateInfo(), so that it cannot be misused. Bug: 819793 Change-Id: I0f68225ceb513aaef695fd2a673d09598ad8b3c5 Reviewed-on: https://chromium-review.googlesource.com/1010835 Commit-Queue: Patrick Monette <pmonette@chromium.org> Reviewed-by:
-
Kenneth Russell authored
virtual/gpu/fast/canvas/canvas-composite-video.html seen flaky in site_per_process_webkit_layout_tests. Tbr: junov@chromium.org No-Try: True Bug: 800745 Change-Id: Iecb754d9fe2f447c6421b5e7d4e02ecba47b967f Reviewed-on: https://chromium-review.googlesource.com/1011405Reviewed-by:
-
Pedro Amaral authored
Previously we only exposed the view height minus the sum of both top and bottom browser control heights. Now we also expose the top and bottom browser control heights. This allows the tab switcher to correctly compute the top height offset. BUG: 826360 Change-Id: Ia031479621673968dc14fa1649c4f7a2d5fb2736 Reviewed-on: https://chromium-review.googlesource.com/1006365Reviewed-by:
Matthew Jones <mdjones@chromium.org> Reviewed-by:
-
Moe Ahmadi authored
1. Exits 'edit' mode after saving or canceling changes to an address, credit card, or contact info. 2. Changes the label of the edit screen's 'Done' button to 'Save'. TBR=sdefresne@ Bug: 814856 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ie69697299e92d7f8a922104f84c002c37b5a442b Reviewed-on: https://chromium-review.googlesource.com/1007865 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
-
Amirhossein Simjour authored
This is used to support html <select> element when it uses Android AlertDialogs. Bug:819876 Change-Id: Ibe005b8227d623430e79a563521700258cc34d69 Reviewed-on: https://chromium-review.googlesource.com/1010479 Commit-Queue: Amirhossein Simjour <asimjour@chromium.org> Reviewed-by:
Biao She <bshe@chromium.org> Cr-Commit-Position: refs/heads/master@{#550387}
-
Tibor Goldschwendt authored
A result of this change is the removal of offset on incognito NTP if browser controls are hidden. Bug: 821992 Change-Id: I1aa499bf01ec2b259d4d9bcd5b8fee1edd42a5b2 Reviewed-on: https://chromium-review.googlesource.com/1001499Reviewed-by:
-
Jonathan Backer authored
This is copying from GLES2Decoder. The state_ resets were introduced in https://chromiumcodereview.appspot.com/10573004/ as part of https://bugs.chromium.org/p/chromium/issues/detail?id=130617 This bug is WebGL specific. However, RasterDecoder uses GL for texture copying (e.g. gles2_cmd_copy_texture_chromium.cc) and some basic state restoration for virtual contexts. It's possible that this change is not strictly required, but it certainly doesn't hurt and it keeps the GLES2DecoderImpl and RasterDecoderImpl as similar as possible to maximize compatibility when sharing code. Bug: 789238 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: I036b6f864d20aaa6bb8a0d85b2feb74b595c5ece Reviewed-on: https://chromium-review.googlesource.com/1010766 Commit-Queue: Jonathan Backer <backer@chromium.org> Reviewed-by:
-
Kevin Marshall authored
TBR=fukino@chromium.org Bug: 831211 Change-Id: I173d35915b89bcd346d473d41f01db718808574c Reviewed-on: https://chromium-review.googlesource.com/1010829 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
-
Tetsui Ohkubo authored
This CL adds separator between notification center part and system tray part of UnifiedSystemTray. Screenshot: https://screenshot.googleplex.com/i2G4zquv1Gz UX spec: http://shortn/_U7QSE9nhSk TEST=manual BUG=828752 Change-Id: Ia1719faada97c0886b1a8d8526e5a8777f0dd222 Reviewed-on: https://chromium-review.googlesource.com/1006296Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#550383}
-
Sam McNally authored
QuickOffice needs to be able to enable fullscreen to support its present functionality. Implement the fullscreen-related WebContentsDelegate methods in MimeHandlerViewGuest by forwarding to the embedding WebContentsDelegate. Also override EmbedderFullscreenToggled to forward fullscreen exit reporting to the guest. Bug: 819761 Tbr: lazyboy@chromium.org Change-Id: Ia334104375cf8d92a0b45b92cb1b6e80a1e9c407 Reviewed-on: https://chromium-review.googlesource.com/1004394 Commit-Queue: Sam McNally <sammc@chromium.org> Reviewed-by:
Ehsan Karamad <ekaramad@chromium.org> Cr-Commit-Position: refs/heads/master@{#550382}
-
Matt Giuca authored
BrowserView's toolbar() will have null app_menu_button() and browser_actions() in hosted app windows, which could result in null dereferences in certain corner cases. This adds null checks where it's obviously safe to do so. A better fix would be to use toolbar_button_provider() to access the hosted app window's real app menu button and actions container, but that requires more careful consideration. Bug: 826596, 811982 Change-Id: Iea0a0ea3acffe5ed1016e61bd8ae2fe36971c79d Reviewed-on: https://chromium-review.googlesource.com/1010002Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Matt Giuca <mgiuca@chromium.org> Cr-Commit-Position: refs/heads/master@{#550381}
-
Christian Fremerey authored
This reverts commit cfec3da6. Reason for revert: Checking if flakiness is still happening on Mac ASAN bot Original change's description: > Disable WebRtcBrowserTest.NegotiateNonCryptoCall on Mac > > TBR=chfremer@chromium.org > > Bug: 810321 > Change-Id: Ia12ba855ab9fb8e66c9d48c8e66cf91f76f72e08 > Reviewed-on: https://chromium-review.googlesource.com/908461 > Reviewed-by: Olga Sharonova <olka@chromium.org> > Commit-Queue: Olga Sharonova <olka@chromium.org> > Cr-Commit-Position: refs/heads/master@{#535381} TBR=olka@chromium.org,chfremer@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 810321 Change-Id: I6e021bca934428cf9c94a34d4f0e874c25868033 Reviewed-on: https://chromium-review.googlesource.com/1010842 Commit-Queue: Christian Fremerey <chfremer@chromium.org> Reviewed-by:
Christian Fremerey <chfremer@chromium.org> Cr-Commit-Position: refs/heads/master@{#550380}
-
https://pdfium.googlesource.com/pdfium.git/+log/e6d4559a1b6c..9e625db795ca $ git log e6d4559a1..9e625db79 --date=short --no-merges --format='%ad %ae %s' 2018-04-12 thestig Slightly simplify CPDF_Color::SetColorSpace(). 2018-04-12 tsepez Return pdfium::span<char> from ByteString::GetBuffer(). 2018-04-12 tsepez Return pdfium::span<wchar_t> from WideString::GetBuffer(). 2018-04-12 rharrison Tidy up types in CBC_PDF417HighLevelEncoder 2018-04-12 rharrison Fail decode if continuing to get insufficient size error 2018-04-12 thestig Make the input data to CPDF_ColorSpace::GetRGB() const. 2018-04-12 thestig Change CPDF{XFA}_Page::PageToDevice() to take a CFX_PointF. 2018-04-12 thestig Add return value to FPDF_DeviceToPage(). 2018-04-12 thestig Add CPDF_Page::DeviceToPage() / PageToDevice(). 2018-04-12 thestig Change some CPDFXFA_Page methods to take rects and points. 2018-04-12 thestig Change GetDisplayMatrix methods to take FX_RECT. 2018-04-12 thestig Make CFX_Font take FX_RECT out params by pointer. Created with: roll-dep src/third_party/pdfium BUG=chromium:802242 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: Iccdcc7755116bfb7f322ba62beb28cbb11f61bc1 Reviewed-on: https://chromium-review.googlesource.com/1011127 Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#550379}
-
Brandon Jones authored
This makes it easier to hit the 60FPS ideal while encouraging apps to use some antialiasing by default. Should be paired with a future CL that bumps the default buffer resolution slightly to help compensate for the quality loss. Bug: 832185 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I7f089b4cde6c2d55d3965a4927d05a2a74520e2b Reviewed-on: https://chromium-review.googlesource.com/1010774Reviewed-by:
Klaus Weidner <klausw@chromium.org> Reviewed-by:
Ian Vollick <vollick@chromium.org> Commit-Queue: Ian Vollick <vollick@chromium.org> Cr-Commit-Position: refs/heads/master@{#550378}
-
Kevin Marshall authored
TBR=piman@chromium.org Bug: 832191 Change-Id: I8fae4dd27cad4dea281e8198fa878d9a8cab3a94 Reviewed-on: https://chromium-review.googlesource.com/1010981Reviewed-by:
-
Toni Barzic authored
BUG=830945 Change-Id: I1d2481f3b2cdc874fb36a72e951b9602cc37b56e Reviewed-on: https://chromium-review.googlesource.com/1005441Reviewed-by:
Jenny Zhang <jennyz@chromium.org> Commit-Queue: Toni Barzic <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#550376}
-
Kenneth Russell authored
virtual/modern-media-controls/media/controls/modern/ tap-to-hide-controls.html fast/forms/search/search-appearance-basic.html Tbr: steimel@chromium.org, wangxianzhu@chromium.org Bug: 831720, 832274 Change-Id: I301a309a7469d4cd54beaddb2c7f7a4727510ecb Reviewed-on: https://chromium-review.googlesource.com/1011044 Commit-Queue: Kenneth Russell <kbr@chromium.org> Reviewed-by: -
Katie D authored
This bug was not user-facing, but caused an error to be logged to the console. Bug: 832196 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I8be6a1515b8f2344e39d53bb3fe90bb04bce4c54 Reviewed-on: https://chromium-review.googlesource.com/1010704Reviewed-by:
David Tseng <dtseng@chromium.org> Commit-Queue: Katie Dektar <katie@chromium.org> Cr-Commit-Position: refs/heads/master@{#550374}
-
Chris Blume authored
This patch introduces a child allocator for surface IDs. This allows a renderer to create a surface as it needs (such as when creating the Cast dialog), as opposed to request a surface from the parent. BUG=789259 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel Change-Id: I36acec47f083dffd029aba56431eda720579e63e Reviewed-on: https://chromium-review.googlesource.com/861643 Commit-Queue: Fady Samuel <fsamuel@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Cr-Commit-Position: refs/heads/master@{#550373}
-
Anita Woodruff authored
- Now when the '--enable-features=NotificationsWithMojo' flag is set, ServiceWorkerRegistration.getNotifications() will be routed through the notification mojo service, instead of via legacy IPC. R=peter@chromium.org Bug: 796991 Change-Id: Ie3d24ae753238da90c316187a4f2b84b4b52d531 Reviewed-on: https://chromium-review.googlesource.com/976222 Commit-Queue: Anita Woodruff <awdf@chromium.org> Reviewed-by:
Oliver Chang <ochang@chromium.org> Reviewed-by:
-
Zach Helfinstein authored
Replacing placeholder sounds taken from ChromeVox. Bug: 829004 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I3aec21ab109b6afafab677b97d11de0df4fc5bfb Reviewed-on: https://chromium-review.googlesource.com/999031 Commit-Queue: Zach Helfinstein <zhelfins@chromium.org> Reviewed-by:
-
Tibor Goldschwendt authored
Change-Id: I55b37b0f0578a22bb1812768658af31f6235da8d Reviewed-on: https://chromium-review.googlesource.com/1007969 Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org> Reviewed-by:
Becky Zhou <huayinz@chromium.org> Cr-Commit-Position: refs/heads/master@{#550370}
-
Yue Li authored
- Add AssistantSettingsManager mojom API. - The AssistantSettingsManager will call AssistantManagerService to communicates with internal assistant manager that talks to the server. - The AssistantSettingsManager will use internal utils to construct and unwrap internal proto strings. Bug: 819782 Change-Id: I420d8e70543d665587bd2dfdc803afae9d21a310 Reviewed-on: https://chromium-review.googlesource.com/953654 Commit-Queue: Yue Li <updowndota@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#550369}
-
