Skip to content
This repository has been archived by the owner on Apr 3, 2020. It is now read-only.

Cameo webview proto #24

Merged
merged 4 commits into from
Jul 1, 2013
Merged

Cameo webview proto #24

merged 4 commits into from
Jul 1, 2013

Conversation

ds-hwang
Copy link
Contributor

@ds-hwang ds-hwang commented Jul 1, 2013

Work in progress for libraryfication.

@Trybot
Copy link

Trybot commented Jul 1, 2013

Trybot Dashboard


The building Status of PATCH

[content_linux] Building... LOGS
[content_win] Building... LOGS
[content_android] Building... LOGS

drott pushed a commit that referenced this pull request Jul 1, 2013
Dongseong's changes towards generating a library.
@drott drott merged commit 0809676 into crosswalk-project:cameo_webview_proto Jul 1, 2013
huningxin pushed a commit to huningxin/chromium-croswalk that referenced this pull request May 4, 2014
Leaks: Linux LSan:
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%282%29/builds/1508

E.g.:
Indirect leak of 1544 byte(s) in 1 object(s) allocated from:
    #0 0x4fc78b in operator new(unsigned long) /usr/local/google/home/thakis/src/chrome/src/third_party/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:62
    #1 0x232ebeb in allocate /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/ext/new_allocator.h:92
    #2 0x232ebeb in _M_allocate /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/bits/stl_vector.h:150
    #3 0x232ebeb in _M_allocate_and_copy<std::move_iterator<__gnu_cxx::_Hashtable_node<std::pair<const std::basic_string<char>, content::SiteInstance *> > **> > /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/bits/stl_vector.h:1052
    #4 0x232ebeb in reserve /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/bits/vector.tcc:74
    crosswalk-project#5 0x232ebeb in _M_initialize_buckets /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/backward/hashtable.h:577
    crosswalk-project#6 0x232ebeb in __gnu_cxx::hashtable<std::pair<std::string const, content::SiteInstance*>, std::string, __gnu_cxx::hash<std::string>, std::_Select1st<std::pair<std::string const, content::SiteInstance*> >, std::equal_to<std::string>, std::allocator<content::SiteInstance*> >::hashtable(unsigned long, __gnu_cxx::hash<std::string> const&, std::equal_to<std::string> const&, std::allocator<std::pair<std::string const, content::SiteInstance*> > const&) /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/backward/hashtable.h:334
    crosswalk-project#7 0x232cb58 in RefCounted /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/ext/hash_map:126
    crosswalk-project#8 0x232cb58 in content::BrowsingInstance::BrowsingInstance(content::BrowserContext*) content/browser/browsing_instance.cc:18
    crosswalk-project#9 0x28d4131 in content::SiteInstance::Create(content::BrowserContext*) content/browser/site_instance_impl.cc:220
    crosswalk-project#10 0x2db9c92 in CreateTestWebContents content/public/test/test_renderer_host.cc:138
    crosswalk-project#11 0x2db9c92 in content::RenderViewHostTestHarness::SetUp() content/public/test/test_renderer_host.cc:182
    crosswalk-project#12 0xda5e2d in content::RenderFrameHostManagerTest::SetUp() content/browser/frame_host/render_frame_host_manager_unittest.cc:204
    crosswalk-project#13 0x2eea0f2 in HandleExceptionsInMethodIfSupported<testing::Test, void> testing/gtest/src/gtest.cc:2045
    crosswalk-project#14 0x2eea0f2 in testing::Test::Run() testing/gtest/src/gtest.cc:2057
    crosswalk-project#15 0x2eec3a9 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2237
    crosswalk-project#16 0x2eed183 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2344
    crosswalk-project#17 0x2efe463 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4065
    crosswalk-project#18 0x2efda30 in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2045
    crosswalk-project#19 0x2efda30 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:3697
    crosswalk-project#20 0x2e7712c in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2231
    crosswalk-project#21 0x2e7712c in base::TestSuite::Run() base/test/test_suite.cc:213
    crosswalk-project#22 0x2e6aebd in Run base/callback.h:401
    crosswalk-project#23 0x2e6aebd in base::(anonymous namespace)::LaunchUnitTestsInternal(int, char**, base::Callback<int ()> const&, int) base/test/launcher/unit_test_launcher.cc:494
    crosswalk-project#24 0x1bfec66 in main content/test/run_all_unittests.cc:14
    crosswalk-project#25 0x7f853fc5876c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

> Introduce RenderFrameProxyHost object and use it in RFHM.
> 
> This is the first CL in a series to create RenderFrameProxy(Host) infrastructure. Before the Blink codebase is ready to transform local and remote frames, the proxy objects will keep internally the existing RF/RFH in swapped out state. This CL creates the browser side proxy object and wraps the swapped out RFH.
> 
> BUG=357747
> 
> Review URL: https://codereview.chromium.org/217163007

[email protected]

Review URL: https://codereview.chromium.org/236003002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@263367 0039d316-1c4b-4281-b951-d872f2087c98
rakuco pushed a commit that referenced this pull request Aug 11, 2014
…ps://codereview.chromium.org/339433003/)

Reason for revert:
Broke linux lsan: http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%282%29/builds/4075/steps/browser_tests/logs/EndToEnd

Direct leak of 336 byte(s) in 1 object(s) allocated from:
    #0 0x514ffb in operator new(unsigned long) /usr/local/google/work/chromium/src/third_party/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:55
    #1 0xbdd835 in extensions::CastStreamingApiTestWithPixelOutput_EndToEnd_Test::RunTestOnMainThread() chrome/browser/extensions/api/cast_streaming/cast_streaming_apitest.cc:314
<truncated, full output is in gzipped JSON output at end of step>
    #23 0x3e70dd1 in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2045
    #24 0x3e70dd1 in testing::Test::Run() testing/gtest/src/gtest.cc:2057
    #25 0x3e73169 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2237
    #26 0x3e73ef6 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2344
    #27 0x3e8700a in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4065
    #28 0x3e86640 in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2045
    #29 0x3e86640 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:3697
    #30 0x35a505c in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2231
    #31 0x35a505c in base::TestSuite::Run() base/test/test_suite.cc:227
    #32 0x2118154 in ChromeBrowserTestSuiteRunner::RunTestSuite(int, char**) chrome/test/base/browser_tests_main.cc:14
    #33 0xce70779 in content::LaunchTests(content::TestLauncherDelegate*, int, int, char**) content/public/test/test_launcher.cc:468
    #34 0x33ff4bb in LaunchChromeTests(int, ChromeTestSuiteRunner*, int, char**) chrome/test/base/chrome_test_launcher.cc:129
    #35 0x2118048 in main chrome/test/base/browser_tests_main.cc:21
    #36 0x7f2add6cd76c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

Original issue's description:
> Fix test CastStreamingApiTestWithPixelOutput.EndToEnd.
> Update end_to_end_sender.js to use standard WebAudio API.
> Modify unit test to use a larger window size (previous window size was
> too small, which negatively impacted the median color computation
> routine.)
> 
> BUG=349599
> [email protected]
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=279210

