|
Revert of RELAND: ShapeDetection: use mojom::Bitmap for mojo interface. (patchset #2 id:20001 of https://codereview.chromium.org/2753413002/ )
Reason for revert:
Broke shapedetection/detection-on-worker.html
shapedetection/detection-security-test.html reliably:
18:14:35.412 4150 [10306:39683:0319/181429.446067:1229254778376:WARNING:url_request_job_manager.cc(90)] Failed to map: layout-test-mojom://content/shell/renderer/layout_test/frame_interface_registry
18:14:35.413 4150 [10306:39683:0319/181429.446194:1229254899173:WARNING:url_request_job_manager.cc(90)] Failed to map: layout-test-mojom://content/shell/renderer/layout_test/interface_registry
18:14:35.413 4150 [10308:771:0319/181429.450430:1229259136455:ERROR:mojo_context_state.cc(199)] Failed to fetch source for module "content/shell/renderer/layout_test/frame_interface_registry"
18:14:35.413 4150 [10308:771:0319/181429.450505:1229259209121:ERROR:mojo_context_state.cc(199)] Failed to fetch source for module "content/shell/renderer/layout_test/interface_registry"
18:14:35.413 4150 [10308:771:0319/181429.452582:1229261287723:ERROR:mojo_context_state.cc(199)] Failed to fetch source for module "skia/public/interfaces/bitmap.mojom"
18:14:35.414 4111 [46631/51882] shapedetection/detection-security-test.html failed unexpectedly (test timed out)
Original issue's description:
> RELAND: ShapeDetection: use mojom::Bitmap for mojo interface.
>
> Original CL got reverted due to shapedetection/detection-on-worker.html
> fail to fetch source for module skia/public/interfaces/bitmap.mojom.
>
> This CL includes the generated JS bindings for skia/public/interfaces/bitmap.mojom
> in the layout tests archive.
>
> Original CL description ------------------------------------------------
> ShapeDetection: use mojom::Bitmap for mojo interface.
>
> This CL uses mojo::Bitmap for mojo ShapeDetection interfaces definition,
> so that the Detect API is completely flexible and friendly.
>
> BUG= 665488
> TEST(Layout)=
> third_party/WebKit/LayoutTests/fast/shapedetection/shapedetection-creation.html
> third_party/WebKit/LayoutTests/fast/shapedetection/shapedetection-empty-input.html
> third_party/WebKit/LayoutTests/shapedetection/detection-HTMLCanvasElement.html
> third_party/WebKit/LayoutTests/shapedetection/detection-HTMLImageElement.html
> third_party/WebKit/LayoutTests/shapedetection/detection-HTMLVideoElement.html
> third_party/WebKit/LayoutTests/shapedetection/detection-ImageBitmap.html
> third_party/WebKit/LayoutTests/shapedetection/detection-ImageData.html
>
> Review-Url: https://codereview.chromium.org/2629433003
> Cr-Commit-Position: refs/heads/master@{#446142}
> (cherry picked from commit 33adf4c85dd553eb607bcf9c02c5fa60c8738a68)
>
> Conflicts:
> third_party/WebKit/Source/modules/shapedetection/FaceDetector.cpp
> third_party/WebKit/Source/modules/shapedetection/TextDetector.cpp
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.mac:mac_optional_gpu_tests_rel
>
> TBR=rockot@chromium.org, dtrainor@chromium.org, haraken@chromium.org, bsalomon@google.com, tsepez@chromium.org
> since the gist of the CL hasn't changed.
>
> Review-Url: https://codereview.chromium.org/2681913003
> Cr-Original-Commit-Position: refs/heads/master@{#450582}
> Review-Url: https://codereview.chromium.org/2753413002
> Cr-Commit-Position: refs/heads/master@{#458010}
> Committed: https://chromium.googlesource.com/chromium/src/+/b1caeee1ee08c8b3bcaa423c8aa4a9f06e9ce40a
TBR=bsalomon@google.com,dtrainor@chromium.org,haraken@chromium.org,rockot@chromium.org,tsepez@chromium.org,mcasas@chromium.org,dpranke@chromium.org,junwei.fu@intel.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 665488
Review-Url: https://codereview.chromium.org/2755393002
Cr-Commit-Position: refs/heads/master@{#458033}
Committed: https://chromium.googlesource.com/chromium/src/+/0919785cfd8b70ffe8814e146fbc500b3e284b2b
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+256 lines, -168 lines) |
Patch |
|
M |
AUTHORS
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
BUILD.gn
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
chrome/android/BUILD.gn
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
chrome/android/java/src/org/chromium/chrome/browser/shapedetection/BarcodeDetectionImpl.java
|
View
|
|
4 chunks |
+6 lines, -16 lines |
0 comments
|
Download
|
|
M |
chrome/android/java/src/org/chromium/chrome/browser/shapedetection/TextDetectionImpl.java
|
View
|
|
4 chunks |
+6 lines, -16 lines |
0 comments
|
Download
|
|
M |
content/public/android/BUILD.gn
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
content/public/android/java/src/org/chromium/content/browser/shapedetection/FaceDetectionImpl.java
|
View
|
|
2 chunks |
+11 lines, -23 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/DEPS
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
services/shape_detection/barcode_detection_impl_mac.h
|
View
|
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/barcode_detection_impl_mac.mm
|
View
|
|
1 chunk |
+5 lines, -3 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/barcode_detection_impl_mac_unittest.mm
|
View
|
|
3 chunks |
+19 lines, -5 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/detection_utils_mac.h
|
View
|
|
1 chunk |
+4 lines, -3 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/detection_utils_mac.mm
|
View
|
|
1 chunk |
+42 lines, -12 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/face_detection_impl_mac.h
|
View
|
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/face_detection_impl_mac.mm
|
View
|
|
1 chunk |
+5 lines, -3 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/face_detection_impl_mac_unittest.mm
|
View
|
|
1 chunk |
+14 lines, -2 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/public/interfaces/BUILD.gn
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
services/shape_detection/public/interfaces/barcodedetection.mojom
|
View
|
|
2 chunks |
+4 lines, -3 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/public/interfaces/facedetection.mojom
|
View
|
|
2 chunks |
+4 lines, -3 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/public/interfaces/textdetection.mojom
|
View
|
|
2 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/text_detection_impl_mac.h
|
View
|
|
1 chunk |
+3 lines, -1 line |
0 comments
|
Download
|
|
M |
services/shape_detection/text_detection_impl_mac.mm
|
View
|
|
1 chunk |
+5 lines, -3 lines |
0 comments
|
Download
|
|
M |
services/shape_detection/text_detection_impl_mac_unittest.mm
|
View
|
|
2 chunks |
+18 lines, -5 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/LayoutTests/shapedetection/resources/mock-barcodedetection.js
|
View
|
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/LayoutTests/shapedetection/resources/mock-facedetection.js
|
View
|
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/LayoutTests/shapedetection/resources/mock-textdetection.js
|
View
|
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/BUILD.gn
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/BarcodeDetector.h
|
View
|
|
1 chunk |
+3 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/BarcodeDetector.cpp
|
View
|
|
2 chunks |
+9 lines, -5 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/DEPS
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/FaceDetector.h
|
View
|
|
1 chunk |
+3 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/FaceDetector.cpp
|
View
|
|
2 chunks |
+6 lines, -3 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/ShapeDetector.h
|
View
|
|
2 chunks |
+3 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/ShapeDetector.cpp
|
View
|
|
5 chunks |
+59 lines, -33 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/TextDetector.h
|
View
|
|
1 chunk |
+3 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/shapedetection/TextDetector.cpp
|
View
|
|
2 chunks |
+6 lines, -3 lines |
0 comments
|
Download
|
Total messages: 6 (3 generated)
|