Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1653)

Issue 7571045: Add translate & scale optimizations (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 9 months ago by robertphillips
Modified:
11 years, 9 months ago
Reviewers:
DerekS
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Something to do during a meeting. The translate peephole will probably be useful (mainly for removing useless commands). The scale optimization - not so much. Executing desk_amazon.skp 5: 11 Executing desk_baidu.skp 5: 2 Executing desk_blogger.skp 5: 15 Executing desk_booking.skp 5: 3 Executing desk_br337.skp 5: 4 Executing desk_chalkboard.skp 5: 2 6: 1 Executing desk_css3gradients.skp 5: 2 Executing desk_ebay.skp 5: 3 Executing desk_espn.skp 5: 6 Executing desk_facebook.skp 5: 2 Executing desk_gmailthread.skp 5: 2 Executing desk_googleplus.skp 5: 42 Executing desk_googlespreadsheet.skp 5: 2 Executing desk_googlespreadsheetdashed.skp 5: 2 Executing desk_gws.skp 5: 2 Executing desk_linkedin.skp 5: 3 Executing desk_pinterest.skp 5: 3 Executing desk_pokemonwiki.skp 5: 2 Executing desk_sfgate.skp 5: 12 Executing desk_techcrunch.skp 5: 9 Executing desk_twitter.skp 5: 3 Executing desk_weather.skp 5: 16 Executing desk_wordpress.skp 5: 4 Executing desk_wowwiki.skp 5: 81 Executing desk_yahooanswers.skp 5: 3 Executing desk_yahoogames.skp 5: 8 Executing desk_yahoonews.skp 5: 8 Executing desk_yahoosports.skp 5: 18 Executing desk_youtube.skp 5: 5 Executing mobi_wikipedia.skp 5: 2 Executing tabl_androidpolice.skp 5: 3 Executing tabl_cnet.skp 5: 7 Executing tabl_cnn.skp 5: 19 Executing tabl_culturalsolutions.skp 5: 2 Executing tabl_cuteoverload.skp 5: 9 Executing tabl_deviantart.skp 5: 5 Executing tabl_digg.skp 5: 2 Executing tabl_engadget.skp 5: 14 Executing tabl_frantzen.skp 5: 4 Executing tabl_gamedeksiam.skp 5: 4 Executing tabl_gmail.skp 5: 2 Executing tabl_googleblog.skp 5: 25 Executing tabl_googlecalendar.skp 5: 3 Executing tabl_gspro.skp 5: 10 Executing tabl_hsfi.skp 5: 26 Executing tabl_mercurynews.skp 5: 5 Executing tabl_mlb.skp 5: 18 Executing tabl_mozilla.skp 5: 2 Executing tabl_nofolo.skp 5: 2 Executing tabl_nytimes.skp 5: 4 Executing tabl_onlinewsj.skp 5: 16 Executing tabl_pravda.skp 5: 7 Executing tabl_sahadan.skp 5: 6 Executing tabl_slashdot.skp 5: 9 Executing tabl_techmeme.skp 5: 2 Executing tabl_theverge.skp 5: 22 Executing tabl_transformice.skp 5: 2 Executing tabl_ukwsj.skp 5: 12 Executing tabl_vnexpress.skp 5: 19 Executing tabl_worldjournal.skp 5: 31 opt 5: 569 opt 6: 1

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+51 lines, -0 lines) Patch
M debugger/SkDrawCommand.h View 2 chunks +8 lines, -0 lines 0 comments Download
M tools/filtermain.cpp View 2 chunks +43 lines, -0 lines 0 comments Download

Messages

Total messages: 3
robertphillips
11 years, 9 months ago (2013-03-14 18:26:59 UTC) #1
DerekS
lgtm. Sorry for the delay, I forget to check appspot for reviews now that most ...
11 years, 9 months ago (2013-03-15 16:58:43 UTC) #2
robertphillips
11 years, 9 months ago (2013-03-17 18:46:46 UTC) #3
Message was sent while issue was closed.
committed as r8182
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b