Chromium Code Reviews
[email protected] (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(187)

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 13933025: Revert Revert 194782 "Native api to get OAuth2 access tokens in Print P..." (Closed)

Created:
7 years, 8 months ago by Vitaly Buka (NO REVIEWS)
Modified:
7 years, 8 months ago
Reviewers:
gene
CC:
chromium-reviews, arv+watch_chromium.org
Visibility:
Public.

Description

Revert Revert 194782 "Native api to get OAuth2 access tokens in Print P..." >> Native api to get OAuth2 access tokens in Print Preview Web UI. >> >> BUG=179229 >> >> Review URL: https://chromiumcodereview.appspot.com/14093016 > Review URL: https://codereview.chromium.org/13898008 [email protected] Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194874

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+128 lines, -0 lines) Patch
M chrome/browser/resources/print_preview/native_layer.js View 4 chunks +24 lines, -0 lines 0 comments Download
M chrome/browser/ui/webui/print_preview/print_preview_handler.h View 3 chunks +11 lines, -0 lines 0 comments Download
M chrome/browser/ui/webui/print_preview/print_preview_handler.cc View 1 2 8 chunks +93 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/13933025/1
7 years, 8 months ago (2013-04-18 07:18:53 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/13933025/5003
7 years, 8 months ago (2013-04-18 07:41:14 UTC) #2
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
7 years, 8 months ago (2013-04-18 08:02:16 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/13933025/16001
7 years, 8 months ago (2013-04-18 08:08:31 UTC) #4
commit-bot: I haz the power
7 years, 8 months ago (2013-04-18 10:21:41 UTC) #5
Message was sent while issue was closed. 
Change committed as 194874

Powered by Google App Engine
This is Rietveld 408576698