Implement issue #1780928: Need support hiding nav keys.

This implements support for devices whose hardware can hide
their navigation keys.  It works much like the existing keyboardHidden
configuration, and for compatibility uses the same configuration
change bit.

Also add FLAG_TURN_ON_SCREEN for windows, which has the system
cause the screen to be turned on when the window is displayed.
Great fun when used with FLAG_SHOW_WHEN_LOCKED!

Change-Id: I0b867f19af85cfd8786a14cea194b34f7bdd9b7a

diff --git a/AaptAssets.cpp b/AaptAssets.cpp
index b00d8b0b8296ccccb013197c1c21866f1a8b31d5..b6b0e63f19aa67570482bee5c011f469f02c519e 100644 (file)
--- a/AaptAssets.cpp
+++ b/AaptAssets.cpp
@@ -187,6 +187,13 @@ AaptGroupEntry::parseNamePart(const String8& part, int* axis, uint32_t* value)
         return 0;
     }
 
         return 0;
     }
 

+    // navigation hidden
+    if (getNavHiddenName(part.string(), &config)) {
+        *axis = AXIS_NAVHIDDEN;
+        *value = config.inputFlags;
+        return 0;
+    }
+

     // navigation
     if (getNavigationName(part.string(), &config)) {
         *axis = AXIS_NAVIGATION;
     // navigation
     if (getNavigationName(part.string(), &config)) {
         *axis = AXIS_NAVIGATION;


@@ -217,7 +224,7 @@ AaptGroupEntry::initFromDirName(const char* dir, String8* resType)
     Vector<String8> parts;
 
     String8 mcc, mnc, loc, layoutsize, layoutlong, orient, den;
     Vector<String8> parts;
 
     String8 mcc, mnc, loc, layoutsize, layoutlong, orient, den;

-    String8 touch, key, keysHidden, nav, size, vers;
+    String8 touch, key, keysHidden, nav, navHidden, size, vers;

 
     const char *p = dir;
     const char *q;
 
     const char *p = dir;
     const char *q;


@@ -393,6 +400,19 @@ AaptGroupEntry::initFromDirName(const char* dir, String8* resType)
         //printf("not keyboard: %s\n", part.string());
     }
 
         //printf("not keyboard: %s\n", part.string());
     }
 

+    // navigation hidden
+    if (getNavHiddenName(part.string())) {
+        navHidden = part;
+
+        index++;
+        if (index == N) {
+            goto success;
+        }
+        part = parts[index];
+    } else {
+        //printf("not navHidden: %s\n", part.string());
+    }
+

     if (getNavigationName(part.string())) {
         nav = part;
 
     if (getNavigationName(part.string())) {
         nav = part;
 


@@ -443,6 +463,7 @@ success:
     this->touchscreen = touch;
     this->keysHidden = keysHidden;
     this->keyboard = key;
     this->touchscreen = touch;
     this->keysHidden = keysHidden;
     this->keyboard = key;

+    this->navHidden = navHidden;

     this->navigation = nav;
     this->screenSize = size;
     this->version = vers;
     this->navigation = nav;
     this->screenSize = size;
     this->version = vers;


@@ -476,6 +497,8 @@ AaptGroupEntry::toString() const
     s += ",";
     s += keyboard;
     s += ",";
     s += ",";
     s += keyboard;
     s += ",";

+    s += navHidden;
+    s += ",";

     s += navigation;
     s += ",";
     s += screenSize;
     s += navigation;
     s += ",";
     s += screenSize;


@@ -528,6 +551,10 @@ AaptGroupEntry::toDirName(const String8& resType) const
         s += "-";
         s += keyboard;
     }
         s += "-";
         s += keyboard;
     }

