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

Issue 2452133002: Revert of [ObjC ARC] Converts ios/chrome/app to ARC. (Closed)

Created:
4 years, 1 month ago by hongchan
Modified:
4 years, 1 month ago
CC:
chromium-reviews, mac-reviews_chromium.org, sdefresne+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of [ObjC ARC] Converts ios/chrome/app to ARC. (patchset #1 id:1 of https://codereview.chromium.org/2449063002/ ) Reason for revert: This CL closed the tree. The following is a partial log from the dump. --- [1265/3927] OBJCXX obj/ios/chrome/browser/translate/translate/never_translate_infobar_controller.o [1266/3927] OBJCXX obj/ios/chrome/app/app/safe_mode_crashing_modules_config.o FAILED: obj/ios/chrome/app/app/safe_mode_crashing_modules_config.o /b/build/slave/cache/cipd/goma/gomacc ../../third_party/llvm-build/Release+Asserts/bin/clang++ -MMD -MF obj/ios/chrome/app/app/safe_mode_crashing_modules_config.o.d -DV8_DEPRECATION_WARNINGS -DNO_TCMALLOC -DDISABLE_NACL -DCHROMIUM_BUILD -DFIELDTRIAL_TESTING_ENABLED -DCR_CLANG_REVISION=284979-1 -DCR_XCODE_VERSION=0800 -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D_DEBUG -DDYNAMIC_ANNOTATIONS_ENABLED=1 -DWTF_USE_DYNAMIC_ANNOTATIONS=1 -I../.. -Igen -fno-strict-aliasing --param=ssp-buffer-size=4 -fstack-protector -fcolor-diagnostics -arch armv7 -Wall -Werror -Wextra -Wno-missing-field-initializers -Wno-unused-parameter -Wno-c++11-narrowing -Wno-covered-switch-default -Wno-deprecated-register -Wno-unneeded-internal-declaration -Wno-inconsistent-missing-override -Wno-shift-negative-value -Wno-undefined-var-template -Wno-nonportable-include-path -Wno-address-of-packed-member -O0 -gdwarf-2 -isysroot /Applications/Xcode8.0.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS10.0.sdk -stdlib=libc++ -miphoneos-version-min=9.0 -fvisibility=hidden -Xclang -load -Xclang ../../third_party/llvm-build/Release+Asserts/lib/libFindBadConstructs.dylib -Xclang -add-plugin -Xclang find-bad-constructs -Wheader-hygiene -Wstring-conversion -fno-threadsafe-statics -fvisibility-inlines-hidden -Wno-undefined-bool-conversion -Wno-tautological-undefined-compare -std=c++11 -fobjc-call-cxx-cdtors -Wobjc-missing-property-synthesis -fno-rtti -fno-exceptions -c ../../ios/chrome/app/safe_mode_crashing_modules_config.mm -o obj/ios/chrome/app/app/safe_mode_crashing_modules_config.o ../../ios/chrome/app/safe_mode_crashing_modules_config.mm:12:2: error: "This file requires ARC support." #error "This file requires ARC support." ^ 1 error generated. [1267/3927] CXX obj/components/browser_sync/browser_sync/profile_sync_service.o ninja: build stopped: subcommand failed. Original issue's description: > [ObjC ARC] Converts ios/chrome/app to ARC. > > Notable changes:none. > > BUG=659150 > TEST=None. > > Committed: https://crrev.com/7a1f218b33faf5d0dd637f6a6854566e6cd80a3d > Cr-Commit-Position: refs/heads/master@{#427722} TBR=eugenebut@chromium.org,thestig@chromium.org,stkhapugin@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=659150

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -18 lines) Patch
M ios/chrome/app/BUILD.gn View 1 chunk +0 lines, -2 lines 0 comments Download
M ios/chrome/app/application_delegate/memory_warning_helper.mm View 1 chunk +0 lines, -4 lines 0 comments Download
M ios/chrome/app/deferred_initialization_runner.mm View 4 chunks +6 lines, -8 lines 0 comments Download
M ios/chrome/app/safe_mode_crashing_modules_config.mm View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
hongchan
Created Revert of [ObjC ARC] Converts ios/chrome/app to ARC.
4 years, 1 month ago (2016-10-26 17:31:29 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2452133002/1
4 years, 1 month ago (2016-10-26 17:32:23 UTC) #3
commit-bot: I haz the power
Failed to apply patch for ios/chrome/app/BUILD.gn: While running git apply --index -3 -p1; error: patch ...
4 years, 1 month ago (2016-10-26 17:33:10 UTC) #5
Eugene But (OOO till 7-30)
lgtm, it's interesting that landed CL has passed CQ :(
4 years, 1 month ago (2016-10-26 17:34:47 UTC) #6
Lei Zhang
4 years, 1 month ago (2016-10-26 17:34:57 UTC) #7
You are behind. It's already been reverted. Please coordinate via #chromium.

Powered by Google App Engine
This is Rietveld 408576698