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

Issue 243463006: Merge 264451 "Fixes the crash for "learn more" link of the dialo..." (Closed)

Created:
6 years, 8 months ago by Jun Mukai
Modified:
6 years, 8 months ago
Reviewers:
Jun Mukai
CC:
chromium-reviews, nkostylev+watch_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org, oshima+watch_chromium.org, dkrahn+watch_chromium.org
Visibility:
Public.

Description

Merge 264451 "Fixes the crash for "learn more" link of the dialo..." > Fixes the crash for "learn more" link of the dialog for v2 apps. > > It assumed a browser but it might not exist in case of v2 apps. > > BUG=363716 > R=dkrahn@chromium.org, bartfab@chromium.org > > Review URL: https://codereview.chromium.org/236203017 TBR=mukai@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=265098

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -9 lines) Patch
M chrome/browser/chromeos/attestation/platform_verification_dialog.h View 3 chunks +2 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/attestation/platform_verification_dialog.cc View 5 chunks +21 lines, -5 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Jun Mukai
6 years, 8 months ago (2014-04-21 22:23:14 UTC) #1
Jun Mukai
6 years, 8 months ago (2014-04-21 22:23:28 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r265098.

Powered by Google App Engine
This is Rietveld 408576698