[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=349599

Review URL: https://codereview.chromium.org/351683003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279231 0039d316-1c4b-4281-b951-d872f2087c98
rakuco pushed a commit that referenced this pull request Aug 11, 2014
…https://codereview.chromium.org/373153002/)

Reason for revert:
Causing memory leaks:

Indirect leak of 328 byte(s) in 1 object(s) allocated from:
    #0 0x515feb in operator new(unsigned long) /usr/local/google/work/chromium/src/third_party/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:55
    #1 0xc334cfb in BookmarkModel::CreatePermanentNode(BookmarkNode::Type) components/bookmarks/browser/bookmark_model.cc:910
    #2 0xc326772 in BookmarkModel::CreateLoadDetails(std::string const&) components/bookmarks/browser/bookmark_model.cc:1006
    #3 0xc326553 in BookmarkModel::Load(PrefService*, std::string const&, base::FilePath const&, scoped_refptr\u003Cbase::SequencedTaskRunner> const&, scoped_refptr\u003Cbase::SequencedTaskRunner> const&) components/bookmarks/browser/bookmark_model.cc:151
    #4 0x232a4d7 in BookmarkModelFactory::BuildServiceInstanceFor(content::BrowserContext*) const chrome/browser/bookmarks/bookmark_model_factory.cc:65
    #5 0xc0bc85f in BrowserContextKeyedServiceFactory::GetServiceForBrowserContext(content::BrowserContext*, bool) components/keyed_service/content/browser_context_keyed_service_factory.cc:91
    #6 0x260de89 in ProfileImpl::DoFinalInit() chrome/browser/profiles/profile_impl.cc:657
    #7 0x260beb5 in ProfileImpl::OnPrefsLoaded(bool) chrome/browser/profiles/profile_impl.cc:881
    #8 0x260ae3e in ProfileImpl::ProfileImpl(base::FilePath const&, Profile::Delegate*, Profile::CreateMode, base::SequencedTaskRunner*) chrome/browser/profiles/profile_impl.cc:492
    #9 0x2608ddb in Profile::CreateProfile(base::FilePath const&, Profile::Delegate*, Profile::CreateMode) chrome/browser/profiles/profile_impl.cc:297
    #10 0x18de165 in ProfileBrowserTest::CreateProfile(base::FilePath const&, Profile::Delegate*, Profile::CreateMode) chrome/browser/profiles/profile_browsertest.cc:106
    #11 0x18df8f8 in ProfileBrowserTest_CreateOldProfileSynchronous_Test::RunTestOnMainThread() chrome/browser/profiles/profile_browsertest.cc:153
    #12 0x33f5434 in InProcessBrowserTest::RunTestOnMainThreadLoop() chrome/test/base/in_process_browser_test.cc:427
    #13 0x2e6ca18 in Run base/callback.h:401
    #14 0x2e6ca18 in ChromeBrowserMainParts::PreMainMessageLoopRunImpl() chrome/browser/chrome_browser_main.cc:1548
    #15 0x2e69666 in ChromeBrowserMainParts::PreMainMessageLoopRun() chrome/browser/chrome_browser_main.cc:975
    #16 0x59381d7 in content::BrowserMainLoop::PreMainMessageLoopRun() content/browser/browser_main_loop.cc:694
    #17 0x5c8f207 in Run base/callback.h:401
    #18 0x5c8f207 in content::StartupTaskRunner::RunAllTasksNow() content/browser/startup_task_runner.cc:45
    #19 0x59342e0 in content::BrowserMainLoop::CreateStartupTasks() content/browser/browser_main_loop.cc:594
    #20 0x5e8a153 in content::BrowserMainRunnerImpl::Initialize(content::MainFunctionParams const&) content/browser/browser_main_runner.cc:106
    #21 0xf0228a0 in content::BrowserMain(content::MainFunctionParams const&) content/browser/browser_main.cc:22
    #22 0xef6cbc4 in content::ContentMainRunnerImpl::Run() content/app/content_main_runner.cc:763
    #23 0xef69d3f in content::ContentMain(content::ContentMainParams const&) content/app/content_main.cc:19
    #24 0xcf5b57f in content::BrowserTestBase::SetUp() content/public/test/browser_test_base.cc:253
    #25 0x33f2563 in InProcessBrowserTest::SetUp() chrome/test/base/in_process_browser_test.cc:210
    #26 0x3e81941 in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2045
    #27 0x3e81941 in testing::Test::Run() testing/gtest/src/gtest.cc:2057
    #28 0x3e83cd9 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2237
    #29 0x3e84a66 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2344
    #30 0x3e97b7a in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4065
    #31 0x3e971b0 in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2045
    #32 0x3e971b0 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:3697
    #33 0x359557c in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2231
    #34 0x359557c in base::TestSuite::Run() base/test/test_suite.cc:227



Original issue's description:
> Fixed use-after-free in LoadCallback in bookmark_storage.cc
> 
> BookmarkStorage isn't ref counted anymore since
> https://codereview.chromium.org/370323002, and the LoadCallback() task
> now gets a WeakPtr to the owning BookmarkStorage. However, it gets a
> raw pointer to the BookmarkLoadDetails object, which is still owned
> by BookmarkStorage and may have been destroyed when the background
> task runs.
> 
> This happened on iOS tests after a recent merge.
> 
> [email protected]
> BUG=165760
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=281830

[email protected],[email protected],[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=165760

Review URL: https://codereview.chromium.org/379643002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281843 0039d316-1c4b-4281-b951-d872f2087c98
rakuco pushed a commit that referenced this pull request Aug 11, 2014
…ttps://codereview.chromium.org/386043002/)

Reason for revert:
Causes LSAN errors on CrOS: http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%283%29/builds/2299/steps/ui_unittests/logs/DoubleTap

e.g.
Indirect leak of 2904 byte(s) in 1 object(s) allocated from:
    #0 0x44edfb in operator new(unsigned long) /usr/local/google/work/chromium/src/third_party/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:55
    #1 0x9fbc47 in CreateStrategy ui/events/gesture_detection/velocity_tracker.cc:201
    #2 0x9fbc47 in ui::VelocityTracker::VelocityTracker(ui::VelocityTracker::Strategy) ui/events/gesture_detection/velocity_tracker.cc:234
    #3 0x9fa322 in ui::VelocityTrackerState::VelocityTrackerState() ui/events/gesture_detection/velocity_tracker_state.cc:18
    #4 0x9f5d64 in ui::GestureDetector::GestureDetector(ui::GestureDetector::Config const&, ui::GestureDetector::GestureListener*, ui::GestureDetector::DoubleTapListener*) ui/events/gesture_detection/gesture_detector.cc:191
    #5 0x1431ddf in GestureListenerImpl ui/events/gesture_detection/gesture_provider.cc:296
    #6 0x1431ddf in ui::GestureProvider::InitGestureDetectors(ui::GestureProvider::Config const&) ui/events/gesture_detection/gesture_provider.cc:629
    #7 0x142ebdd in ui::FilteredGestureProvider::FilteredGestureProvider(ui::GestureProvider::Config const&, ui::GestureProviderClient*) ui/events/gesture_detection/filtered_gesture_provider.cc:19
    #8 0x1410cd3 in ui::GestureProviderAura::GestureProviderAura(ui::GestureProviderAuraClient*) ui/events/gestures/gesture_provider_aura.cc:19
    #9 0x9e9bcd in ui::TouchExplorationController::TouchExplorationController(aura::Window*) ui/chromeos/touch_exploration_controller.cc:33
    #10 0x663c47 in ui::TouchExplorationTest::SwitchTouchExplorationMode(bool) ui/chromeos/touch_exploration_controller_unittest.cc:214
    #11 0x674532 in ui::TouchExplorationTest_DoubleTap_Test::TestBody() ui/chromeos/touch_exploration_controller_unittest.cc:563
    #12 0x7babc8 in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2045
    #13 0x7babc8 in testing::Test::Run() testing/gtest/src/gtest.cc:2061
    #14 0x7bcd08 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2237
    #15 0x7bda46 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2344
    #16 0x7d072a in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4065
    #17 0x7cfd70 in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2045
    #18 0x7cfd70 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:3697
    #19 0x7770b4 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2231
    #20 0x7770b4 in base::TestSuite::Run() base/test/test_suite.cc:227
    #21 0x76e0c1 in Run base/callback.h:401
    #22 0x76e0c1 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::Callback\u003Cint ()> const&, int, bool, base::Callback\u003Cvoid ()> const&) base/test/launcher/unit_test_launcher.cc:498
    #23 0x76da5c in base::LaunchUnitTests(int, char**, base::Callback\u003Cint ()> const&) base/test/launcher/unit_test_launcher.cc:553
    #24 0x59cfaa in main ui/base/test/run_all_unittests.cc:105
    #25 0x7fd0195b276c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

Original issue's description:
> Added wrapper test class for touch_exploration_controller.
> Wrapper class is currently located in the unittest file and is a friend of touch_exploration_controller.
> 
> BUG=377040
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=283741

[email protected],[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=377040

Review URL: https://codereview.chromium.org/404473006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283811 0039d316-1c4b-4281-b951-d872f2087c98
rakuco pushed a commit that referenced this pull request Aug 11, 2014
…://codereview.chromium.org/397233002/)

Reason for revert:
Broke 'Chromium Linux on Linux Clang (dbg)':

http://build.chromium.org/p/chromium.linux/builders/Linux%20Clang%20%28dbg%29/builds/62289

AutofillDialogControllerTest.BillingVsShippingStreetAddress (run #1):
[ RUN      ] AutofillDialogControllerTest.BillingVsShippingStreetAddress
Xlib:  extension "RANDR" missing on display ":9".
/usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/debug/safe_iterator.h:289:
    error: attempt to advance a past-the-end iterator 1 steps, which falls
    outside its valid range.

Objects involved in the operation:
iterator @ 0x0x7fffd4a50d58 {
  state = past-the-end;
  references sequence @ 0x0x7fffd4a50d58
}
Received signal 6
#0 0x7f716d77ecee base::debug::StackTrace::StackTrace()
#1 0x7f716d77e827 base::debug::(anonymous namespace)::StackDumpSignalHandler()
#2 0x7f7168451cb0 \u003Cunknown>
#3 0x7f71648cf425 gsignal
#4 0x7f71648d2b8b abort
#5 0x7f7164ed25ad __gnu_debug::_Error_formatter::_M_error()
#6 0x0000026271ce __gnu_debug::_Safe_iterator\u003C>::operator+=()
#7 0x0000026270f5 __gnu_debug::_Safe_iterator\u003C>::operator+()
#8 0x000005506f39 i18n::addressinput::(anonymous namespace)::CombineLinesForLanguage()
#9 0x00000550703a i18n::addressinput::GetStreetAddressLinesAsSingleLine()
#10 0x000005548289 autofill::(anonymous namespace)::FillStreetAddress()
#11 0x000005547b91 autofill::AutofillField::FillFormField()
#12 0x0000055907cd autofill::FormStructure::FillFields()
#13 0x000004972335 autofill::AutofillDialogControllerImpl::Show()
#14 0x000001ec0f2a autofill::(anonymous namespace)::AutofillDialogControllerTest::SetUpControllerWithFormData()
#15 0x000001ece5c8 autofill::AutofillDialogControllerTest_BillingVsShippingStreetAddress_Test::TestBody()
#16 0x0000039fcc63 testing::internal::HandleSehExceptionsInMethodIfSupported\u003C>()
#17 0x0000039e9f9e testing::internal::HandleExceptionsInMethodIfSupported\u003C>()
#18 0x0000039e0e05 testing::Test::Run()
#19 0x0000039e151b testing::TestInfo::Run()
#20 0x0000039e1b0a testing::TestCase::Run()
#21 0x0000039e6118 testing::internal::UnitTestImpl::RunAllTests()
#22 0x0000039f5973 testing::internal::HandleSehExceptionsInMethodIfSupported\u003C>()
#23 0x0000039ebb0e testing::internal::HandleExceptionsInMethodIfSupported\u003C>()
#24 0x0000039e5da4 testing::UnitTest::Run()
#25 0x000005b72b61 RUN_ALL_TESTS()
#26 0x000005b71be7 base::TestSuite::Run()
#27 0x000005b8455d content::UnitTestTestSuite::Run()
#28 0x000003aa40d2 base::internal::RunnableAdapter\u003C>::Run()
#29 0x000003aa403c base::internal::InvokeHelper\u003C>::MakeItSo()
#30 0x000003aa3fea base::internal::Invoker\u003C>::Run()
#31 0x000002a29e9e base::Callback\u003C>::Run()
#32 0x000005b6b830 base::(anonymous namespace)::LaunchUnitTestsInternal()
#33 0x000005b6b527 base::LaunchUnitTests()
#34 0x000003aa3db5 main
#35 0x7f71648ba76d __libc_start_main
#36 0x000000715159 \u003Cunknown>
  r8: 00007f715f665980  r9: 00007fffd4a507d8 r10: 0000000000000008 r11: 0000000000000206
 r12: 00007fffd4a50aa0 r13: 0000000000000001 r14: 0000000000000001 r15: 0000000000000000
  di: 00000000000016b2  si: 00000000000016b2  bp: 0000000000000002  bx: 00007fffd4a50a30
  dx: 0000000000000006  ax: 0000000000000000  cx: ffffffffffffffff  sp: 00007fffd4a50828
  ip: 00007f71648cf425 efl: 0000000000000206 cgf: 0000000000000033 erf: 0000000000000000
 trp: 0000000000000000 msk: 0000000000000000 cr2: 0000000000000000


Original issue's description:
> Use language-specific street address line separators
> 
> Fill street address into a single-line text input field with separators
> that depend on the language code of the profile.
> 
> TEST=components_unittests:AutofillFieldTest.FillStreetAddress*
> BUG=270261
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284335

[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=270261

Review URL: https://codereview.chromium.org/401103004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284340 0039d316-1c4b-4281-b951-d872f2087c98
darktears pushed a commit that referenced this pull request Aug 21, 2014
…um.org/420603011/)

Reason for revert:
Appears to have broken Linux Chromium OS ASan LSan Tests (3) (stats):
http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%283%29

athena_unittests failed 13
Basic
Accelerators
AppSelection
BasicTransition
VirtualKeyboardTransition
Basic
CreateContainer
GrabAndMouseCapture
GrabInputContainer
GrabShouldNotBlockVirtualKeyboard
NonActivatableContainer
Zorder
Empty

[ RUN      ] ActivityManagerTest.Basic
=================================================================
==8808==ERROR: AddressSanitizer: heap-use-after-free on address 0x60200000ad70 at pc 0x00000054eebe bp 0x7fff975aaa70 sp 0x7fff975aaa68
READ of size 8 at 0x60200000ad70 thread T0
#0 0x54eebd in __find<__gnu_cxx::__normal_iterator<athena::WindowManagerObserver **, std::vector<athena::WindowManagerObserver *, std::allocator<athena::WindowManagerObserver *> > >, athena::WindowManagerObserver *> /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/bits/stl_algo.h:190:8
#1 0x54eebd in find<__gnu_cxx::__normal_iterator<athena::WindowManagerObserver **, std::vector<athena::WindowManagerObserver *, std::allocator<athena::WindowManagerObserver *> > >, athena::WindowManagerObserver *> /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/bits/stl_algo.h:4402
#2 0x54eebd in ObserverListBase<athena::WindowManagerObserver>::RemoveObserver(athena::WindowManagerObserver*) base/observer_list.h:168
#3 0x560f79 in ~SplitViewController athena/wm/split_view_controller.cc:63:3
#4 0x560f79 in athena::SplitViewController::~SplitViewController() athena/wm/split_view_controller.cc:62
#5 0x54d340 in operator() base/memory/scoped_ptr.h:137:5
#6 0x54d340 in ~scoped_ptr_impl base/memory/scoped_ptr.h:220
#7 0x54d340 in ~scoped_ptr base/memory/scoped_ptr.h:432
#8 0x54d340 in athena::(anonymous namespace)::WindowManagerImpl::~WindowManagerImpl() athena/wm/window_manager_impl.cc:124
#9 0x54d57d in athena::(anonymous namespace)::WindowManagerImpl::~WindowManagerImpl() athena/wm/window_manager_impl.cc:116:41
#10 0x567127 in athena::ShutdownAthena() athena/main/athena_launcher.cc:86:3
#11 0x5657af in athena::test::AthenaTestHelper::TearDown() athena/test/athena_test_helper.cc:98:3
#12 0x5645b6 in athena::test::AthenaTestBase::TearDown() athena/test/athena_test_base.cc:51:3
#13 0x4fc938 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2610:5
#14 0x4fd676 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2728:5
#15 0x5138f5 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4591:11
#16 0x512ee6 in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2418:12
#17 0x512ee6 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4209
#18 0x1602b74 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2304:10
#19 0x1602b74 in base::TestSuite::Run() base/test/test_suite.cc:227
#20 0x15fa231 in Run base/callback.h:401:12
#21 0x15fa231 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::Callback<int ()> const&, int, bool, base::Callback<void ()> const&) base/test/launcher/unit_test_launcher.cc:498
#22 0x15faf7e in base::LaunchUnitTestsSerially(int, char**, base::Callback<int ()> const&) base/test/launcher/unit_test_launcher.cc:564:10
#23 0x4d74ca in main athena/test/athena_unittests.cc:51:10
#24 0x7f9020d2d76c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226
#25 0x4d726c in _start (/mnt/data/b/build/slave/Linux_Chromium_OS_ASan_LSan_Tests__3_/build/src/out/Release/athena_unittests+0x4d726c)

Please follow instructions on how to setup ASan on your workstation here: http://www.chromium.org/developers/testing/addresssanitizer
LSan may also be of use: http://www.chromium.org/developers/testing/leaksanitizer
You may also want run your patch through the linux_lsan try bot.

Original issue's description:
> Split Screen mode implementation.
> 
> Implements Split Screen mode and window cycling behavior.
> 
> BUG=383421
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288593

TBR=oshima,sadrul,mfomitchev,mukai
NOTRY=true

Review URL: https://codereview.chromium.org/451363002

Cr-Commit-Position: refs/heads/master@{#288607}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288607 0039d316-1c4b-4281-b951-d872f2087c98
darktears pushed a commit that referenced this pull request Aug 22, 2014
…um.org/420603011/)

Reason for revert:
Appears to have broken Linux Chromium OS ASan LSan Tests (3) (stats):
http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%283%29

athena_unittests failed 13
Basic
Accelerators
AppSelection
BasicTransition
VirtualKeyboardTransition
Basic
CreateContainer
GrabAndMouseCapture
GrabInputContainer
GrabShouldNotBlockVirtualKeyboard
NonActivatableContainer
Zorder
Empty

[ RUN      ] ActivityManagerTest.Basic
=================================================================
==8808==ERROR: AddressSanitizer: heap-use-after-free on address 0x60200000ad70 at pc 0x00000054eebe bp 0x7fff975aaa70 sp 0x7fff975aaa68
READ of size 8 at 0x60200000ad70 thread T0
#0 0x54eebd in __find<__gnu_cxx::__normal_iterator<athena::WindowManagerObserver **, std::vector<athena::WindowManagerObserver *, std::allocator<athena::WindowManagerObserver *> > >, athena::WindowManagerObserver *> /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/bits/stl_algo.h:190:8
#1 0x54eebd in find<__gnu_cxx::__normal_iterator<athena::WindowManagerObserver **, std::vector<athena::WindowManagerObserver *, std::allocator<athena::WindowManagerObserver *> > >, athena::WindowManagerObserver *> /usr/lib/gcc/x86_64-linux-gnu/4.6/../../../../include/c++/4.6/bits/stl_algo.h:4402
#2 0x54eebd in ObserverListBase<athena::WindowManagerObserver>::RemoveObserver(athena::WindowManagerObserver*) base/observer_list.h:168
#3 0x560f79 in ~SplitViewController athena/wm/split_view_controller.cc:63:3
#4 0x560f79 in athena::SplitViewController::~SplitViewController() athena/wm/split_view_controller.cc:62
#5 0x54d340 in operator() base/memory/scoped_ptr.h:137:5
#6 0x54d340 in ~scoped_ptr_impl base/memory/scoped_ptr.h:220
#7 0x54d340 in ~scoped_ptr base/memory/scoped_ptr.h:432
#8 0x54d340 in athena::(anonymous namespace)::WindowManagerImpl::~WindowManagerImpl() athena/wm/window_manager_impl.cc:124
#9 0x54d57d in athena::(anonymous namespace)::WindowManagerImpl::~WindowManagerImpl() athena/wm/window_manager_impl.cc:116:41
#10 0x567127 in athena::ShutdownAthena() athena/main/athena_launcher.cc:86:3
#11 0x5657af in athena::test::AthenaTestHelper::TearDown() athena/test/athena_test_helper.cc:98:3
#12 0x5645b6 in athena::test::AthenaTestBase::TearDown() athena/test/athena_test_base.cc:51:3
#13 0x4fc938 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2610:5
#14 0x4fd676 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2728:5
#15 0x5138f5 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4591:11
#16 0x512ee6 in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2418:12
#17 0x512ee6 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4209
#18 0x1602b74 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2304:10
#19 0x1602b74 in base::TestSuite::Run() base/test/test_suite.cc:227
#20 0x15fa231 in Run base/callback.h:401:12
#21 0x15fa231 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::Callback<int ()> const&, int, bool, base::Callback<void ()> const&) base/test/launcher/unit_test_launcher.cc:498
#22 0x15faf7e in base::LaunchUnitTestsSerially(int, char**, base::Callback<int ()> const&) base/test/launcher/unit_test_launcher.cc:564:10
#23 0x4d74ca in main athena/test/athena_unittests.cc:51:10
#24 0x7f9020d2d76c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226
#25 0x4d726c in _start (/mnt/data/b/build/slave/Linux_Chromium_OS_ASan_LSan_Tests__3_/build/src/out/Release/athena_unittests+0x4d726c)

Please follow instructions on how to setup ASan on your workstation here: http://www.chromium.org/developers/testing/addresssanitizer
LSan may also be of use: http://www.chromium.org/developers/testing/leaksanitizer
You may also want run your patch through the linux_lsan try bot.

Original issue's description:
> Split Screen mode implementation.
> 
> Implements Split Screen mode and window cycling behavior.
> 
> BUG=383421
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288593

TBR=oshima,sadrul,mfomitchev,mukai
NOTRY=true

Review URL: https://codereview.chromium.org/451363002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288607 0039d316-1c4b-4281-b951-d872f2087c98
darktears pushed a commit that referenced this pull request Sep 29, 2014
Broke ASan/LSan:

http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%282%29/builds/6376

@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@Direct leak of 136 byte(s) in 1 object(s) allocated from:@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #0 0x4f295b in operator new(unsigned long) (/b/build/slave/Linux_ASan_LSan_Tests__2_/build/src/out/Release/browser_tests+0x4f295b)@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #1 0x4f41e33 in GrGpuGL::onCreateIndexBuffer(unsigned long, bool) third_party/skia/src/gpu/gl/GrGpuGL.cpp:1374:13@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #2 0x4ec41b3 in createIndexBuffer third_party/skia/src/gpu/GrGpu.cpp:159:12@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #3 0x4ec41b3 in GrGpu::getQuadIndexBuffer() const third_party/skia/src/gpu/GrGpu.cpp:273@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #4 0x4ecbad0 in GrInOrderDrawBuffer::onDrawRect(SkRect const&, SkRect const*, SkMatrix const*) third_party/skia/src/gpu/GrInOrderDrawBuffer.cpp:174:34@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #5 0x4e998f8 in drawRect third_party/skia/src/gpu/GrDrawTarget.h:384:9@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #6 0x4e998f8 in drawSimpleRect third_party/skia/src/gpu/GrDrawTarget.h:391@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #7 0x4e998f8 in GrContext::drawRect(GrPaint const&, SkRect const&, GrStrokeInfo const*) third_party/skia/src/gpu/GrContext.cpp:904@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #8 0xf895e0c in SkAlphaThresholdFilterImpl::asNewEffect(GrEffect**, GrTexture*, SkMatrix const&, SkIRect const&) const third_party/skia/src/effects/SkAlphaThresholdFilter.cpp:287:17@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #9 0x4d531c9 in SkImageFilter::filterImageGPU(SkImageFilter::Proxy*, SkBitmap const&, SkImageFilter::Context const&, SkBitmap*, SkIPoint*) const third_party/skia/src/core/SkImageFilter.cpp:272:5@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #10 0xf8b4aad in filter_texture(SkBaseDevice*, GrContext*, GrTexture*, SkImageFilter const*, int, int, SkImageFilter::Context const&, SkBitmap*, SkIPoint*) third_party/skia/src/gpu/SkGpuDevice.cpp:1398:16@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #11 0xf8b3d42 in SkGpuDevice::drawSprite(SkDraw const&, SkBitmap const&, int, int, SkPaint const&) third_party/skia/src/gpu/SkGpuDevice.cpp:1434:13@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #12 0x4d16942 in SkCanvas::drawSprite(SkBitmap const&, int, int, SkPaint const*) third_party/skia/src/core/SkCanvas.cpp:1194:13@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #13 0xe2828c3 in cc::ApplyImageFilter(scoped_ptr\u003Ccc::GLRenderer::ScopedUseGrContext, base::DefaultDeleter\u003Ccc::GLRenderer::ScopedUseGrContext> >, cc::ResourceProvider*, gfx::Point const&, gfx::Vector2dF const&, SkImageFilter*, cc::ScopedResource*) cc/output/gl_renderer.cc:722:3@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #14 0xe275474 in cc::GLRenderer::DrawRenderPassQuad(cc::DirectRenderer::DrawingFrame*, cc::RenderPassDrawQuad const*) cc/output/gl_renderer.cc:1051:13@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #15 0xe4598a4 in cc::DirectRenderer::DrawRenderPass(cc::DirectRenderer::DrawingFrame*, cc::RenderPass const*) cc/output/direct_renderer.cc:385:7@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #16 0xe458480 in cc::DirectRenderer::DrawFrame(cc::ScopedPtrVector\u003Ccc::RenderPass>*, float, gfx::Rect const&, gfx::Rect const&, bool) cc/output/direct_renderer.cc:232:5@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #17 0xe3bd5cc in cc::LayerTreeHostImpl::DrawLayers(cc::LayerTreeHostImpl::FrameData*, base::TimeTicks) cc/trees/layer_tree_host_impl.cc:1578:5@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #18 0xe3fb78e in cc::SingleThreadProxy::DoComposite(base::TimeTicks, cc::LayerTreeHostImpl::FrameData*) cc/trees/single_thread_proxy.cc:438:7@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #19 0xe3fade5 in cc::SingleThreadProxy::CompositeImmediately(base::TimeTicks) cc/trees/single_thread_proxy.cc:356:7@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #20 0x651589b in ui::Compositor::Draw() ui/compositor/compositor.cc:234:5@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #21 0x345c43f in Run base/callback.h:401:12@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #22 0x345c43f in base::debug::TaskAnnotator::RunTask(char const*, char const*, base::PendingTask const&) base/debug/task_annotator.cc:62@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #23 0x339b64c in base::MessageLoop::RunTask(base::PendingTask const&) base/message_loop/message_loop.cc:435:3@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #24 0x339c710 in DeferOrRunPendingTask base/message_loop/message_loop.cc:445:5@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #25 0x339c710 in base::MessageLoop::DoWork() base/message_loop/message_loop.cc:552@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #26 0x3445422 in HandleDispatch base/message_loop/message_pump_glib.cc:267:7@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #27 0x3445422 in base::(anonymous namespace)::WorkSourceDispatch(_GSource*, int (*)(void*), void*) base/message_loop/message_pump_glib.cc:109@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #28 0x7f5f9395dd12 in g_main_dispatch /build/buildd/glib2.0-2.32.4/./glib/gmain.c:2539@@@
@@@STEP_LOG_LINE@FindStayVisibleOnAnchorLoad@    #29 0x7f5f9395dd12 in g_main_context_dispatch /build/buildd/glib2.0-2.32.4/./glib/gmain.c:3075@@@

> Skia roll de21804:94ce9ac
> 
> https://skia.googlesource.com/skia/+log/de218041b99e8f9af0e80b34452b0e8d7f770765..94ce9ac8624dbb45656b8f5c992fad9c9ff3ee5f
> 
> CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_layout_rel
> [email protected]
> BUG=
> 
> Review URL: https://codereview.chromium.org/485173003

[email protected],[email protected]

Review URL: https://codereview.chromium.org/485523003

Cr-Commit-Position: refs/heads/master@{#290648}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290648 0039d316-1c4b-4281-b951-d872f2087c98
darktears pushed a commit that referenced this pull request Sep 29, 2014
…r_tests compiling on MacViews (patchset #7 of https://codereview.chromium.org/486063002/)

Reason for revert:
browser_test fails in Linux ASAN/LSAN test.

WebDialogBrowserTest.SizeWindow (run #1):
[ RUN      ] WebDialogBrowserTest.SizeWindow
[30383:30383:0820/232516:ERROR:browser_main_loop.cc(162)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
Xlib:  extension "RANDR" missing on display ":9".
Xlib:  extension "RANDR" missing on display ":9".
[30383:30383:0820/232516:WARNING:password_store_factory.cc(215)] Using basic (unencrypted) store for password storage. See http://code.google.com/p/chromium/wiki/LinuxPasswordStorage for more information about password storage options.
[30473:30473:0820/232516:ERROR:renderer_main.cc(204)] Running without renderer sandbox
[30496:30496:0820/232516:ERROR:renderer_main.cc(204)] Running without renderer sandbox

=================================================================
==30383==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 128 byte(s) in 1 object(s) allocated from:
    #0 0x4f2bbb in operator new(unsigned long) (/b/build/slave/Linux_ASan_LSan_Tests__3_/build/src/out/Release/browser_tests+0x4f2bbb)
    #1 0x1dace96 in WebDialogBrowserTest_SizeWindow_Test::RunTestOnMainThread() chrome/browser/ui/views/web_dialog_view_browsertest.cc:95:3
    #2 0x32aaae3 in InProcessBrowserTest::RunTestOnMainThreadLoop() chrome/test/base/in_process_browser_test.cc:434:5
    #3 0x2d6de03 in Run base/callback.h:401:12
    #4 0x2d6de03 in ChromeBrowserMainParts::PreMainMessageLoopRunImpl() chrome/browser/chrome_browser_main.cc:1555
    #5 0x2d6a146 in ChromeBrowserMainParts::PreMainMessageLoopRun() chrome/browser/chrome_browser_main.cc:980:18
    #6 0x59d80b7 in content::BrowserMainLoop::PreMainMessageLoopRun() content/browser/browser_main_loop.cc:736:5
    #7 0x5d3ac48 in Run base/callback.h:401:12
    #8 0x5d3ac48 in content::StartupTaskRunner::RunAllTasksNow() content/browser/startup_task_runner.cc:45
    #9 0x59d40a0 in content::BrowserMainLoop::CreateStartupTasks() content/browser/browser_main_loop.cc:636:3
    #10 0x5f3839d in content::BrowserMainRunnerImpl::Initialize(content::MainFunctionParams const&) content/browser/browser_main_runner.cc:106:5
    #11 0xf90f1a5 in content::BrowserMain(content::MainFunctionParams const&) content/browser/browser_main.cc:22:19
    #12 0xf8416c0 in content::ContentMainRunnerImpl::Run() content/app/content_main_runner.cc:764:12
    #13 0xf83db5f in content::ContentMain(content::ContentMainParams const&) content/app/content_main.cc:19:15
    #14 0xd6b57e1 in content::BrowserTestBase::SetUp() content/public/test/browser_test_base.cc:257:3
    #15 0x32a773b in InProcessBrowserTest::SetUp() chrome/test/base/in_process_browser_test.cc:217:3
    #16 0x3de9f81 in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2418:12
    #17 0x3de9f81 in testing::Test::Run() testing/gtest/src/gtest.cc:2430
    #18 0x3dec559 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2610:5
    #19 0x3ded5b6 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2728:5
    #20 0x3e03055 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4591:11
    #21 0x3e02664 in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2418:12
    #22 0x3e02664 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4209
    #23 0x348e02f in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2304:10
    #24 0x348e02f in base::TestSuite::Run() base/test/test_suite.cc:227
    #25 0x20fb8cd in ChromeBrowserTestSuiteRunner::RunTestSuite(int, char**) chrome/test/base/browser_tests_main.cc:14:12
    #26 0xd737de7 in content::LaunchTests(content::TestLauncherDelegate*, int, int, char**) content/public/test/test_launcher.cc:469:12
    #27 0x32a4b8b in LaunchChromeTests(int, ChromeTestSuiteRunner*, int, char**) chrome/test/base/chrome_test_launcher.cc:124:10
    #28 0x20fb7b8 in main chrome/test/base/browser_tests_main.cc:21:10
    #29 0x7feeddc6676c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

-----------------------------------------------------
Suppressions used:
  count      bytes template
   1767      75328 libfontconfig
-----------------------------------------------------

SUMMARY: AddressSanitizer: 128 byte(s) leaked in 1 allocation(s).
[0820/232518:ERROR:nacl_helper_linux.cc(282)] NaCl helper process running without a sandbox!
Most likely you need to configure your SUID sandbox correctly

WebDialogBrowserTest.SizeWindow (run #2):
[ RUN      ] WebDialogBrowserTest.SizeWindow
[16767:16767:0820/232720:ERROR:browser_main_loop.cc(162)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
Xlib:  extension "RANDR" missing on display ":9".
Xlib:  extension "RANDR" missing on display ":9".
[16767:16767:0820/232720:WARNING:password_store_factory.cc(215)] Using basic (unencrypted) store for password storage. See http://code.google.com/p/chromium/wiki/LinuxPasswordStorage for more information about password storage options.
[16871:16871:0820/232720:ERROR:renderer_main.cc(204)] Running without renderer sandbox
[16911:16911:0820/232720:ERROR:renderer_main.cc(204)] Running without renderer sandbox

=================================================================
==16767==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 128 byte(s) in 1 object(s) allocated from:
    #0 0x4f2bbb in operator new(unsigned long) (/b/build/slave/Linux_ASan_LSan_Tests__3_/build/src/out/Release/browser_tests+0x4f2bbb)
    #1 0x1dace96 in WebDialogBrowserTest_SizeWindow_Test::RunTestOnMainThread() chrome/browser/ui/views/web_dialog_view_browsertest.cc:95:3
    #2 0x32aaae3 in InProcessBrowserTest::RunTestOnMainThreadLoop() chrome/test/base/in_process_browser_test.cc:434:5
    #3 0x2d6de03 in Run base/callback.h:401:12
    #4 0x2d6de03 in ChromeBrowserMainParts::PreMainMessageLoopRunImpl() chrome/browser/chrome_browser_main.cc:1555
    #5 0x2d6a146 in ChromeBrowserMainParts::PreMainMessageLoopRun() chrome/browser/chrome_browser_main.cc:980:18
    #6 0x59d80b7 in content::BrowserMainLoop::PreMainMessageLoopRun() content/browser/browser_main_loop.cc:736:5
    #7 0x5d3ac48 in Run base/callback.h:401:12
    #8 0x5d3ac48 in content::StartupTaskRunner::RunAllTasksNow() content/browser/startup_task_runner.cc:45
    #9 0x59d40a0 in content::BrowserMainLoop::CreateStartupTasks() content/browser/browser_main_loop.cc:636:3
    #10 0x5f3839d in content::BrowserMainRunnerImpl::Initialize(content::MainFunctionParams const&) content/browser/browser_main_runner.cc:106:5
    #11 0xf90f1a5 in content::BrowserMain(content::MainFunctionParams const&) content/browser/browser_main.cc:22:19
    #12 0xf8416c0 in content::ContentMainRunnerImpl::Run() content/app/content_main_runner.cc:764:12
    #13 0xf83db5f in content::ContentMain(content::ContentMainParams const&) content/app/content_main.cc:19:15
    #14 0xd6b57e1 in content::BrowserTestBase::SetUp() content/public/test/browser_test_base.cc:257:3
    #15 0x32a773b in InProcessBrowserTest::SetUp() chrome/test/base/in_process_browser_test.cc:217:3
    #16 0x3de9f81 in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2418:12
    #17 0x3de9f81 in testing::Test::Run() testing/gtest/src/gtest.cc:2430
    #18 0x3dec559 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2610:5
    #19 0x3ded5b6 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2728:5
    #20 0x3e03055 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4591:11
    #21 0x3e02664 in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2418:12
    #22 0x3e02664 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4209
    #23 0x348e02f in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2304:10
    #24 0x348e02f in base::TestSuite::Run() base/test/test_suite.cc:227
    #25 0x20fb8cd in ChromeBrowserTestSuiteRunner::RunTestSuite(int, char**) chrome/test/base/browser_tests_main.cc:14:12
    #26 0xd737de7 in content::LaunchTests(content::TestLauncherDelegate*, int, int, char**) content/public/test/test_launcher.cc:469:12
    #27 0x32a4b8b in LaunchChromeTests(int, ChromeTestSuiteRunner*, int, char**) chrome/test/base/chrome_test_launcher.cc:124:10
    #28 0x20fb7b8 in main chrome/test/base/browser_tests_main.cc:21:10
    #29 0x7fb13f28476c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

-----------------------------------------------------
Suppressions used:
  count      bytes template
   1767      75328 libfontconfig
-----------------------------------------------------

SUMMARY: AddressSanitizer: 128 byte(s) leaked in 1 allocation(s).
[0820/232722:ERROR:nacl_helper_linux.cc(282)] NaCl helper process running without a sandbox!
Most likely you need to configure your SUID sandbox correctly

WebDialogBrowserTest.SizeWindow (run #3):
[ RUN      ] WebDialogBrowserTest.SizeWindow
[16994:16994:0820/232723:ERROR:browser_main_loop.cc(162)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
Xlib:  extension "RANDR" missing on display ":9".
Xlib:  extension "RANDR" missing on display ":9".
[16994:16994:0820/232724:WARNING:password_store_factory.cc(215)] Using basic (unencrypted) store for password storage. See http://code.google.com/p/chromium/wiki/LinuxPasswordStorage for more information about password storage options.
[17026:17026:0820/232724:ERROR:renderer_main.cc(204)] Running without renderer sandbox
[17038:17038:0820/232724:ERROR:renderer_main.cc(204)] Running without renderer sandbox

=================================================================
==16994==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 128 byte(s) in 1 object(s) allocated from:
    #0 0x4f2bbb in operator new(unsigned long) (/b/build/slave/Linux_ASan_LSan_Tests__3_/build/src/out/Release/browser_tests+0x4f2bbb)
    #1 0x1dace96 in WebDialogBrowserTest_SizeWindow_Test::RunTestOnMainThread() chrome/browser/ui/views/web_dialog_view_browsertest.cc:95:3
    #2 0x32aaae3 in InProcessBrowserTest::RunTestOnMainThreadLoop() chrome/test/base/in_process_browser_test.cc:434:5
    #3 0x2d6de03 in Run base/callback.h:401:12
    #4 0x2d6de03 in ChromeBrowserMainParts::PreMainMessageLoopRunImpl() chrome/browser/chrome_browser_main.cc:1555
    #5 0x2d6a146 in ChromeBrowserMainParts::PreMainMessageLoopRun() chrome/browser/chrome_browser_main.cc:980:18
    #6 0x59d80b7 in content::BrowserMainLoop::PreMainMessageLoopRun() content/browser/browser_main_loop.cc:736:5
    #7 0x5d3ac48 in Run base/callback.h:401:12
    #8 0x5d3ac48 in content::StartupTaskRunner::RunAllTasksNow() content/browser/startup_task_runner.cc:45
    #9 0x59d40a0 in content::BrowserMainLoop::CreateStartupTasks() content/browser/browser_main_loop.cc:636:3
    #10 0x5f3839d in content::BrowserMainRunnerImpl::Initialize(content::MainFunctionParams const&) content/browser/browser_main_runner.cc:106:5
    #11 0xf90f1a5 in content::BrowserMain(content::MainFunctionParams const&) content/browser/browser_main.cc:22:19
    #12 0xf8416c0 in content::ContentMainRunnerImpl::Run() content/app/content_main_runner.cc:764:12
    #13 0xf83db5f in content::ContentMain(content::ContentMainParams const&) content/app/content_main.cc:19:15
    #14 0xd6b57e1 in content::BrowserTestBase::SetUp() content/public/test/browser_test_base.cc:257:3
    #15 0x32a773b in InProcessBrowserTest::SetUp() chrome/test/base/in_process_browser_test.cc:217:3
    #16 0x3de9f81 in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2418:12
    #17 0x3de9f81 in testing::Test::Run() testing/gtest/src/gtest.cc:2430
    #18 0x3dec559 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2610:5
    #19 0x3ded5b6 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2728:5
    #20 0x3e03055 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4591:11
    #21 0x3e02664 in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2418:12
    #22 0x3e02664 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4209
    #23 0x348e02f in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2304:10
    #24 0x348e02f in base::TestSuite::Run() base/test/test_suite.cc:227
    #25 0x20fb8cd in ChromeBrowserTestSuiteRunner::RunTestSuite(int, char**) chrome/test/base/browser_tests_main.cc:14:12
    #26 0xd737de7 in content::LaunchTests(content::TestLauncherDelegate*, int, int, char**) content/public/test/test_launcher.cc:469:12
    #27 0x32a4b8b in LaunchChromeTests(int, ChromeTestSuiteRunner*, int, char**) chrome/test/base/chrome_test_launcher.cc:124:10
    #28 0x20fb7b8 in main chrome/test/base/browser_tests_main.cc:21:10
    #29 0x7f0688ff376c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

-----------------------------------------------------
Suppressions used:
  count      bytes template
   1767      75328 libfontconfig
-----------------------------------------------------

SUMMARY: AddressSanitizer: 128 byte(s) leaked in 1 allocation(s).
[0820/232725:ERROR:nacl_helper_linux.cc(282)] NaCl helper process running without a sandbox!
Most likely you need to configure your SUID sandbox correctly

WebDialogBrowserTest.SizeWindow (run #4):
[ RUN      ] WebDialogBrowserTest.SizeWindow
[17073:17073:0820/232726:ERROR:browser_main_loop.cc(162)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
Xlib:  extension "RANDR" missing on display ":9".
Xlib:  extension "RANDR" missing on display ":9".
[17073:17073:0820/232726:WARNING:password_store_factory.cc(215)] Using basic (unencrypted) store for password storage. See http://code.google.com/p/chromium/wiki/LinuxPasswordStorage for more information about password storage options.
[17105:17105:0820/232726:ERROR:renderer_main.cc(204)] Running without renderer sandbox
[17116:17116:0820/232726:ERROR:renderer_main.cc(204)] Running without renderer sandbox

=================================================================
==17073==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 128 byte(s) in 1 object(s) allocated from:
    #0 0x4f2bbb in operator new(unsigned long) (/b/build/slave/Linux_ASan_LSan_Tests__3_/build/src/out/Release/browser_tests+0x4f2bbb)
    #1 0x1dace96 in WebDialogBrowserTest_SizeWindow_Test::RunTestOnMainThread() chrome/browser/ui/views/web_dialog_view_browsertest.cc:95:3
    #2 0x32aaae3 in InProcessBrowserTest::RunTestOnMainThreadLoop() chrome/test/base/in_process_browser_test.cc:434:5
    #3 0x2d6de03 in Run base/callback.h:401:12
    #4 0x2d6de03 in ChromeBrowserMainParts::PreMainMessageLoopRunImpl() chrome/browser/chrome_browser_main.cc:1555
    #5 0x2d6a146 in ChromeBrowserMainParts::PreMainMessageLoopRun() chrome/browser/chrome_browser_main.cc:980:18
    #6 0x59d80b7 in content::BrowserMainLoop::PreMainMessageLoopRun() content/browser/browser_main_loop.cc:736:5
    #7 0x5d3ac48 in Run base/callback.h:401:12
    #8 0x5d3ac48 in content::StartupTaskRunner::RunAllTasksNow() content/browser/startup_task_runner.cc:45
    #9 0x59d40a0 in content::BrowserMainLoop::CreateStartupTasks() content/browser/browser_main_loop.cc:636:3
    #10 0x5f3839d in content::BrowserMainRunnerImpl::Initialize(content::MainFunctionParams const&) content/browser/browser_main_runner.cc:106:5
    #11 0xf90f1a5 in content::BrowserMain(content::MainFunctionParams const&) content/browser/browser_main.cc:22:19
    #12 0xf8416c0 in content::ContentMainRunnerImpl::Run() content/app/content_main_runner.cc:764:12
    #13 0xf83db5f in content::ContentMain(content::ContentMainParams const&) content/app/content_main.cc:19:15
    #14 0xd6b57e1 in content::BrowserTestBase::SetUp() content/public/test/browser_test_base.cc:257:3
    #15 0x32a773b in InProcessBrowserTest::SetUp() chrome/test/base/in_process_browser_test.cc:217:3
    #16 0x3de9f81 in HandleExceptionsInMethodIfSupported\u003Ctesting::Test, void> testing/gtest/src/gtest.cc:2418:12
    #17 0x3de9f81 in testing::Test::Run() testing/gtest/src/gtest.cc:2430
    #18 0x3dec559 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2610:5
    #19 0x3ded5b6 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2728:5
    #20 0x3e03055 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4591:11
    #21 0x3e02664 in HandleExceptionsInMethodIfSupported\u003Ctesting::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2418:12
    #22 0x3e02664 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4209
    #23 0x348e02f in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2304:10
    #24 0x348e02f in base::TestSuite::Run() base/test/test_suite.cc:227
    #25 0x20fb8cd in ChromeBrowserTestSuiteRunner::RunTestSuite(int, char**) chrome/test/base/browser_tests_main.cc:14:12
    #26 0xd737de7 in content::LaunchTests(content::TestLauncherDelegate*, int, int, char**) content/public/test/test_launcher.cc:469:12
    #27 0x32a4b8b in LaunchChromeTests(int, ChromeTestSuiteRunner*, int, char**) chrome/test/base/chrome_test_launcher.cc:124:10
    #28 0x20fb7b8 in main chrome/test/base/browser_tests_main.cc:21:10
    #29 0x7f5f8410c76c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

-----------------------------------------------------
Suppressions used:
  count      bytes template
   1767      75328 libfontconfig
-----------------------------------------------------

SUMMARY: AddressSanitizer: 128 byte(s) leaked in 1 allocation(s).
[0820/232728:ERROR:nacl_helper_linux.cc(282)] NaCl helper process running without a sandbox!
Most likely you need to configure your SUID sandbox correctly

Original issue's description:
> MacViews: Fix WebDialogBrowserTest.SizeWindow to get browser_tests compiling on MacViews
> 
> gyp changes are made to filter out toolkit-views browser_tests that
> aren't yet compiled in to a toolkit-views Chrome binary on Mac.
> 
> One, WebDialogBrowserTest.SizeWindow, was a disabled test (disabled for
> 4 years :o). It was preventing browser_tests compiling on MacViews
> because it was passing a gfx::NativeWindow to CreateWindowWithParent,
> which takes a view.
> 
> The parent isn't important for the test - it's just adding widget
> context. This CL changes the parent to be web_contents->GetNativeView()
> instead so the test compiles.
> 
> To ensure nothing breaks, the test needs to be enabled. According to the
> comment, the reasons for it being disabled on Windows still seem
> relevant. However, on Linux the test was just timing out due to a quirk
> of the test. So the quirk is fixed, and WebDialogBrowserTest.SizeWindow
> is enabled on Linux Aura.
> 
> The test is also enabled on Mac, but currently fails because
> NativeViewHost isn't implemented yet. That's coming, and leaving the
> test enabled will ensure we check it.
> 
> With this change browser_tests compiles and links on toolkit-views Mac.
> 
> BUG=404979, 399191, 52602
> TEST=browser_tests
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290998

[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=404979, 399191, 52602

Review URL: https://codereview.chromium.org/490123002

Cr-Commit-Position: refs/heads/master@{#291014}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291014 0039d316-1c4b-4281-b951-d872f2087c98
darktears pushed a commit that referenced this pull request Sep 29, 2014
…hset #5 id:100001 of https://codereview.chromium.org/516293007/)

Reason for revert:
Chrome OS build with DCHECKs enabled fails on start. This CL breaks assumption, that GAIA extension can be loaded without performing IO-operations.

Log and stack trace:
[16275:16275:0903/161435:WARNING:renderer_freezer.cc(55)] Cgroup freezer does not exist or is not writable. Processes will not be frozen during suspend.
[16275:16275:0903/161435:WARNING:configuration_policy_pref_store.cc(30)] Policy RemoteAccessClientFirewallTraversal: This policy has been deprecated.
[16275:16275:0903/161437:FATAL:thread_restrictions.cc(38)] Function marked as IO-only was called from a thread that disallows IO!  If this thread really should be allowed to make IO calls, adjust the call to base::ThreadRestrictions::SetIOAllowed() in this thread's startup.
#0 0x7fe3122e555e base::debug::StackTrace::StackTrace()
#1 0x7fe31237c462 logging::LogMessage::~LogMessage()
#2 0x7fe3124824af base::ThreadRestrictions::AssertIOAllowed()
#3 0x7fe3123630a1 base::PathExists()
#4 0x7fe321cfd5f3 extensions::(anonymous namespace)::CollectPlatformSpecificResourceArchs()
#5 0x7fe321cfad22 extensions::Extension::InitFromValue()
#6 0x7fe321cfa490 extensions::Extension::Create()
#7 0x7fe321cfa114 extensions::Extension::Create()
#8 0x7fe3233ebe1b extensions::ComponentLoader::Load()
#9 0x7fe3233ec3cf extensions::ComponentLoader::Add()
#10 0x7fe3233ec32a extensions::ComponentLoader::Add()
#11 0x7fe3233ec2b0 extensions::ComponentLoader::Add()
#12 0x7fe32437505f (anonymous namespace)::LoadGaiaAuthExtension()
#13 0x7fe324374d5e extensions::GaiaAuthExtensionLoader::LoadIfNeeded()
#14 0x7fe324237f7e ScopedGaiaAuthExtension::ScopedGaiaAuthExtension()
#15 0x7fe322f68bb2 chromeos::WebUILoginView::Init()
#16 0x7fe322f556ef chromeos::LoginDisplayHostImpl::InitLoginWindowAndView()
#17 0x7fe322f52640 chromeos::LoginDisplayHostImpl::LoadURL()
#18 0x7fe322f5226a chromeos::LoginDisplayHostImpl::StartWizard()
#19 0x7fe322f54acd chromeos::LoginDisplayHostImpl::StartPostponedWebUI()
#20 0x7fe322f541f8 chromeos::LoginDisplayHostImpl::Observe()
#21 0x7fe322f54bfd chromeos::LoginDisplayHostImpl::Observe()
#22 0x7fe31b222377 content::NotificationServiceImpl::Notify()
#23 0x7fe322d51a24 chromeos::(anonymous namespace)::UserWallpaperDelegate::OnWallpaperAnimationFinished()
#24 0x7fe31188692f ash::RootWindowController::OnWallpaperAnimationFinished()
#25 0x7fe3117c633a ash::(anonymous namespace)::ShowWallpaperAnimationObserver::OnImplicitAnimationsCompleted()
#26 0x7fe311f72d30 ui::ImplicitAnimationObserver::CheckCompleted()
#27 0x7fe311f72cd5 ui::ImplicitAnimationObserver::SetActive()
#28 0x7fe311f965d5 ui::ScopedLayerAnimationSettings::~ScopedLayerAnimationSettings()
#29 0x7fe3117c5ec7 ash::DesktopBackgroundWidgetController::StartAnimating()
#30 0x7fe3117bbb0c ash::DesktopBackgroundController::InstallDesktopController()
#31 0x7fe3117bbc03 ash::DesktopBackgroundController::InstallDesktopControllerForAllWindows()
#32 0x7fe3117bb3cc ash::DesktopBackgroundController::SetDesktopBackgroundImageMode()
#33 0x7fe3117bb096 ash::DesktopBackgroundController::SetWallpaperImage()
#34 0x7fe322f9a6b9 chromeos::WallpaperManager::DoSetDefaultWallpaper()
#35 0x7fe322f99b73 chromeos::WallpaperManager::PendingWallpaper::ProcessRequest()
#36 0x7fe322fbac72 base::internal::RunnableAdapter<>::Run()
#37 0x7fe322fbabe9 base::internal::InvokeHelper<>::MakeItSo()
#38 0x7fe322fbaba5 base::internal::Invoker<>::Run()
#39 0x7fe3122cc9ce base::Callback<>::Run()
#40 0x7fe312488f86 base::Timer::RunScheduledTask()
#41 0x7fe3124890bc base::BaseTimerTaskInternal::Run()
#42 0x7fe312489382 base::internal::RunnableAdapter<>::Run()
#43 0x7fe3124892ec base::internal::InvokeHelper<>::MakeItSo()
#44 0x7fe312489295 base::internal::Invoker<>::Run()
#45 0x7fe3122cc9ce base::Callback<>::Run()
#46 0x7fe3122eb9b3 base::debug::TaskAnnotator::RunTask()
#47 0x7fe3123a1c57 base::MessageLoop::RunTask()
#48 0x7fe3123a1d9b base::MessageLoop::DeferOrRunPendingTask()
#49 0x7fe3123a228d base::MessageLoop::DoDelayedWork()
#50 0x7fe3122a1f25 base::MessagePumpGlib::Run()
#51 0x7fe3123a17f0 base::MessageLoop::RunHandler()
#52 0x7fe3124093b2 base::RunLoop::Run()
#53 0x7fe324217e4d ChromeBrowserMainParts::MainMessageLoopRun()
#54 0x7fe31ad2a8bf content::BrowserMainLoop::RunMainMessageLoopParts()
#55 0x7fe31ad344c7 content::BrowserMainRunnerImpl::Run()
#56 0x7fe31ad251b1 content::BrowserMain()
#57 0x7fe31abb997f content::RunNamedProcessTypeMain()
#58 0x7fe31abbcce8 content::ContentMainRunnerImpl::Run()
#59 0x7fe31abb8ee5 content::ContentMain()
#60 0x7fe3206d3505 ChromeMain
#61 0x7fe3206d34b2 main

Cannot upload crash dump: cannot exec /sbin/crash_reporter

Crash_reporter failed to process crash report

Original issue's description:
> Enable forced extension updates on NaCl arch mismatch
>
> This makes extensions aware of the platforms for which
> they have platform-specific resources installed, if any.
>
> This also hooks up the extension update code with some
> additional logic to place an extension in forced-update
> mode if it has platform-specific resources which don't
> match the current NaCl architecture.
>
> BUG=409948
> TEST=install an extension which uses NaCl (QuickOffice for example). Rename the _platform-specific/<your-nacl-arch> directory some something else and force an update (e.g. via chrome://extensions button). Observe that a new CRX is downloaded and installed.
>
> Committed: https://chromium.googlesource.com/chromium/src/+/4a92281fa5d331860d65a59ba45dc882a5c71df4

[email protected],[email protected],[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=409948

Review URL: https://codereview.chromium.org/532183003

Cr-Commit-Position: refs/heads/master@{#293128}
darktears pushed a commit that referenced this pull request Sep 29, 2014
…::ExecuteCommands. (patchset #1 id:1 of https://codereview.chromium.org/547783002/)

Reason for revert:
Broke Linux ChromiumOS Tests (dbg)(3)

http://build.chromium.org/p/chromium.webkit/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%283%29/builds/1206

BrowserTestBase signal handler received SIGTERM. Backtrace:
#0 0x7f3bcf3aa67e base::debug::StackTrace::StackTrace()
#1 0x0000041c884a content::(anonymous namespace)::DumpStackTraceSignalHandler()
#2 0x7f3bc709f4a0 <unknown>
#3 0x7f3bc7151a43 __poll
#4 0x7f3bc7b8dff6 <unknown>
#5 0x7f3bc7b8e124 g_main_context_iteration
#6 0x7f3bcf366f35 base::MessagePumpGlib::Run()
#7 0x7f3bcf468180 base::MessageLoop::RunHandler()
#8 0x7f3bcf4cfcc2 base::RunLoop::Run()
#9 0x0000041e0729 content::RunThisRunLoop()
#10 0x0000041e06ba content::RunMessageLoop()
#11 0x000000668feb ExtensionApiTest::ResultCatcher::GetNextResult()
#12 0x0000006a30dd extensions::CommandsApiTest_ContinuePropagation_Test::RunTestOnMainThread()
#13 0x0000006a3712 extensions::CommandsApiTest_ContinuePropagation_Test::RunTestOnMainThread()
#14 0x00000176be8b InProcessBrowserTest::RunTestOnMainThreadLoop()
#15 0x0000041c85c8 content::BrowserTestBase::ProxyRunTestOnMainThreadLoop()
#16 0x0000041c9cf2 base::internal::RunnableAdapter<>::Run()
#17 0x0000041c9c69 base::internal::InvokeHelper<>::MakeItSo()
#18 0x0000041c9c25 base::internal::Invoker<>::Run()
#19 0x00000063f0de base::Callback<>::Run()
#20 0x0000042dddd2 ChromeBrowserMainParts::PreMainMessageLoopRunImpl()
#21 0x0000042dca92 ChromeBrowserMainParts::PreMainMessageLoopRun()
#22 0x000002e77daf chromeos::ChromeBrowserMainPartsChromeos::PreMainMessageLoopRun()
#23 0x7f3bd7bd2e96 content::BrowserMainLoop::PreMainMessageLoopRun()
#24 0x7f3bd7bd96f2 base::internal::RunnableAdapter<>::Run()
#25 0x7f3bd7bd965c base::internal::InvokeHelper<>::MakeItSo()
#26 0x7f3bd7bd960a base::internal::Invoker<>::Run()
#27 0x7f3bd80a852e base::Callback<>::Run()
#28 0x7f3bd841bd2b content::StartupTaskRunner::RunAllTasksNow()
#29 0x7f3bd7bd12f0 content::BrowserMainLoop::CreateStartupTasks()
#30 0x7f3bd7bdcc72 content::BrowserMainRunnerImpl::Initialize()
#31 0x7f3bd7bcda55 content::BrowserMain()
#32 0x7f3bd7a6224f content::RunNamedProcessTypeMain()
#33 0x7f3bd7a655b8 content::ContentMainRunnerImpl::Run()
#34 0x7f3bd7a617b5 content::ContentMain()
#35 0x0000041c82cb content::BrowserTestBase::SetUp()
#36 0x00000176afc3 InProcessBrowserTest::SetUp()
#37 0x000000674823 ExtensionBrowserTest::SetUp()
#38 0x000000674852 ExtensionBrowserTest::SetUp()
#39 0x0000018298a3 testing::internal::HandleSehExceptionsInMethodIfSupported<>()
#40 0x00000181686e testing::internal::HandleExceptionsInMethodIfSupported<>()
#41 0x00000180ad73 testing::Test::Run()
#42 0x00000180b4cb testing::TestInfo::Run()
#43 0x00000180baca testing::TestCase::Run()
#44 0x000001811009 testing::internal::UnitTestImpl::RunAllTests()
#45 0x0000018225b3 testing::internal::HandleSehExceptionsInMethodIfSupported<>()
#46 0x0000018185fe testing::internal::HandleExceptionsInMethodIfSupported<>()
#47 0x000001810c97 testing::UnitTest::Run()
#48 0x0000041a5531 RUN_ALL_TESTS()
#49 0x0000041a4547 base::TestSuite::Run()
#50 0x0000008026d2 InteractiveUITestSuiteRunner::RunTestSuite()
#51 0x000001769618 (anonymous namespace)::ChromeTestLauncherDelegate::RunTestSuite()
#52 0x0000041d9efb content::LaunchTests()
#53 0x00000176954b LaunchChromeTests()
#54 0x00000080262f main
#55 0x7f3bc708a76d __libc_start_main
#56 0x0000005ffda9 <unknown>
[123/334] CommandsApiTest.ContinuePropagation (TIMED OUT)

Original issue's description:
> Fixes HasEventListener check in ExtensionKeybindingRegistry::ExecuteCommands.
>
> ExtensionKeybindingRegistry::ExecuteCommands functions in two distinct ways -- execute all commands based on an accelerator, or, execute an accelerator for a given extension. The former behavior is implied by passing an empty string.
>
> Previously, only the latter case was handled when trying to continue propagating keys.
>
> BUG=407163
> TEST=try bots (interactive_ui_tests --gtest_filter=CommandsApiTest.*).
>
> Committed: https://chromium.googlesource.com/chromium/src/+/189a2ed4d209231d517b0e64a722722dead3f17a

[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=407163

Review URL: https://codereview.chromium.org/552533003

Cr-Commit-Position: refs/heads/master@{#293780}
darktears pushed a commit that referenced this pull request Sep 29, 2014
…disable GCM (patchset #8 id:150001 of https://codereview.chromium.org/561943002/)

Reason for revert:
this CL might be breaking browser_tests and interactive_ui_tests on ChromeOS because it's starting a request on startup.  here's an example:

[15345:15369:0918/132152:ERROR:leak_tracker.h(97)] Leaked 0x11cd70da7cc0 which was allocated by:
[15345:15369:0918/132152:ERROR:leak_tracker.h(98)] #0 0x7f44b584025e base::debug::StackTrace::StackTrace()
#1 0x000000dfc481 base::debug::LeakTracker<>::LeakTracker()
#2 0x000000df3e8d SystemURLRequestContextGetter::SystemURLRequestContextGetter()
#3 0x000000df57f1 IOThread::InitSystemRequestContext()
#4 0x000000df570b IOThread::system_url_request_context_getter()
#5 0x000001443d9e BrowserProcessImpl::system_request_context()
#6 0x000001028a8e SafeBrowsingService::Initialize()
#7 0x0000014462c9 BrowserProcessImpl::CreateSafeBrowsingService()
#8 0x000001446196 BrowserProcessImpl::safe_browsing_service()
#9 0x0000014b9a06 ChromeResourceDispatcherHostDelegate::ChromeResourceDispatcherHostDelegate()
#10 0x0000014466dc BrowserProcessImpl::ResourceDispatcherHostCreated()
#11 0x0000008a88e5 chrome::ChromeContentBrowserClient::ResourceDispatcherHostCreated()
#12 0x7f44be489a2f content::ResourceDispatcherHostImpl::ResourceDispatcherHostImpl()
#13 0x7f44be060f44 content::BrowserMainLoop::BrowserThreadsStarted()
#14 0x7f44be068302 base::internal::RunnableAdapter<>::Run()
#15 0x7f44be06826c base::internal::InvokeHelper<>::MakeItSo()
#16 0x7f44be06821a base::internal::Invoker<>::Run()
#17 0x7f44be54e46e base::Callback<>::Run()
#18 0x7f44be8d223b content::StartupTaskRunner::RunAllTasksNow()
#19 0x7f44be05ff97 content::BrowserMainLoop::CreateStartupTasks()
#20 0x7f44be06b824 content::BrowserMainRunnerImpl::Initialize()
#21 0x7f44be05c7df content::BrowserMain()
#22 0x7f44bdee5f6f content::RunNamedProcessTypeMain()
#23 0x7f44bdee9268 content::ContentMainRunnerImpl::Run()
#24 0x7f44bdee5505 content::ContentMain()
#25 0x0000041aa8cb content::BrowserTestBase::SetUp()
#26 0x00000173e117 InProcessBrowserTest::SetUp()
#27 0x000000671180 ExtensionBrowserTest::SetUp()
#28 0x0000006711b2 ExtensionBrowserTest::SetUp()
#29 0x0000017fb993 testing::internal::HandleSehExceptionsInMethodIfSupported<>()
#30 0x0000017e8bfe testing::internal::HandleExceptionsInMethodIfSupported<>()
#31 0x0000017dd1e3 testing::Test::Run()
#32 0x0000017dd92b testing::TestInfo::Run()
#33 0x0000017ddf1a testing::TestCase::Run()
#34 0x0000017e3413 testing::internal::UnitTestImpl::RunAllTests()
#35 0x0000017f47d3 testing::internal::HandleSehExceptionsInMethodIfSupported<>()
#36 0x0000017ea96e testing::internal::HandleExceptionsInMethodIfSupported<>()
#37 0x0000017e30b1 testing::UnitTest::Run()
#38 0x000004187e41 RUN_ALL_TESTS()
#39 0x000004186e67 base::TestSuite::Run()
#40 0x0000007fcd62 InteractiveUITestSuiteRunner::RunTestSuite()
#41 0x00000173c7b8 (anonymous namespace)::ChromeTestLauncherDelegate::RunTestSuite()
#42 0x0000041bc2fb content::LaunchTests()
#43 0x00000173c6e9 LaunchChromeTests()
#44 0x0000007fccbf main
#45 0x7f44ad8ed76d __libc_start_main
#46 0x000000601389 <unknown>

[15345:15369:0918/132152:FATAL:leak_tracker.h(102)] Check failed: 0u == count (0 vs. 1)
#0 0x7f44b584025e base::debug::StackTrace::StackTrace()
#1 0x7f44b58d7a62 logging::LogMessage::~LogMessage()
#2 0x000000dfdd01 base::debug::LeakTracker<>::CheckForLeaks()
#3 0x000000df8e13 IOThread::CleanUp()
#4 0x7f44be0810b6 content::BrowserThreadImpl::CleanUp()
#5 0x7f44be07e852 content::BrowserProcessSubThread::CleanUp()
#6 0x7f44b59d0601 base::Thread::ThreadMain()
#7 0x7f44b59bba3c base::(anonymous namespace)::ThreadFunc()
#8 0x7f44b1aa9e9a start_thread
#9 0x7f44ad9c03fd clone

Original issue's description:
> Add GCMChannelStatusSyncer to schedule requests and enable/disable GCM
>
> BUG=384041
> TEST=new tests added
>
> Committed: https://crrev.com/3c23f4a188e171998f3042ad62f4aa5717e66d63
> Cr-Commit-Position: refs/heads/master@{#295524}

[email protected],[email protected],[email protected],[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=384041

Review URL: https://codereview.chromium.org/582913003

Cr-Commit-Position: refs/heads/master@{#295549}
darktears pushed a commit that referenced this pull request Sep 30, 2014
…410) (patchset #1 id:1 of https://codereview.chromium.org/590143002/)

Reason for revert:
Linux ASAN memory leaks after "Roll src/third_party/WebKit 08e97a9:0f6e289 (svn 182386:182407)"
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%282%29/builds/7724/steps/content_unittests/logs/stdio

==30366==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 34 byte(s) in 1 object(s) allocated from:
#0 0x50482b in malloc (/b/build/slave/Linux_ASan_LSan_Tests__2_/build/src/out/Release/content_unittests+0x50482b)
#1 0x58b7516 in partitionAllocGenericFlags third_party/WebKit/Source/wtf/PartitionAlloc.h:538:20
#2 0x58b7516 in partitionAllocGeneric third_party/WebKit/Source/wtf/PartitionAlloc.h:554
#3 0x58b7516 in createUninitialized third_party/WebKit/Source/wtf/text/StringImpl.cpp:299
#4 0x58b7516 in WTF::StringImpl::create(unsigned char const*, unsigned int) third_party/WebKit/Source/wtf/text/StringImpl.cpp:418
#5 0x58e809f in WTF::String::fromUTF8(unsigned char const*, unsigned long) third_party/WebKit/Source/wtf/text/WTFString.cpp:911:16
#6 0x58ffa2f in fromUTF8 third_party/WebKit/Source/wtf/text/WTFString.h:400:67
#7 0x58ffa2f in blink::WebString::fromUTF8(char const*, unsigned long) third_party/WebKit/Source/platform/exported/WebString.cpp:91
#8 0x20a09ac in WebString<23> third_party/WebKit/public/platform/WebString.h:113:16
#9 0x20a09ac in content::MockMediaStreamRegistry::AddVideoTrack(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) content/renderer/media/mock_media_stream_registry.cc:36
#10 0x217484a in VideoSourceHandlerTest_OpenWithoutClose_Test content/renderer/media/video_source_handler_unittest.cc:97:1
#11 0x217484a in testing::internal::TestFactoryImpl<content::VideoSourceHandlerTest_OpenWithoutClose_Test>::CreateTest() testing/gtest/include/gtest/internal/gtest-internal.h:486
#12 0x2f13c63 in HandleExceptionsInMethodIfSupported<testing::internal::TestFactoryBase, testing::Test *> testing/gtest/src/gtest.cc:2418:12
#13 0x2f13c63 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2601
#14 0x2f14df6 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2728:5
#15 0x2f28853 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4591:11
#16 0x2f27e54 in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2418:12
#17 0x2f27e54 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4209
#18 0x2eb1e53 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2304:10
#19 0x2eb1e53 in base::TestSuite::Run() base/test/test_suite.cc:226
#20 0x2ea717b in Run base/callback.h:401:12
#21 0x2ea717b in base::(anonymous namespace)::LaunchUnitTestsInternal(base::Callback<int ()> const&, int, bool, base::Callback<void ()> const&) base/test/launcher/unit_test_launcher.cc:498
#22 0x2ea6c70 in base::LaunchUnitTests(int, char**, base::Callback<int ()> const&) base/test/launcher/unit_test_launcher.cc:553:10
#23 0x1ff5d0c in main content/test/run_all_unittests.cc:14:10
#24 0x7f375d21876c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

Original issue's description:
> Roll src/third_party/WebKit 0f6e289:a63b92e (svn 182407:182410)
>
> Summary of changes available at:
> https://chromium.googlesource.com/chromium/blink/+log/0f6e289..a63b92e
>
> [email protected],[email protected],[email protected]
>
> Committed: https://crrev.com/09e1192db2fc7c5295eb373626b77945ca10aee3
> Cr-Commit-Position: refs/heads/master@{#295959}

[email protected],[email protected],[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/593693004

Cr-Commit-Position: refs/heads/master@{#295961}
darktears pushed a commit that referenced this pull request Sep 30, 2014
…407) (patchset #1 id:1 of https://codereview.chromium.org/588163003/)

Reason for revert:
Linux ASAN memory leaks after "Roll src/third_party/WebKit 08e97a9:0f6e289 (svn 182386:182407)"
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%282%29/builds/7724/steps/content_unittests/logs/stdio

==30366==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 34 byte(s) in 1 object(s) allocated from:
#0 0x50482b in malloc (/b/build/slave/Linux_ASan_LSan_Tests__2_/build/src/out/Release/content_unittests+0x50482b)
#1 0x58b7516 in partitionAllocGenericFlags third_party/WebKit/Source/wtf/PartitionAlloc.h:538:20
#2 0x58b7516 in partitionAllocGeneric third_party/WebKit/Source/wtf/PartitionAlloc.h:554
#3 0x58b7516 in createUninitialized third_party/WebKit/Source/wtf/text/StringImpl.cpp:299
#4 0x58b7516 in WTF::StringImpl::create(unsigned char const*, unsigned int) third_party/WebKit/Source/wtf/text/StringImpl.cpp:418
#5 0x58e809f in WTF::String::fromUTF8(unsigned char const*, unsigned long) third_party/WebKit/Source/wtf/text/WTFString.cpp:911:16
#6 0x58ffa2f in fromUTF8 third_party/WebKit/Source/wtf/text/WTFString.h:400:67
#7 0x58ffa2f in blink::WebString::fromUTF8(char const*, unsigned long) third_party/WebKit/Source/platform/exported/WebString.cpp:91
#8 0x20a09ac in WebString<23> third_party/WebKit/public/platform/WebString.h:113:16
#9 0x20a09ac in content::MockMediaStreamRegistry::AddVideoTrack(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) content/renderer/media/mock_media_stream_registry.cc:36
#10 0x217484a in VideoSourceHandlerTest_OpenWithoutClose_Test content/renderer/media/video_source_handler_unittest.cc:97:1
#11 0x217484a in testing::internal::TestFactoryImpl<content::VideoSourceHandlerTest_OpenWithoutClose_Test>::CreateTest() testing/gtest/include/gtest/internal/gtest-internal.h:486
#12 0x2f13c63 in HandleExceptionsInMethodIfSupported<testing::internal::TestFactoryBase, testing::Test *> testing/gtest/src/gtest.cc:2418:12
#13 0x2f13c63 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2601
#14 0x2f14df6 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2728:5
#15 0x2f28853 in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4591:11
#16 0x2f27e54 in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2418:12
#17 0x2f27e54 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4209
#18 0x2eb1e53 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2304:10
#19 0x2eb1e53 in base::TestSuite::Run() base/test/test_suite.cc:226
#20 0x2ea717b in Run base/callback.h:401:12
#21 0x2ea717b in base::(anonymous namespace)::LaunchUnitTestsInternal(base::Callback<int ()> const&, int, bool, base::Callback<void ()> const&) base/test/launcher/unit_test_launcher.cc:498
#22 0x2ea6c70 in base::LaunchUnitTests(int, char**, base::Callback<int ()> const&) base/test/launcher/unit_test_launcher.cc:553:10
#23 0x1ff5d0c in main content/test/run_all_unittests.cc:14:10
#24 0x7f375d21876c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226

Original issue's description:
> Roll src/third_party/WebKit 08e97a9:0f6e289 (svn 182386:182407)
>
> Summary of changes available at:
> https://chromium.googlesource.com/chromium/blink/+log/08e97a9..0f6e289
>
> [email protected],[email protected],[email protected]
>
> Committed: https://crrev.com/f57e3cf2113ac892f16777e5ab225f3c68b94ede
> Cr-Commit-Position: refs/heads/master@{#295950}

[email protected],[email protected],[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/592783002

Cr-Commit-Position: refs/heads/master@{#295962}
rakuco pushed a commit that referenced this pull request Oct 10, 2014
VideoEncodeAccelerator::GetSupportedProfiles now runs in GPU
process because the profiles were added to GPUInfo. Add
kEnableWebRtcHWVp8Encoding to GPU process to fix the missing
encode profile.

BUG=418762
TEST=Run apprtc loopback and check the histogram.
[email protected]

Review URL: https://codereview.chromium.org/618633002

Cr-Commit-Position: refs/heads/master@{#297445}
(cherry picked from commit ebd596b)

Review URL: https://codereview.chromium.org/620963002

Cr-Commit-Position: refs/branch-heads/2171@{#24}
Cr-Branched-From: 267aeeb-refs/heads/master@{#297060}
rakuco pushed a commit that referenced this pull request Jan 26, 2015
Cr-Commit-Position: refs/branch-heads/2272@{#24}
Cr-Branched-From: 827a380-refs/heads/master@{#310958}
rakuco pushed a commit that referenced this pull request Mar 2, 2015
Return the y coordinate of the PopupTouchHandleDrawable position when
it is requested. Turns out, returning the x coordinate is bad.

BUG=461598

Review URL: https://codereview.chromium.org/931283008

Cr-Commit-Position: refs/heads/master@{#317936}
(cherry picked from commit 048b41f)

Review URL: https://codereview.chromium.org/952263002

Cr-Commit-Position: refs/branch-heads/2311@{#24}
Cr-Branched-From: 09b7de5-refs/heads/master@{#317474}
rakuco pushed a commit that referenced this pull request Jun 8, 2015
Cr-Commit-Position: refs/branch-heads/2403@{#24}
Cr-Branched-From: f54b809-refs/heads/master@{#330231}
rakuco pushed a commit that referenced this pull request Jul 14, 2015
…leveldb::DestroyDB

When a database directory contains files not related to LevelDB, DestroyDB()
does not delete the directory and leaves those files. This non-empty database
directory infinitely fails ServiceWorkerDatabase::LazyOpen during a session.
(See my comment #24 in the issue for more details).

To avoid that, this CL directly deletes the directory instead of DestroyDB.

BUG=468926
TEST=content_unittests --gtest_filter=ServiceWorkerResourceStorageDiskTest.DeleteAndStartOver*
[email protected]

Review URL: https://codereview.chromium.org/1176203007

Cr-Commit-Position: refs/heads/master@{#335893}
(cherry picked from commit 0bd5fd7)

Review URL: https://codereview.chromium.org/1214793006.

Cr-Commit-Position: refs/branch-heads/2403@{#413}
Cr-Branched-From: f54b809-refs/heads/master@{#330231}
smishenk pushed a commit to smishenk/chromium-crosswalk that referenced this pull request Apr 29, 2016
… receives."

This reverts commit 8cf7497.

The change depends on a change in webrtc that's not in M49 webrtc branch.
Reverting it to make M49 chromium branch build with M49 webrtc..

BUG=578714
[email protected]

Review URL: https://codereview.chromium.org/1607993005 .

Cr-Commit-Position: refs/branch-heads/2623@{crosswalk-project#24}
Cr-Branched-From: 92d7753-refs/heads/master@{#369907}
huningxin pushed a commit to huningxin/chromium-croswalk that referenced this pull request May 17, 2016
…data.

If the user clears their state, we should presume that they're signed out
of websites as well. This patch sets 'skip_zero_click' accordingly by
wiring a new method on 'PasswordStore' to the 'BrowsingDataRemover' on
one end, and all the password store backends on the other.

BUG=589779
[email protected], [email protected], [email protected]

Review URL: https://codereview.chromium.org/1732723005

Cr-Commit-Position: refs/heads/master@{#378211}
(cherry picked from commit 6fd6144)

Review URL: https://codereview.chromium.org/1750363002 .

Cr-Commit-Position: refs/branch-heads/2661@{crosswalk-project#24}
Cr-Branched-From: ef6f6ae-refs/heads/master@{#378081}
fujunwei pushed a commit to fujunwei/chromium-crosswalk that referenced this pull request May 27, 2016
Due to a regression, the bottom of the Content was not always
displayed properly. The problem was updateTopControlsState()
method not being called properly once the Content starts
loading.

BUG=554842

Review URL: https://codereview.chromium.org/1452913002

Cr-Commit-Position: refs/heads/master@{#359969}
(cherry picked from commit 85e6acc)

Review URL: https://codereview.chromium.org/1455803004 .

Cr-Commit-Position: refs/branch-heads/2564@{crosswalk-project#24}
Cr-Branched-From: 1283eca-refs/heads/master@{#359700}
huningxin pushed a commit to huningxin/chromium-croswalk that referenced this pull request Oct 9, 2016
Intent-To-Ship thread: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/0S-jPuCy8ws
BUG=564479, 596832

Review-Url: https://codereview.chromium.org/1993033003
Cr-Commit-Position: refs/heads/master@{#395223}
(cherry picked from commit e0cb6a3)

Review URL: https://codereview.chromium.org/2008673003 .

Cr-Commit-Position: refs/branch-heads/2743@{crosswalk-project#24}
Cr-Branched-From: 2b3ae3b-refs/heads/master@{#394939}
imreotto pushed a commit to tenta-browser/chromium-crosswalk that referenced this pull request Nov 2, 2017
Some devices don't support Device Orientation events. These are required
in order for MediaControlsOrientationLockDelegate to unlock the screen
orientation after rotate-to-fullscreen enters fullscreen, so without
Device Orientation events you can rotate-to-enter-fullscreen but not
rotate-to-exit-fullscreen.

This is a poor user experience, so this patch disables
rotate-to-fullscreen completely on devices that don't support Device
Orientation events.

Also fixes an uninitialized member variable in DeviceOrientationData.

[email protected]

(cherry picked from commit 48a2ccc)

Bug: 760737
Change-Id: Ic328cf548eb672eab4f030990f5d01f0a2ebca40
Reviewed-on: https://chromium-review.googlesource.com/645981
Commit-Queue: John Mellor <[email protected]>
Reviewed-by: Mounir Lamouri <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#499219}
Reviewed-on: https://chromium-review.googlesource.com/650727
Reviewed-by: John Mellor <[email protected]>
Cr-Commit-Position: refs/branch-heads/3202@{crosswalk-project#24}
Cr-Branched-From: fa6a5d8-refs/heads/master@{#499098}
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants