Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(26)

Issue 11410051: Moved AlphaBlend rasterization to renderer process like it was before 152681. (Closed)

Created:
8 years, 1 month ago by Vitaly Buka (NO REVIEWS)
Modified:
8 years, 1 month ago
Reviewers:
Lei Zhang, Nico, gene
CC:
chromium-reviews, darin-cc_chromium.org
Visibility:
Public.

Description

Moved AlphaBlend rasterization to renderer process like it was before 152681. 152681 moved all rasterization from renderer to browser process and introduced bug 152204. The bug was fixed by 161954, CL returns AlphaBlend rasterization to similar algorithm as it was before 152681, but I still keeps AlphaBlend in browser process. For some unknown reasons AlphaBlend rasterization on XP works only in renderer process. BUG=160154 NOTRY=true Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=167271

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M chrome/browser/printing/print_view_manager.cc View 1 chunk +0 lines, -7 lines 0 comments Download
M chrome/renderer/print_web_view_helper_win.cc View 2 chunks +8 lines, -1 line 0 comments Download

Messages

Total messages: 10 (0 generated)
Vitaly Buka (NO REVIEWS)
8 years, 1 month ago (2012-11-12 23:58:48 UTC) #1
gene
lgtm
8 years, 1 month ago (2012-11-13 00:00:53 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vitalybuka@chromium.org/11410051/1
8 years, 1 month ago (2012-11-13 00:03:56 UTC) #3
commit-bot: I haz the power
Presubmit check for 11410051-1 failed and returned exit status 1. Running presubmit commit checks ...
8 years, 1 month ago (2012-11-13 00:03:59 UTC) #4
Vitaly Buka (NO REVIEWS)
8 years, 1 month ago (2012-11-13 00:05:40 UTC) #5
Nico
lgtm stamp Please update the Cl description with what was broken and how this change ...
8 years, 1 month ago (2012-11-13 00:25:40 UTC) #6
Vitaly Buka (NO REVIEWS)
Done On 2012/11/13 00:25:40, Nico wrote: > lgtm stamp > > Please update the Cl ...
8 years, 1 month ago (2012-11-13 00:34:52 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vitalybuka@chromium.org/11410051/1
8 years, 1 month ago (2012-11-13 00:35:40 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vitalybuka@chromium.org/11410051/11001
8 years, 1 month ago (2012-11-13 00:47:20 UTC) #9
commit-bot: I haz the power
8 years, 1 month ago (2012-11-13 00:48:46 UTC) #10
Change committed as 167271

Powered by Google App Engine
This is Rietveld 408576698