Move references to fixed bug pdfium:507 to appropriate DEPS file.
The layering is as it should be, with only xfa/*.{cpp,h} including
fwl/, and fwl/ not including others. Add one DEPS file to enforce
fwl/ not including fxfa/, and one DEPS file to take away fwl/ from
another fxfa/ sub-directory.
Bug: pdfium:507
Change-Id: I2e87b0b8765c49f60d62f51ac7b660595aaaa98b
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/90332
Auto-Submit: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
diff --git a/xfa/DEPS b/xfa/DEPS
index 02cbbab..a279684 100644
--- a/xfa/DEPS
+++ b/xfa/DEPS
@@ -3,7 +3,5 @@
'+fxbarcode',
'+fxjs/gc',
'+v8/include/cppgc',
-
- # xfa/fwl should be standalone. https://crbug.com/pdfium/507
'-xfa/fwl',
]
diff --git a/xfa/fwl/DEPS b/xfa/fwl/DEPS
new file mode 100644
index 0000000..8bea3a3
--- /dev/null
+++ b/xfa/fwl/DEPS
@@ -0,0 +1,4 @@
+include_rules = [
+ # fwl is standalone vis-a-vis fxfa. https://crbug.com/pdfium/507
+ '-xfa/fxfa',
+]
diff --git a/xfa/fxfa/DEPS b/xfa/fxfa/DEPS
index 2efe354..2498160 100644
--- a/xfa/fxfa/DEPS
+++ b/xfa/fxfa/DEPS
@@ -1,7 +1,5 @@
include_rules = [
'+fxjs',
'+v8/include/cppgc',
-
- # xfa/fwl should be standalone. https://crbug.com/pdfium/507
'+xfa/fwl',
]
diff --git a/xfa/fxfa/fm2js/DEPS b/xfa/fxfa/fm2js/DEPS
index e19a721..df6a09f 100644
--- a/xfa/fxfa/fm2js/DEPS
+++ b/xfa/fxfa/fm2js/DEPS
@@ -1,6 +1,4 @@
include_rules = [
'+third_party/icu',
-
- # xfa/fwl should be standalone. https://crbug.com/pdfium/507
'-xfa/fwl',
]
diff --git a/xfa/fxfa/layout/DEPS b/xfa/fxfa/layout/DEPS
new file mode 100644
index 0000000..4931b25
--- /dev/null
+++ b/xfa/fxfa/layout/DEPS
@@ -0,0 +1,4 @@
+include_rules = [
+ '-xfa/fwl',
+]
+
diff --git a/xfa/fxfa/parser/DEPS b/xfa/fxfa/parser/DEPS
index 22337ef..4931b25 100644
--- a/xfa/fxfa/parser/DEPS
+++ b/xfa/fxfa/parser/DEPS
@@ -1,5 +1,4 @@
include_rules = [
- # xfa/fwl should be standalone. https://crbug.com/pdfium/507
'-xfa/fwl',
]
