Re: [oe] [meta-browser][PATCH] chromium: Add missing impl side painting patch

2014-09-29 Thread Otavio Salvador 
On Thu, Sep 25, 2014 at 6:32 AM, Carlos Rafael Giani
 wrote:
> Signed-off-by: Carlos Rafael Giani 

I pushed it but I amended the commit log to:

commit e988a2c5010d74ce3557c37ea045ce710d70c7d7 (HEAD, master)
Author: Carlos Rafael Giani 
Date:   Thu Sep 25 11:32:29 2014 +0200

chromium: Add missing impl side painting patch

Disable rasterization whitelist, unlocking impl side painting. It is
still in an experimental stage, but promises better performance so it
needs to be enabled by PACKAGECONFIG.

Signed-off-by: Carlos Rafael Giani 
Signed-off-by: Otavio Salvador 


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-browser][PATCH] chromium: Add missing impl side painting patch

2014-09-25 Thread Gary Thomas 

On 2014-09-25 05:17, Carlos Rafael Giani wrote:

On 09/25/2014 12:52 PM, Gary Thomas wrote:

On 2014-09-25 03:32, Carlos Rafael Giani wrote:

Signed-off-by: Carlos Rafael Giani 
---
  ...terization-whitelist-unlocking-impl-side-.patch | 73 ++
  recipes-browser/chromium/chromium_37.0.2062.94.bb  |  2 +-
  2 files changed, 74 insertions(+), 1 deletion(-)
  create mode 100644 
recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch

diff --git 
a/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
b/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
new file mode 100644
index 000..3e9d269
--- /dev/null
+++ 
b/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
@@ -0,0 +1,73 @@
+From 96a5a12ecd5b144bfeaa1d2116c2d1996bbb8f6c Mon Sep 17 00:00:00 2001
+From: Carlos Rafael Giani 
+Date: Mon, 22 Sep 2014 23:15:42 +0200
+Subject: [PATCH] Disable rasterization whitelist, unlocking impl side painting
+
+Signed-off-by: Carlos Rafael Giani 
+---
+ gpu/config/software_rendering_list_json.cc | 49 --
+ 1 file changed, 49 deletions(-)
+
+diff --git a/gpu/config/software_rendering_list_json.cc 
b/gpu/config/software_rendering_list_json.cc
+index 52614eb..bd0bf01 100644
+--- a/gpu/config/software_rendering_list_json.cc
 b/gpu/config/software_rendering_list_json.cc
+@@ -1144,55 +1144,6 @@ LONG_STRING_CONST(
+   "features": [
+ "all"
+   ]
+-},
+-{
+-  "id": 96,
+-  "description": "GPU rasterization is whitelisted on N4, N5, N7 and Moto 
X",
+-  "cr_bugs": [362779],
+-  "exceptions": [
+-{
+-  "os": {
+-"type": "android"
+-  },
+-  "machine_model_name": ["Nexus 4", "Nexus 5", "Nexus 7",
+- "XT1049", "XT1050", "XT1052", "XT1053",
+- "XT1055", "XT1056", "XT1058", "XT1060"]
+-},
+-{
+-  "os": {
+-"type": "android",
+-"version": {
+-  "op": ">=",
+-  "value": "4.4.99"
+-}
+-  }
+-}
+-  ],
+-  "features": [
+-"gpu_rasterization"
+-  ]
+-},
+-{
+-  "id": 97,
+-  "description": "Additional GPU rasterization whitelist for field trial",
+-  "cr_bugs": [380694],
+-  "exceptions": [
+-{
+-  "os": {
+-"type": "android"
+-  },
+-  "machine_model_name": ["HTC One",
+- "C5303", "C6603", "C6903",
+- "GT-I8262", "GT-I8552", "GT-I9195", 
"GT-I9300",
+- "GT-I9500", "GT-I9505", "GT-N7100",
+- "SAMSUNG-SCH-I337", "SCH-I545", "SGH-M919",
+- "SM-N900", "SM-N9005", "SPH-L720",
+- "XT907", "XT1032", "XT1033", "XT1080"]
+-}
+-  ],
+-  "features": [
+-"gpu_rasterization_field_trial"
+-  ]
+ }
+   ]
+ }
+--
+1.9.1
+
diff --git a/recipes-browser/chromium/chromium_37.0.2062.94.bb 
b/recipes-browser/chromium/chromium_37.0.2062.94.bb
index 510b644..3d4524b 100644
--- a/recipes-browser/chromium/chromium_37.0.2062.94.bb
+++ b/recipes-browser/chromium/chromium_37.0.2062.94.bb
@@ -8,7 +8,7 @@ SRC_URI = "\
  file://oe-defaults.gypi \
  ${@bb.utils.contains('PACKAGECONFIG', 'component-build', 
'file://component-build.gypi', '', d)} \
  ${@bb.utils.contains('PACKAGECONFIG', 'ignore-lost-context', 
'file://remove-linux-accel-canvas-from-blacklist.patch', '', d)} \
-${@bb.utils.contains('PACKAGECONFIG', 'impl-side-painting', 
'file://disable-rasterization-whitelist-unlocking-impl-side-painting.patch', 
'', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'impl-side-painting', 
'file://0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch', '', 
d)} \
  ${@bb.utils.contains('PACKAGECONFIG', 'disable-api-keys-info-bar', 
'file://0002-Disable-API-keys-info-bar.patch', '', d)} \
  file://unistd-2.patch \
  file://google-chrome \



Why is this important since it appears to only affect Android
based devices (and this is for OE builds)?



By default, it is only enabled for certain Android devices. With this patch, it 
is enabled for all devices, including for OE builds. impl side painting is 
still in an experimental
stage, but promises better performance, which is why I made it an opt-in 
PACKAGECONFIG.


That makes more sense, thanks.

--

Gary Thomas |  Consulting for the
MLB Associates  |Embedded world

--
___
Openembedded-de

Re: [oe] [meta-browser][PATCH] chromium: Add missing impl side painting patch

2014-09-25 Thread Carlos Rafael Giani 

On 09/25/2014 12:52 PM, Gary Thomas wrote:

On 2014-09-25 03:32, Carlos Rafael Giani wrote:

Signed-off-by: Carlos Rafael Giani 
---
  ...terization-whitelist-unlocking-impl-side-.patch | 73 
++

  recipes-browser/chromium/chromium_37.0.2062.94.bb  |  2 +-
  2 files changed, 74 insertions(+), 1 deletion(-)
  create mode 100644 
recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch


diff --git 
a/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch 
b/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch 


new file mode 100644
index 000..3e9d269
--- /dev/null
+++ 
b/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch

@@ -0,0 +1,73 @@
+From 96a5a12ecd5b144bfeaa1d2116c2d1996bbb8f6c Mon Sep 17 00:00:00 2001
+From: Carlos Rafael Giani 
+Date: Mon, 22 Sep 2014 23:15:42 +0200
+Subject: [PATCH] Disable rasterization whitelist, unlocking impl 
side painting

+
+Signed-off-by: Carlos Rafael Giani 
+---
+ gpu/config/software_rendering_list_json.cc | 49 
--

+ 1 file changed, 49 deletions(-)
+
+diff --git a/gpu/config/software_rendering_list_json.cc 
b/gpu/config/software_rendering_list_json.cc

+index 52614eb..bd0bf01 100644
+--- a/gpu/config/software_rendering_list_json.cc
 b/gpu/config/software_rendering_list_json.cc
