astojilj

I/GFX all (18555): took 0.001231ms for 2320*1024 texture 4
I/GFX all (18555): took 0.001154ms for 192*192 texture 4
I/GFX all (18555): took 0.001384ms for 192*192 texture 4
I/GFX all (18555): took 0.001384ms for 48*48 texture 4
I/GFX all (18555): took 0.001385ms for 192*192 texture 4
I/GFX all (18555): took 0.001154ms for 192*192 texture 4
I/GFX all (18555): took 0.001076ms for 192*192 texture 4
I/GFX all (18555): took 0.001231ms for 2320*1024 texture 4
I/GFX all (18555): took 0.001770ms for 2320*1024 texture 4
I/GFX all (18555): took 0.001385ms for 2320*1024 texture 4

astojilj

I/GFX all before(20523): took 0.172307ms for 2320*1024 texture 4
I/GFX all before(20523): took 0.099000ms for 2320*1024 texture 4
I/GFX all before(20523): took 0.437923ms for 2320*1024 texture 4
I/GFX all before(20523): took 0.314308ms for 2320*1024 texture 4
I/GFX all before(20523): took 0.028307ms for 192*192 texture 4
I/GFX all before(20523): took 0.016385ms for 192*192 texture 4
I/GFX all before(20523): took 0.015308ms for 48*48 texture 4
I/GFX all before(20523): took 0.019923ms for 192*192 texture 4
I/GFX all before(20523): took 0.050385ms for 192*192 texture 4

astojilj

I/GFX 1   ( 3269): took 0.001615ms for 2320*1024 texture 4
I/GFX 2   ( 3269): took 0.631077ms for 2320*1024 texture 4
I/GFX 3   ( 3269): took 0.008846ms for 2320*1024 texture 4
I/GFX 1   ( 3269): took 0.001693ms for 192*192 texture 4
I/GFX 2   ( 3269): took 0.068923ms for 192*192 texture 4
I/GFX 3   ( 3269): took 0.007769ms for 192*192 texture 4
I/GFX 1   ( 3269): took 0.002769ms for 192*192 texture 4
I/GFX 2   ( 3269): took 0.014462ms for 192*192 texture 4
I/GFX 3   ( 3269): took 0.008308ms for 192*192 texture 4
I/GFX 1   ( 3269): took 0.001769ms for 48*48 texture 4

astojilj

I/GFX 1   ( 4299): took 0.001461ms for 2320*1024 texture 4
I/GFX 2_1 ( 4299): took 0.384077ms for 2320*1024 texture 4
I/GFX 2_2 ( 4299): took 0.001077ms for 2320*1024 texture 4
I/GFX 2_3 ( 4299): took 0.000538ms for 2320*1024 texture 4
I/GFX 3   ( 4299): took 0.007846ms for 2320*1024 texture 4
I/GFX 1   ( 4299): took 0.001616ms for 192*192 texture 4
I/GFX 2_1 ( 4299): took 0.025153ms for 192*192 texture 4
I/GFX 2_2 ( 4299): took 0.000769ms for 192*192 texture 4
I/GFX 2_3 ( 4299): took 0.000616ms for 192*192 texture 4
I/GFX 3   ( 4299): took 0.006461ms for 192*192 texture 4

astojilj

Before - total on decoding:3019ms
After - total on decoding:  909ms

Trace before the patch
======================
I/GFXGIF  (14754): took 3.968385ms for 512*384 frame 1 depends on 0
I/     GFX(14754): decoding frame 2
I/GFXGIF  (14754): took 4.166769ms for 512*384 frame 2 depends on 1
I/     GFX(14754): decoding frame 3
I/GFXGIF  (14754): took 7.444692ms for 512*384 frame 3 depends on 2

astojilj

diff --git a/third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp b/third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp
index 09f931b..a9a80e0 100644
--- a/third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp
+++ b/third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp
@@ -294,7 +294,7 @@ SkBitmap ImageFrameGenerator::tryToResumeDecode(size_t index, const SkISize& sca
             if (*it)
                 decodedFrameCount++;
         }
-        removeDecoder = m_frameCount && (decodedFrameCount == m_frameCount);
+        removeDecoder = m_frameCount && (decodedFrameCount == m_frameCount) && (index == m_frameCount - 1);

astojilj

diff --git a/src/core/SkImageCacherator.cpp b/src/core/SkImageCacherator.cpp
index 9074924..767900c 100644
--- a/src/core/SkImageCacherator.cpp
+++ b/src/core/SkImageCacherator.cpp
@@ -81,6 +81,29 @@ static bool check_output_bitmap(const SkBitmap& bitmap, uint32_t expectedID) {
     return true;
 }
 
+#define SKGR_DEBUG
+#ifdef SKGR_DEBUG

astojilj

C:\xwalk\src>out\Debug\xwalk.exe --enable-logging --v=1 '"C:\Program Files\usecase-webapi-xwalk-tests\usecase-webapi-xwalk-tests\manifest.windows.json"

C:\xwalk\src>[8096:7560:0126/152723:INFO:skopts.cpp(86)] sse 4.2 detected

[8096:7560:0126/152723:INFO:skopts.cpp(88)] avx2 detected

[8096:7560:0126/152723:ERROR:xwalk_platform_notification_service.cc(103)] NOT IMPLEMENTED
[8096:7560:0126/152723:VERBOSE1:media_stream_manager.cc(450)] ~MediaStreamManager
[8096:7560:0126/152723:VERBOSE1:notification_service_impl.cc(145)] 1 notification observer(s) leaked of notification type 14
[8096:7560:0126/152723:VERBOSE1:xwalk_runner.cc(56)] Destroying XWalkRunner object.

astojilj

C:\xwalk\src>ninja -C out\Debug_x64 xwalk
ninja: Entering directory `out\Debug_x64'
[6/14276] CXX obj\v8\src\base\utils\v8_libbase.random-number-generator.obj
FAILED: ninja -t msvc -e environment.x64 -- "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\amd64\cl.exe" /nologo /showIncludes /FC @obj\v8\src\base\utils\v8_libbase.random-number-generator.obj.rsp /c ..\..\v8\src\base\utils\random-number-generator.cc /Foobj\v8\src\base\utils\v8_libbase.random-number-generator.obj /Fdobj\v8\tools\gyp\v8_libbase.cc.pdb
cl : Command line warning D9002 : ignoring unknown option '/arch:SSE2'
c:\xwalk\src\v8\src\base\build_config.h(132): fatal error C1189: #error:  Target architecture ia32 is only supported on ia32 host
[6/14276] ACTION Generating resources from ../webui/resources/webui_resources.grd
ninja: build stopped: subcommand failed.

astojilj

C:\xwalk\src>python xwalk/gyp_xwalk
Enabled Psyco JIT.
Traceback (most recent call last):
  File "xwalk/gyp_xwalk", line 375, in <module>
    sys.exit(main())
  File "xwalk/gyp_xwalk", line 262, in main
    gyp_environment.SetEnvironment()
  File "C:\xwalk\src\build\gyp_environment.py", line 18, in SetEnvironment
    gyp_helper.apply_chromium_gyp_env()
  File "C:\xwalk\src\build\gyp_helper.py", line 68, in apply_chromium_gyp_env