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

Issue 159763: Rolling back change 22245. (Closed)

Created:
11 years, 4 months ago by bradn
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com, darin (slow to review), Erik does not do reviews, brettw, jam, Ben Goodger (Google)
Base URL:
svn://chrome-svn.corp.google.com/chrome/trunk/src/
Visibility:
Public.

Description

Rolling back change 22245. Too much red :-( BUG=None TEST=None TBR=erikkay Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22247

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+128 lines, -715 lines) Patch
M chrome/browser/browser.h View 3 chunks +0 lines, -7 lines 0 comments Download
M chrome/browser/browser.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M chrome/browser/extensions/extension_function_dispatcher.h View 3 chunks +0 lines, -6 lines 0 comments Download
M chrome/browser/extensions/extension_function_dispatcher.cc View 5 chunks +0 lines, -18 lines 0 comments Download
M chrome/browser/extensions/extension_host.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/extensions/extension_host.cc View 3 chunks +2 lines, -9 lines 0 comments Download
M chrome/browser/extensions/extension_shelf_model.h View 6 chunks +39 lines, -60 lines 0 comments Download
M chrome/browser/extensions/extension_shelf_model.cc View 9 chunks +24 lines, -47 lines 0 comments Download
M chrome/browser/extensions/extension_shelf_model_unittest.cc View 1 chunk +5 lines, -5 lines 0 comments Download
D chrome/browser/extensions/extension_toolstrip_api.h View 1 chunk +0 lines, -33 lines 0 comments Download
D chrome/browser/extensions/extension_toolstrip_api.cc View 1 chunk +0 lines, -114 lines 0 comments Download
M chrome/browser/gtk/extension_shelf_gtk.h View 2 chunks +2 lines, -3 lines 0 comments Download
M chrome/browser/gtk/extension_shelf_gtk.cc View 5 chunks +6 lines, -19 lines 0 comments Download
M chrome/browser/views/extensions/extension_shelf.h View 4 chunks +3 lines, -11 lines 0 comments Download
M chrome/browser/views/extensions/extension_shelf.cc View 14 chunks +38 lines, -110 lines 0 comments Download
M chrome/browser/views/frame/browser_view.cc View 2 chunks +6 lines, -12 lines 0 comments Download
M chrome/chrome.gyp View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/common/common_resources.grd View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/extension_api.json View 2 chunks +1 line, -31 lines 0 comments Download
D chrome/common/extensions/docs/toolstrip.html View 1 chunk +0 lines, -46 lines 0 comments Download
M chrome/renderer/extensions/extension_api_client_unittest.cc View 1 chunk +0 lines, -41 lines 0 comments Download
M chrome/renderer/renderer_resources.grd View 1 chunk +1 line, -1 line 0 comments Download
D chrome/test/data/extensions/samples/mappy/manifest.json View 1 chunk +0 lines, -11 lines 0 comments Download
D chrome/test/data/extensions/samples/mappy/mappy_content_script.js View 1 chunk +0 lines, -55 lines 0 comments Download
D chrome/test/data/extensions/samples/mappy/mappy_toolstrip.html View 1 chunk +0 lines, -68 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
bradn
I think I'm gonna need to roll this back out Erik, sorry. After a complete ...
11 years, 4 months ago (2009-08-02 00:12:55 UTC) #1
Erik does not do reviews
LGTM :-( Do we know who owns this bot? I have no way of trying ...
11 years, 4 months ago (2009-08-02 02:28:18 UTC) #2
bradn
I think its rspangler. According to dave moore the only difference between this and a ...
11 years, 4 months ago (2009-08-02 02:37:31 UTC) #3
bradn
So apparently the way you can reproduce it is to set the GYP_DEFINES environment variable ...
11 years, 4 months ago (2009-08-02 17:44:46 UTC) #4
bradn
11 years, 4 months ago (2009-08-02 17:46:15 UTC) #5
Oh and be sure to run gclient runhooks --force after that.
Followed by hammer --implicit-deps-changed chrome

-BradN

On Sun, Aug 2, 2009 at 10:44 AM, Bradley Nelson <bradnelson@google.com>wrote:

> So apparently the way you can reproduce it is to set the GYP_DEFINES
> environment variable to: chromeos=1 branding=Chrome
>
> -BradN
>
>
>
> On Sat, Aug 1, 2009 at 7:36 PM, Bradley Nelson <bradnelson@google.com>wrote:
>
>> I think its rspangler.
>> According to dave moore the only difference between this and a generic
>> chrome builder is that OS_CHROMEOS gets set.
>> This affects a range of #defines, but I can't see any obvious connection
>> with the failures.
>> If I catch someone from their team I'll ask them to take a look.
>>
>> -BradN
>>
>>
>> On Sat, Aug 1, 2009 at 7:28 PM, <erikkay@chromium.org> wrote:
>>
>>> LGTM :-(
>>>
>>> Do we know who owns this bot?  I have no way of trying to repro / fix
>>> this bug.
>>>
>>>
>>>
>>> http://codereview.chromium.org/159763
>>>
>>
>>
>

Powered by Google App Engine
This is Rietveld 408576698