+@@ -1144,55 +1144,6 @@ LONG_STRING_CONST(
+   "features": [
+ "all"
+   ]
+-},
+-{
+-  "id": 96,
+-  "description": "GPU rasterization is whitelisted on N4, N5, 
N7 and Moto X",

+-  "cr_bugs": [362779],
+-  "exceptions": [
+-{
+-  "os": {
+-"type": "android"
+-  },
+-  "machine_model_name": ["Nexus 4", "Nexus 5", "Nexus 7",
+- "XT1049", "XT1050", "XT1052", 
"XT1053",
+- "XT1055", "XT1056", "XT1058", 
"XT1060"]

+-},
+-{
+-  "os": {
+-"type": "android",
+-"version": {
+-  "op": ">=",
+-  "value": "4.4.99"
+-}
+-  }
+-}
+-  ],
+-  "features": [
+-"gpu_rasterization"
+-  ]
+-},
+-{
+-  "id": 97,
+-  "description": "Additional GPU rasterization whitelist for 
field trial",

+-  "cr_bugs": [380694],
+-  "exceptions": [
+-{
+-  "os": {
+-"type": "android"
+-  },
+-  "machine_model_name": ["HTC One",
+- "C5303", "C6603", "C6903",
+- "GT-I8262", "GT-I8552", 
"GT-I9195", "GT-I9300",

+- "GT-I9500", "GT-I9505", "GT-N7100",
+- "SAMSUNG-SCH-I337", "SCH-I545", 
"SGH-M919",

+- "SM-N900", "SM-N9005", "SPH-L720",
+- "XT907", "XT1032", "XT1033", 
"XT1080"]

+-}
+-  ],
+-  "features": [
+-"gpu_rasterization_field_trial"
+-  ]
+ }
+   ]
+ }
+--
+1.9.1
+
diff --git a/recipes-browser/chromium/chromium_37.0.2062.94.bb 
b/recipes-browser/chromium/chromium_37.0.2062.94.bb

index 510b644..3d4524b 100644
--- a/recipes-browser/chromium/chromium_37.0.2062.94.bb
+++ b/recipes-browser/chromium/chromium_37.0.2062.94.bb
@@ -8,7 +8,7 @@ SRC_URI = "\
  file://oe-defaults.gypi \
  ${@bb.utils.contains('PACKAGECONFIG', 'component-build', 
'file://component-build.gypi', '', d)} \
  ${@bb.utils.contains('PACKAGECONFIG', 
'ignore-lost-context', 
'file://remove-linux-accel-canvas-from-blacklist.patch', '', d)} \
-${@bb.utils.contains('PACKAGECONFIG', 'impl-side-painting', 
'file://disable-rasterization-whitelist-unlocking-impl-side-painting.patch', 
'', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'impl-side-painting', 
'file://0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch', 
'', d)} \
  ${@bb.utils.contains('PACKAGECONFIG', 
'disable-api-keys-info-bar', 
'file://0002-Disable-API-keys-info-bar.patch', '', d)} \

  file://unistd-2.patch \
  file://google-chrome \



Why is this important since it appears to only affect Android
based devices (and this is for OE builds)?



By default, it is only enabled for certain Android devices. With this 
patch, it is enabled for all devices, including for OE builds. impl side 
painting is still in an experimental stage, but promises better 
performance, which is why I made it an opt-in PACKAGECONFIG.

--
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-browser][PATCH] chromium: Add missing impl side painting patch

2014-09-25 Thread Gary Thomas 

On 2014-09-25 03:32, Carlos Rafael Giani wrote:

Signed-off-by: Carlos Rafael Giani 
---
  ...terization-whitelist-unlocking-impl-side-.patch | 73 ++
  recipes-browser/chromium/chromium_37.0.2062.94.bb  |  2 +-
  2 files changed, 74 insertions(+), 1 deletion(-)
  create mode 100644 
recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch

diff --git 
a/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
 
b/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
new file mode 100644
index 000..3e9d269
--- /dev/null
+++ 
b/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
@@ -0,0 +1,73 @@
+From 96a5a12ecd5b144bfeaa1d2116c2d1996bbb8f6c Mon Sep 17 00:00:00 2001
+From: Carlos Rafael Giani 
+Date: Mon, 22 Sep 2014 23:15:42 +0200
+Subject: [PATCH] Disable rasterization whitelist, unlocking impl side painting
+
+Signed-off-by: Carlos Rafael Giani 
+---
+ gpu/config/software_rendering_list_json.cc | 49 --
+ 1 file changed, 49 deletions(-)
+
+diff --git a/gpu/config/software_rendering_list_json.cc 
b/gpu/config/software_rendering_list_json.cc
+index 52614eb..bd0bf01 100644
+--- a/gpu/config/software_rendering_list_json.cc
 b/gpu/config/software_rendering_list_json.cc
