webnn: Move LiteRT CPU inference from GPU to renderer process [chromium/src : main]

1 view
Skip to first unread message

Fu, Junwei (Gerrit)

unread,
May 25, 2026, 10:13:35 PM (5 hours ago) May 25
to Hu, Ningxin, Reilly Grant, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, Jiewei Qian, blink-...@chromium.org
Attention needed from Hu, Ningxin and Reilly Grant

Fu, Junwei voted and added 2 comments

Votes added by Fu, Junwei

Commit-Queue+1

2 comments

Patchset-level comments
File-level comment, Patchset 9 (Latest):
Fu, Junwei . resolved

Thanks for review, done the comments.

File services/webnn/tflite/context_provider_tflite.cc
Line 83, Patchset 3:#if BUILDFLAG(WEBNN_USE_LITERT)
Reilly Grant . resolved

Can you put this logic in `services/webnn/public/cpp/in_process_context_provider.cc` so we don't have a cross-dependency between the TFLite and LiteRT backends.

Fu, Junwei

The function call `CreateWebNNContext` is a mojo interface, do you mean add a argument in constructor `ContextProviderTflite::ContextProviderTflite` from `in_process_context_provider.cc` that will be called back with the argument? How about renaming `ContextProviderTflite` to `ContextProviderInRenderer` for both TFLite and LiteRT backend?

Reilly Grant

Yes, rename `ContextProviderTflite` to `ContextProviderInRenderer` and have it cover both cases. As we experiment with using ORT or CoreML in the renderer as well it will expand beyond just TFLite and LiteRT.

Fu, Junwei

Done

Open in Gerrit

Related details

Attention is currently required from:
  • Hu, Ningxin
  • Reilly Grant
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I9df466762d16f5787c2012bd83f4ff4fd3e72038
Gerrit-Change-Number: 7825509
Gerrit-PatchSet: 9
Gerrit-Owner: Fu, Junwei <junw...@intel.com>
Gerrit-Reviewer: Fu, Junwei <junw...@intel.com>
Gerrit-Reviewer: Hu, Ningxin <ningx...@intel.com>
Gerrit-Reviewer: Reilly Grant <rei...@chromium.org>
Gerrit-CC: Jiewei Qian <q...@chromium.org>
Gerrit-Attention: Reilly Grant <rei...@chromium.org>
Gerrit-Attention: Hu, Ningxin <ningx...@intel.com>
Gerrit-Comment-Date: Tue, 26 May 2026 02:13:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Fu, Junwei <junw...@intel.com>
Comment-In-Reply-To: Reilly Grant <rei...@chromium.org>
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages