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

Side by Side Diff: content/browser/android/web_contents_observer_android.h

Issue 266053003: Remove page IDs from three WebContentsObserver APIs that don't need it. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge with Avi's change Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_ANDROID_WEB_CONTENTS_OBSERVER_ANDROID_H_ 5 #ifndef CONTENT_BROWSER_ANDROID_WEB_CONTENTS_OBSERVER_ANDROID_H_
6 #define CONTENT_BROWSER_ANDROID_WEB_CONTENTS_OBSERVER_ANDROID_H_ 6 #define CONTENT_BROWSER_ANDROID_WEB_CONTENTS_OBSERVER_ANDROID_H_
7 7
8 #include <jni.h> 8 #include <jni.h>
9 9
10 #include "base/android/jni_weak_ref.h" 10 #include "base/android/jni_weak_ref.h"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 virtual void DidFailLoad(int64 frame_id, 47 virtual void DidFailLoad(int64 frame_id,
48 const GURL& validated_url, 48 const GURL& validated_url,
49 bool is_main_frame, 49 bool is_main_frame,
50 int error_code, 50 int error_code,
51 const base::string16& error_description, 51 const base::string16& error_description,
52 RenderViewHost* render_view_host) OVERRIDE; 52 RenderViewHost* render_view_host) OVERRIDE;
53 virtual void DidNavigateMainFrame(const LoadCommittedDetails& details, 53 virtual void DidNavigateMainFrame(const LoadCommittedDetails& details,
54 const FrameNavigateParams& params) OVERRIDE; 54 const FrameNavigateParams& params) OVERRIDE;
55 virtual void DidNavigateAnyFrame(const LoadCommittedDetails& details, 55 virtual void DidNavigateAnyFrame(const LoadCommittedDetails& details,
56 const FrameNavigateParams& params) OVERRIDE; 56 const FrameNavigateParams& params) OVERRIDE;
57 virtual void DidFirstVisuallyNonEmptyPaint(int32 page_id) OVERRIDE; 57 virtual void DidFirstVisuallyNonEmptyPaint() OVERRIDE;
58 virtual void DidStartProvisionalLoadForFrame( 58 virtual void DidStartProvisionalLoadForFrame(
59 int64 frame_id, 59 int64 frame_id,
60 int64 parent_frame_id, 60 int64 parent_frame_id,
61 bool is_main_frame, 61 bool is_main_frame,
62 const GURL& validated_url, 62 const GURL& validated_url,
63 bool is_error_page, 63 bool is_error_page,
64 bool is_iframe_srcdoc, 64 bool is_iframe_srcdoc,
65 RenderViewHost* render_view_host) OVERRIDE; 65 RenderViewHost* render_view_host) OVERRIDE;
66 virtual void DidCommitProvisionalLoadForFrame( 66 virtual void DidCommitProvisionalLoadForFrame(
67 int64 frame_id, 67 int64 frame_id,
(...skipping 21 matching lines...) Expand all
89 89
90 JavaObjectWeakGlobalRef weak_java_observer_; 90 JavaObjectWeakGlobalRef weak_java_observer_;
91 91
92 DISALLOW_COPY_AND_ASSIGN(WebContentsObserverAndroid); 92 DISALLOW_COPY_AND_ASSIGN(WebContentsObserverAndroid);
93 }; 93 };
94 94
95 bool RegisterWebContentsObserverAndroid(JNIEnv* env); 95 bool RegisterWebContentsObserverAndroid(JNIEnv* env);
96 } // namespace content 96 } // namespace content
97 97
98 #endif // CONTENT_BROWSER_ANDROID_WEB_CONTENTS_OBSERVER_ANDROID_H_ 98 #endif // CONTENT_BROWSER_ANDROID_WEB_CONTENTS_OBSERVER_ANDROID_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698