+    if (this->navHidden != "") {
+        s += "-";
+        s += navHidden;
+    }

     if (this->navigation != "") {
         s += "-";
         s += navigation;
     if (this->navigation != "") {
         s += "-";
         s += navigation;


@@ -852,6 +879,30 @@ bool AaptGroupEntry::getKeyboardName(const char* name,
     return false;
 }
 
     return false;
 }
 

+bool AaptGroupEntry::getNavHiddenName(const char* name,
+                                       ResTable_config* out)
+{
+    uint8_t mask = 0;
+    uint8_t value = 0;
+    if (strcmp(name, kWildcardName) == 0) {
+        mask = out->MASK_NAVHIDDEN;
+        value = out->NAVHIDDEN_ANY;
+    } else if (strcmp(name, "navexposed") == 0) {
+        mask = out->MASK_NAVHIDDEN;
+        value = out->NAVHIDDEN_NO;
+    } else if (strcmp(name, "navhidden") == 0) {
+        mask = out->MASK_NAVHIDDEN;
+        value = out->NAVHIDDEN_YES;
+    }
+
+    if (mask != 0) {
+        if (out) out->inputFlags = (out->inputFlags&~mask) | value;
+        return true;
+    }
+
+    return false;
+}
+

 bool AaptGroupEntry::getNavigationName(const char* name,
                                      ResTable_config* out)
 {
 bool AaptGroupEntry::getNavigationName(const char* name,
                                      ResTable_config* out)
 {


@@ -953,6 +1004,7 @@ int AaptGroupEntry::compare(const AaptGroupEntry& o) const
     if (v == 0) v = touchscreen.compare(o.touchscreen);
     if (v == 0) v = keysHidden.compare(o.keysHidden);
     if (v == 0) v = keyboard.compare(o.keyboard);
     if (v == 0) v = touchscreen.compare(o.touchscreen);
     if (v == 0) v = keysHidden.compare(o.keysHidden);
     if (v == 0) v = keyboard.compare(o.keyboard);

+    if (v == 0) v = navHidden.compare(o.navHidden);

     if (v == 0) v = navigation.compare(o.navigation);
     if (v == 0) v = screenSize.compare(o.screenSize);
     if (v == 0) v = version.compare(o.version);
     if (v == 0) v = navigation.compare(o.navigation);
     if (v == 0) v = screenSize.compare(o.screenSize);
     if (v == 0) v = version.compare(o.version);


@@ -973,6 +1025,7 @@ ResTable_config AaptGroupEntry::toParams() const
     getTouchscreenName(touchscreen.string(), &params);
     getKeysHiddenName(keysHidden.string(), &params);
     getKeyboardName(keyboard.string(), &params);
     getTouchscreenName(touchscreen.string(), &params);
     getKeysHiddenName(keysHidden.string(), &params);
     getKeyboardName(keyboard.string(), &params);

+    getNavHiddenName(navHidden.string(), &params);

     getNavigationName(navigation.string(), &params);
     getScreenSizeName(screenSize.string(), &params);
     getVersionName(version.string(), &params);
     getNavigationName(navigation.string(), &params);
     getScreenSizeName(screenSize.string(), &params);
     getVersionName(version.string(), &params);

diff --git a/AaptAssets.h b/AaptAssets.h
index 865efd1eff8b6228ecb26eed7c2117dfef17a0eb..26500a34886d4ce89f988b284c605bda44068511 100644 (file)
--- a/AaptAssets.h
+++ b/AaptAssets.h
@@ -37,6 +37,7 @@ enum {
     AXIS_TOUCHSCREEN,
     AXIS_KEYSHIDDEN,
     AXIS_KEYBOARD,
     AXIS_TOUCHSCREEN,
     AXIS_KEYSHIDDEN,
     AXIS_KEYBOARD,

+    AXIS_NAVHIDDEN,

     AXIS_NAVIGATION,
     AXIS_SCREENSIZE,
     AXIS_VERSION
     AXIS_NAVIGATION,
     AXIS_SCREENSIZE,
     AXIS_VERSION


@@ -64,6 +65,7 @@ public:
     String8 touchscreen;
     String8 keysHidden;
     String8 keyboard;
     String8 touchscreen;
     String8 keysHidden;
     String8 keyboard;

+    String8 navHidden;

     String8 navigation;
     String8 screenSize;
     String8 version;
     String8 navigation;
     String8 screenSize;
     String8 version;


@@ -83,6 +85,7 @@ public:
     static bool getKeysHiddenName(const char* name, ResTable_config* out = NULL);
     static bool getKeyboardName(const char* name, ResTable_config* out = NULL);
     static bool getNavigationName(const char* name, ResTable_config* out = NULL);
     static bool getKeysHiddenName(const char* name, ResTable_config* out = NULL);
     static bool getKeyboardName(const char* name, ResTable_config* out = NULL);
     static bool getNavigationName(const char* name, ResTable_config* out = NULL);

+    static bool getNavHiddenName(const char* name, ResTable_config* out = NULL);

     static bool getScreenSizeName(const char* name, ResTable_config* out = NULL);
     static bool getVersionName(const char* name, ResTable_config* out = NULL);
 
     static bool getScreenSizeName(const char* name, ResTable_config* out = NULL);
     static bool getVersionName(const char* name, ResTable_config* out = NULL);