+@@ -1144,55 +1144,6 @@ LONG_STRING_CONST(
+   "features": [
+ "all"
+   ]
+-},
+-{
+-  "id": 96,
+-  "description": "GPU rasterization is whitelisted on N4, N5, N7 and Moto 
X",
+-  "cr_bugs": [362779],
+-  "exceptions": [
+-{
+-  "os": {
+-"type": "android"
+-  },
+-  "machine_model_name": ["Nexus 4", "Nexus 5", "Nexus 7",
+- "XT1049", "XT1050", "XT1052", "XT1053",
+- "XT1055", "XT1056", "XT1058", "XT1060"]
+-},
+-{
+-  "os": {
+-"type": "android",
+-"version": {
+-  "op": ">=",
+-  "value": "4.4.99"
+-}
+-  }
+-}
+-  ],
+-  "features": [
+-"gpu_rasterization"
+-  ]
+-},
+-{
+-  "id": 97,
+-  "description": "Additional GPU rasterization whitelist for field trial",
+-  "cr_bugs": [380694],
+-  "exceptions": [
+-{
+-  "os": {
+-"type": "android"
+-  },
+-  "machine_model_name": ["HTC One",
+- "C5303", "C6603", "C6903",
+- "GT-I8262", "GT-I8552", "GT-I9195", 
"GT-I9300",
+- "GT-I9500", "GT-I9505", "GT-N7100",
+- "SAMSUNG-SCH-I337", "SCH-I545", "SGH-M919",
+- "SM-N900", "SM-N9005", "SPH-L720",
+- "XT907", "XT1032", "XT1033", "XT1080"]
+-}
+-  ],
+-  "features": [
+-"gpu_rasterization_field_trial"
+-  ]
+ }
+   ]
+ }
+--
+1.9.1
+
diff --git a/recipes-browser/chromium/chromium_37.0.2062.94.bb 
b/recipes-browser/chromium/chromium_37.0.2062.94.bb
index 510b644..3d4524b 100644
--- a/recipes-browser/chromium/chromium_37.0.2062.94.bb
+++ b/recipes-browser/chromium/chromium_37.0.2062.94.bb
@@ -8,7 +8,7 @@ SRC_URI = "\
  file://oe-defaults.gypi \
  ${@bb.utils.contains('PACKAGECONFIG', 'component-build', 
'file://component-build.gypi', '', d)} \
  ${@bb.utils.contains('PACKAGECONFIG', 'ignore-lost-context', 
'file://remove-linux-accel-canvas-from-blacklist.patch', '', d)} \
-${@bb.utils.contains('PACKAGECONFIG', 'impl-side-painting', 
'file://disable-rasterization-whitelist-unlocking-impl-side-painting.patch', 
'', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'impl-side-painting', 
'file://0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch', '', 
d)} \
  ${@bb.utils.contains('PACKAGECONFIG', 'disable-api-keys-info-bar', 
'file://0002-Disable-API-keys-info-bar.patch', '', d)} \
  file://unistd-2.patch \
  file://google-chrome \



Why is this important since it appears to only affect Android
based devices (and this is for OE builds)?

--

Gary Thomas |  Consulting for the
MLB Associates  |Embedded world

--
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

[oe] [meta-browser][PATCH] chromium: Add missing impl side painting patch

2014-09-25 Thread Carlos Rafael Giani 
Signed-off-by: Carlos Rafael Giani 
---
 ...terization-whitelist-unlocking-impl-side-.patch | 73 ++
 recipes-browser/chromium/chromium_37.0.2062.94.bb  |  2 +-
 2 files changed, 74 insertions(+), 1 deletion(-)
 create mode 100644 
recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch

diff --git 
a/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
 
