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

Issue 11828028: [content shell] W3C SVG tests need to run in a smaller window (Closed)

Created:
7 years, 11 months ago by jochen (gone - plz use gerrit)
Modified:
7 years, 11 months ago
Reviewers:
jam, marja
CC:
chromium-reviews, vsevik, jam, sail+watch_chromium.org, joi+watch-content_chromium.org, yurys, darin-cc_chromium.org, pfeldman, jochen+watch_chromium.org
Visibility:
Public.

Description

[content shell] W3C SVG tests need to run in a smaller window BUG=111316 TEST=W3C SVG tests pass R=marja@chromium.org TBR=jam@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=176068

Patch Set 1 #

Patch Set 2 : updates #

Patch Set 3 : updates #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -24 lines) Patch
M content/shell/android/shell_manager.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M content/shell/shell.h View 2 chunks +2 lines, -1 line 0 comments Download
M content/shell/shell.cc View 3 chunks +5 lines, -8 lines 0 comments Download
M content/shell/shell_application_mac.mm View 1 chunk +1 line, -1 line 0 comments Download
M content/shell/shell_browser_main_parts.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/shell/shell_devtools_delegate.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/shell/shell_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/shell/shell_web_contents_view_delegate_gtk.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M content/shell/shell_web_contents_view_delegate_mac.mm View 2 chunks +2 lines, -2 lines 0 comments Download
M content/shell/shell_web_contents_view_delegate_win.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M content/shell/shell_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/shell/webkit_test_controller.cc View 1 chunk +5 lines, -1 line 0 comments Download
M content/test/content_browser_test.cc View 1 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
jochen (gone - plz use gerrit)
7 years, 11 months ago (2013-01-09 20:46:20 UTC) #1
marja
lgtm
7 years, 11 months ago (2013-01-10 09:30:59 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jochen@chromium.org/11828028/1
7 years, 11 months ago (2013-01-10 09:33:20 UTC) #3
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
7 years, 11 months ago (2013-01-10 09:56:26 UTC) #4
jochen (gone - plz use gerrit)
TBR'ing John for content/test/content_browser_test.cc - updated Shell API
7 years, 11 months ago (2013-01-10 09:57:20 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jochen@chromium.org/11828028/12003
7 years, 11 months ago (2013-01-10 09:57:30 UTC) #6
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
7 years, 11 months ago (2013-01-10 10:23:38 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jochen@chromium.org/11828028/14
7 years, 11 months ago (2013-01-10 10:26:17 UTC) #8
commit-bot: I haz the power
7 years, 11 months ago (2013-01-10 14:18:26 UTC) #9
Message was sent while issue was closed.
Change committed as 176068

Powered by Google App Engine
This is Rietveld 408576698