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

Issue 7277088: Fix issues in IT2Me webapp caused by r91658. (Closed)

Created:
9 years, 5 months ago by Sergey Ulanov
Modified:
9 years, 5 months ago
CC:
chromium-reviews, jamiewalch+watch_chromium.org, hclam+watch_chromium.org, simonmorris+watch_chromium.org, wez+watch_chromium.org, dmaclach+watch_chromium.org, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, ajwong+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Fix issues in IT2Me webapp caused by r91658. 1. Elemnt IDs were updated in chroice.html, but not in remoting.js 2. Plugin container must not be hidden, otherwise the plugin doesn't load. BUG=None TEST=IT2Me host works. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=91750

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M remoting/webapp/me2mom/choice.html View 1 chunk +1 line, -1 line 0 comments Download
M remoting/webapp/me2mom/remoting.js View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Sergey Ulanov
9 years, 5 months ago (2011-07-07 19:47:11 UTC) #1
awong
LGTM
9 years, 5 months ago (2011-07-07 19:48:05 UTC) #2
commit-bot: I haz the power
9 years, 5 months ago (2011-07-07 20:39:55 UTC) #3
Try job failure for 7277088-1 (retry) on win for step "compile" (clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

Powered by Google App Engine
This is Rietveld 408576698