b/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
new file mode 100644
index 000..3e9d269
--- /dev/null
+++ 
b/recipes-browser/chromium/chromium/0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch
@@ -0,0 +1,73 @@
+From 96a5a12ecd5b144bfeaa1d2116c2d1996bbb8f6c Mon Sep 17 00:00:00 2001
+From: Carlos Rafael Giani 
+Date: Mon, 22 Sep 2014 23:15:42 +0200
+Subject: [PATCH] Disable rasterization whitelist, unlocking impl side painting
+
+Signed-off-by: Carlos Rafael Giani 
+---
+ gpu/config/software_rendering_list_json.cc | 49 --
+ 1 file changed, 49 deletions(-)
+
+diff --git a/gpu/config/software_rendering_list_json.cc 
b/gpu/config/software_rendering_list_json.cc
+index 52614eb..bd0bf01 100644
+--- a/gpu/config/software_rendering_list_json.cc
 b/gpu/config/software_rendering_list_json.cc
+@@ -1144,55 +1144,6 @@ LONG_STRING_CONST(
+   "features": [
+ "all"
+   ]
+-},
+-{
+-  "id": 96,
+-  "description": "GPU rasterization is whitelisted on N4, N5, N7 and Moto 
X",
+-  "cr_bugs": [362779],
+-  "exceptions": [
+-{
+-  "os": {
+-"type": "android"
+-  },
+-  "machine_model_name": ["Nexus 4", "Nexus 5", "Nexus 7",
+- "XT1049", "XT1050", "XT1052", "XT1053",
+- "XT1055", "XT1056", "XT1058", "XT1060"]
+-},
+-{
+-  "os": {
+-"type": "android",
+-"version": {
+-  "op": ">=",
+-  "value": "4.4.99"
+-}
+-  }
+-}
+-  ],
+-  "features": [
+-"gpu_rasterization"
+-  ]
+-},
+-{
+-  "id": 97,
+-  "description": "Additional GPU rasterization whitelist for field trial",
+-  "cr_bugs": [380694],
+-  "exceptions": [
+-{
+-  "os": {
+-"type": "android"
+-  },
+-  "machine_model_name": ["HTC One",
+- "C5303", "C6603", "C6903",
+- "GT-I8262", "GT-I8552", "GT-I9195", 
"GT-I9300",
+- "GT-I9500", "GT-I9505", "GT-N7100",
+- "SAMSUNG-SCH-I337", "SCH-I545", "SGH-M919",
+- "SM-N900", "SM-N9005", "SPH-L720",
+- "XT907", "XT1032", "XT1033", "XT1080"]
+-}
+-  ],
+-  "features": [
+-"gpu_rasterization_field_trial"
+-  ]
+ }
+   ]
+ }
+-- 
+1.9.1
+
diff --git a/recipes-browser/chromium/chromium_37.0.2062.94.bb 
b/recipes-browser/chromium/chromium_37.0.2062.94.bb
index 510b644..3d4524b 100644
--- a/recipes-browser/chromium/chromium_37.0.2062.94.bb
+++ b/recipes-browser/chromium/chromium_37.0.2062.94.bb
@@ -8,7 +8,7 @@ SRC_URI = "\
 file://oe-defaults.gypi \
 ${@bb.utils.contains('PACKAGECONFIG', 'component-build', 
'file://component-build.gypi', '', d)} \
 ${@bb.utils.contains('PACKAGECONFIG', 'ignore-lost-context', 
'file://remove-linux-accel-canvas-from-blacklist.patch', '', d)} \
-${@bb.utils.contains('PACKAGECONFIG', 'impl-side-painting', 
'file://disable-rasterization-whitelist-unlocking-impl-side-painting.patch', 
'', d)} \
+${@bb.utils.contains('PACKAGECONFIG', 'impl-side-painting', 
'file://0001-Disable-rasterization-whitelist-unlocking-impl-side-.patch', '', 
d)} \
 ${@bb.utils.contains('PACKAGECONFIG', 'disable-api-keys-info-bar', 
'file://0002-Disable-API-keys-info-bar.patch', '', d)} \
 file://unistd-2.patch \
 file://google-chrome \
-- 
1.9.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel