LineageOS/android_development
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix the MergeRootFrameLayout rule by checking the presence of padding.

Browse Source 
Change-Id: I3a8b257025e74f6899233d9b99f7f86b5675e997
This commit is contained in:
Romain Guy
2009-10-07 12:31:17 -07:00
parent 7ce7a7835d
commit 9bda7101b8
2 changed files with 22 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
tools/layoutopt/libs/uix/src/com/android/layoutopt/uix/groovy/LayoutAnalysisCategory.java
View File

@@ -35,6 +35,11 @@ import org.w3c.dom.Element;
* to {@link com.android.layoutopt.uix.LayoutAnalysis} and {@link org.w3c.dom.Node}.
*/
public class LayoutAnalysisCategory {
private static final String ANDROID_PADDING = "android:padding";
private static final String ANDROID_PADDING_LEFT = "android:paddingLeft";
private static final String ANDROID_PADDING_TOP = "android:paddingTop";
private static final String ANDROID_PADDING_RIGHT = "android:paddingRight";
private static final String ANDROID_PADDING_BOTTOM = "android:paddingBottom";
private static final String ANDROID_LAYOUT_WIDTH = "android:layout_width";
private static final String ANDROID_LAYOUT_HEIGHT = "android:layout_height";
private static final String VALUE_FILL_PARENT = "fill_parent";
@@ -99,6 +104,18 @@ public class LayoutAnalysisCategory {
return data == null ? -1 : (Integer) data;
}
/**
* xmlNode.hasPadding()
*
* @return True if the node has one ore more padding attributes.
*/
public static boolean hasPadding(Element element) {
return element.getAttribute(ANDROID_PADDING).length() > 0 ||
element.getAttribute(ANDROID_PADDING_LEFT).length() > 0 ||
element.getAttribute(ANDROID_PADDING_TOP).length() > 0 ||
element.getAttribute(ANDROID_PADDING_BOTTOM).length() > 0 ||
element.getAttribute(ANDROID_PADDING_RIGHT).length() > 0;
}
/**
* Returns whether this node's width is fill_parent.

4
tools/layoutopt/libs/uix/src/resources/rules/MergeRootFrameLayout.rule
View File

@@ -8,9 +8,11 @@
// - The node is a FrameLayout
// - The node is fill_parent in both orientation *or* it has no layout_gravity
// - The node does not have a background nor a foreground
// - The node does not have padding
if (node.isRoot() && node.is("FrameLayout") && !node.'@android:background' &&
!node.'@android:foreground' && ((node.isWidthFillParent() &&
node.isHeightFillParent()) || !node.'@android:layout_gravity')) {
node.isHeightFillParent()) || !node.'@android:layout_gravity') &&
!node.hasPadding()) {
analysis << "The root-level <FrameLayout/> can be replaced with <merge/>"
}