Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(222)

Issue 2700193002: Add mojo service for CopylessPaste in Java (Closed)

Created:
3 years, 10 months ago by wychen
Modified:
3 years, 8 months ago
Reviewers:
Ted C, dglazkov, dcheng
CC:
esprehn, Aaron Boodman, abarth-chromium, agrieve+watch_chromium.org, blink-reviews, blink-reviews-api_chromium.org, chromium-reviews, darin (slow to review), dglazkov+blink, dproctor, qsr+mojo_chromium.org, viettrungluu+watch_chromium.org, yzshen+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add mojo service for CopylessPaste in Java A CopylessPaste mojo service in Java is added. The received data would be forwarded to the app indexing service. BUG=693650

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+69 lines, -0 lines) Patch
A chrome/android/java/src/org/chromium/chrome/browser/copyless_paste/CopylessPasteImpl.java View 1 chunk +43 lines, -0 lines 0 comments Download
M chrome/android/java/src/org/chromium/chrome/browser/mojo/ChromeInterfaceRegistrar.java View 2 chunks +3 lines, -0 lines 0 comments Download
M chrome/android/java_sources.gni View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/DEPS View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/chrome_content_browser_client.cc View 2 chunks +8 lines, -0 lines 0 comments Download
M chrome/browser/chrome_content_browser_manifest_overlay.json View 1 chunk +1 line, -0 lines 0 comments Download
M third_party/WebKit/public/BUILD.gn View 1 chunk +1 line, -0 lines 0 comments Download
A third_party/WebKit/public/platform/modules/document_metadata/OWNERS View 1 chunk +2 lines, -0 lines 0 comments Download
A third_party/WebKit/public/platform/modules/document_metadata/copyless_paste.mojom View 1 chunk +9 lines, -0 lines 0 comments Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 15 (9 generated)
wychen
PTAL
3 years, 10 months ago (2017-02-17 21:42:38 UTC) #4
Ted C
lgtm
3 years, 10 months ago (2017-02-17 22:26:35 UTC) #5
wychen
dcheng@, esprehn@, how does it look on the blink side?
3 years, 10 months ago (2017-02-21 17:05:34 UTC) #6
dcheng
Presumably there's some code in Blink that's going to need to call into the mojo ...
3 years, 10 months ago (2017-02-22 18:38:13 UTC) #11
wychen
On 2017/02/22 18:38:13, dcheng wrote: > Presumably there's some code in Blink that's going to ...
3 years, 10 months ago (2017-02-22 18:45:36 UTC) #12
wychen
3 years, 8 months ago (2017-04-20 19:06:20 UTC) #15
We ended up putting the Mojo server inside Blink. Closing this CL.

Powered by Google App Engine
This is Rietveld 408576698