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

Issue 10025004: Floating point based Point/Size/Rect and Insets (Closed)

Created:
8 years, 8 months ago by oshima
Modified:
8 years, 8 months ago
Reviewers:
sky
CC:
chromium-reviews, pkotwicz
Visibility:
Public.

Description

Floating point based Point/Size/Rect and Insets This is copied from integer version, instead of using template, to minimize conflict with m19 branch, as using template requires updating forward declaration of these classes in many places. I put this behind gyp flag for now so that we can move forward without breaking non DIP build until we can get aura working with DIP. BUG=114664 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=131405

Patch Set 1 #

Patch Set 2 : . #

Patch Set 3 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+713 lines, -0 lines) Patch
M build/common.gypi View 1 2 5 chunks +9 lines, -0 lines 0 comments Download
A ui/gfx/insets_f.h View 1 2 1 chunk +89 lines, -0 lines 0 comments Download
A ui/gfx/insets_f.cc View 1 chunk +16 lines, -0 lines 0 comments Download
A ui/gfx/point_f.h View 1 2 1 chunk +93 lines, -0 lines 0 comments Download
A ui/gfx/point_f.cc View 1 chunk +26 lines, -0 lines 0 comments Download
A ui/gfx/rect_f.h View 1 2 1 chunk +154 lines, -0 lines 0 comments Download
A ui/gfx/rect_f.cc View 1 chunk +204 lines, -0 lines 0 comments Download
A ui/gfx/size_f.h View 1 2 1 chunk +69 lines, -0 lines 0 comments Download
A ui/gfx/size_f.cc View 1 chunk +41 lines, -0 lines 0 comments Download
M ui/ui.gyp View 1 1 chunk +12 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
oshima
8 years, 8 months ago (2012-04-09 17:15:56 UTC) #1
oshima
8 years, 8 months ago (2012-04-09 17:20:25 UTC) #2
sky
LGTM
8 years, 8 months ago (2012-04-09 17:26:23 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/oshima@chromium.org/10025004/7001
8 years, 8 months ago (2012-04-09 17:56:44 UTC) #4
commit-bot: I haz the power
8 years, 8 months ago (2012-04-09 19:27:47 UTC) #5
Change committed as 131405

Powered by Google App Engine
This is Rietveld 408576698