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

Issue 295833002: Revert 271461 "Decouple RVH creation from CrossProcessFrameConne..." (Closed)

Created:
6 years, 7 months ago by ericu
Modified:
6 years, 7 months ago
Reviewers:
nasko
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 271461 "Decouple RVH creation from CrossProcessFrameConne..." This broke the same android tests that failed 2 out of 3 times on your presubmit try jobs. > Decouple RVH creation from CrossProcessFrameConnector. > > The RenderViewHost creation today takes a CrossProcessFrameConnector pointer, which it uses to decide whether to create a top-level or child frame view. Replace this with a boolean and associate the CrossProcessFrameConnector with the view at a later point in time. > > BUG=357747 > > Review URL: https://codereview.chromium.org/270883003 TBR=nasko@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271516

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -65 lines) Patch
M trunk/src/content/browser/frame_host/cross_process_frame_connector.h View 1 chunk +0 lines, -3 lines 0 comments Download
M trunk/src/content/browser/frame_host/cross_process_frame_connector.cc View 3 chunks +15 lines, -24 lines 0 comments Download
M trunk/src/content/browser/frame_host/render_frame_host_manager.h View 2 chunks +3 lines, -6 lines 0 comments Download
M trunk/src/content/browser/frame_host/render_frame_host_manager.cc View 9 chunks +7 lines, -27 lines 0 comments Download
M trunk/src/content/browser/web_contents/web_contents_impl.h View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/content/browser/web_contents/web_contents_impl.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M trunk/src/content/test/test_web_contents.h View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/content/test/test_web_contents.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
ericu
6 years, 7 months ago (2014-05-19 23:47:22 UTC) #1
ericu
Committed patchset #1 manually as r271516.
6 years, 7 months ago (2014-05-19 23:47:35 UTC) #2
boliu
6 years, 7 months ago (2014-05-20 00:18:03 UTC) #3
Message was sent while issue was closed.
Stack from bots below. It's not immediately obvious what's wrong, but I'm
looking:

Stack Trace:
  RELADDR   FUNCTION                                                            
                                                                                
                                                                                
                                                                                
                                      FILE:LINE
  002e26f8  JavaObjectWeakGlobalRef::get(_JNIEnv*) const+8                      
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  00854973  content::ContentViewCoreImpl::UpdateFrameInfo(gfx::Vector2dF const&,
float, gfx::Vector2dF const&, gfx::SizeF const&, gfx::SizeF const&,
gfx::Vector2dF const&, gfx::Vector2dF const&, float)+62                         
                                                                                
                                                   libgcc2.c:0
  0098675d 
content::RenderWidgetHostViewAndroid::OnFrameMetadataUpdated(cc::CompositorFrameMetadata
const&)+120                                                                     
                                                                                
                                                                                
                  libgcc2.c:0
  0098934b 
content::RenderWidgetHostViewAndroid::SynchronousFrameMetadata(cc::CompositorFrameMetadata
const&)+10                                                                      
                                                                                
                                                                                
                libgcc2.c:0
  0085b7cf 
content::SynchronousCompositorOutputSurface::UpdateFrameMetaData(cc::CompositorFrameMetadata
const&)+382                                                                     
                                                                                
                                                                                
              libgcc2.c:0
  0085b8b1 
content::SynchronousCompositorOutputSurface::SwapBuffers(cc::CompositorFrame*)+172
                                                                                
                                                                                
                                                                                
                        libgcc2.c:0
  0027f619  cc::SoftwareRenderer::SwapBuffers(cc::CompositorFrameMetadata
const&)+248                                                                     
                                                                                
                                                                                
                                             libgcc2.c:0
  002bf72d  cc::LayerTreeHostImpl::SwapBuffers(cc::LayerTreeHostImpl::FrameData
const&)+236                                                                     
                                                                                
                                                                                
                                       libgcc2.c:0
  002d4bbd  cc::ThreadProxy::DrawSwapInternal(bool)+588                         
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  002d4f53  cc::ThreadProxy::ScheduledActionDrawAndSwapIfPossible()+170         
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  002af715  cc::Scheduler::DrawAndSwapIfPossible()+10                           
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  002b0e13  cc::Scheduler::ProcessScheduledActions()+266                        
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  002b133d  cc::Scheduler::OnBeginImplFrameDeadline()+112                       
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  002b137d  cc::Scheduler::ScheduleBeginImplFrameDeadline(base::TimeTicks)+16   
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  002b1605  cc::Scheduler::BeginImplFrame(cc::BeginFrameArgs const&)+524        
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  002b1abb  cc::Scheduler::BeginFrame(cc::BeginFrameArgs const&)+278            
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  002be8af  cc::LayerTreeHostImpl::BeginFrame(cc::BeginFrameArgs const&)+8      
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  0085b2b9 
content::SynchronousCompositorOutputSurface::InvokeComposite(gfx::Transform
const&, gfx::Rect, gfx::Rect, bool)+240                                         
                                                                                
                                                                                
                               libgcc2.c:0
  0085b567 
content::SynchronousCompositorOutputSurface::DemandDrawSw(SkCanvas*)+266        
                                                                                
                                                                                
                                                                                
                          libgcc2.c:0
  0085a21b  content::SynchronousCompositorImpl::DemandDrawSw(SkCanvas*)+98      
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  00209d35  android_webview::BrowserViewRenderer::CompositeSW(SkCanvas*)+72     
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  0020adb7  android_webview::BrowserViewRenderer::ForceFakeCompositeSW()+130    
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0
  0020b00b  android_webview::BrowserViewRenderer::FallbackTickFired()+170       
                                                                                
                                                                                
                                                                                
                                      libgcc2.c:0

Powered by Google App Engine
This is Rietveld 408576698